davidedmundson added a comment.

  This patch breaks the "Defaults" button as it will reset the index, but 
you'll still have your custom changes.
  
  I don't see how this patch solves anything, previously you could edit a 
global scheme and the workflow was just as easy; you click Edit, the only 
difference is you had to give it a new name.

INLINE COMMENTS

> colorscm.cpp:164
>              schemeRemoveButton->setEnabled(false);
>              schemeEditButton->setEnabled(false);
>  

This patch doesn't actually address the actual issue that I know you were 
trying to fix - this line.

REPOSITORY
  R119 Plasma Desktop

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

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

Reply via email to