filipf added a comment.

  In general we do need an additional message box which says sync successful or 
failed. And then in the case of failure it should state what failed.
  
  But as far as I can tell the operations won't fail. They're all conditional 
on values being existent or not and if things aren't in order they won't get 
carried out.
  
  I agree with points 2 and 3 as well, but would ask if we could implement it 
all gradually as working with multiple branches is already getting a bit clumsy.
  
  Implementation wise, point 2 is something Plasma elders would understand 
better atm.
  
  As for point 3, I've looked it additionally and it does complicate things but 
might be doable. The question is how to interact with the user. We could copy 
everything to a global directory and then remove it from the user directory (to 
avoid duplicates in kcms). What sucks is that users could no longer easily 
remove these theme files via kcms. So what if we the remove button worked with 
globally installed? Currently I'm not sure of the repercussions for stuff 
installed via package manager.

REPOSITORY
  R123 SDDM Configuration Panel (KCM)

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

To: filipf, #plasma, ngraham, davidedmundson, #vdg
Cc: cfeck, GB_2, ndavis, plasma-devel, LeGast00n, jraleigh, fbampaloukas, 
ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart

Reply via email to