colomar added a comment.

  UI-wise, I'd do it differently.
  Currently, the description of what the checkbox affects and the checkbox 
itself are too far separated, plus it makes it sound like the layout is going 
to be changed even if the checkbox is not checked. Instead what I'd do is, 
below the general description paragraph (above the actual selection), put
  
  [  ] Change the workspace layout to the one in the selected theme
  
  **unchecked** by default, and if it's checked, show directly below
  "Warning: When clicking "Apply", your current workspace layout (including all 
panels and desktop widgets) will be irrevocably lost and changed to the one 
defined in the Look & Feel theme."

REPOSITORY
  rPLASMADESKTOP Plasma Desktop

BRANCH
  arcpatch-D2448_1

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: mart, davidedmundson, #plasma
Cc: colomar, davidedmundson, plasma-devel, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas

Reply via email to