ngraham added a comment.

  In D23279#514843 <https://phabricator.kde.org/D23279#514843>, @davidedmundson 
wrote:
  
  > With this patch how would someone create the first new session?
  
  
  With Kickoff/kicker/dash/Krunner > Switch User, which goes directly to the 
session management screen.
  
  The comment in the code makes it clear that the current behavior of showing 
the button is a bug. If we decide that we like that behavior and do want it to 
be always shown, then it will be inappropriately invisible if 
`sessionsModel.showNewSessionEntry` is ever set to `false`, so something still 
needs to be fixed.
  
  Seems to me that we have two options:
  
  - Land this patch
  - Remove the `visible:` check entirely

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D23279

To: ngraham, mart, #plasma
Cc: davidedmundson, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, 
fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

Reply via email to