filipf added a comment.

  OK so down to only one hack now. If possible I'd really like to keep it 
because it works in 100% of the cases I tested it with and results in better 
visuals; don't know how to fix on the C++ side of things.
  
  Couple of things that might be worthy of being discussed:
  
  1. changes to the layout of the text - @ngraham already had some suggestions, 
@abetts do you have some other ideas?
  2. add a dummy rectangle that would show up when there is no previewImage and 
that would say "Image preview not available"?

REPOSITORY
  R123 SDDM Configuration Panel (KCM)

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

To: filipf, #plasma, #vdg, ngraham
Cc: mmustac, davidedmundson, abetts, rooty, plasma-devel, jraleigh, GB_2, 
ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol, 
mart

Reply via email to