-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.vidsolbach.de/r/310/
-----------------------------------------------------------

Review request for Plasma.


Summary
-------

This patch adds scrollbars if "Scale to content" [1] has been chosen while the 
comic applet was not assigned [2] enough space.

I think this is a bugfix as some comics [3] can not be shown at their original 
and thus most readable size as they are too large for many displays around. 
This gets worse for small displays.

All that would be fixed with the scrollbars, yet I'm not sure if I did 
implement them good enough.


[1] showing them at their original size
[2] manualy resizing by the user
[3] xkcd, Deo Ignito, Questionable Content, Girl Genius to name a few are 
sometimes or allways very large (height)


This addresses bug 171035.


Diffs
-----

  /trunk/KDE/kdeplasma-addons/applets/comic/comic.h
  /trunk/KDE/kdeplasma-addons/applets/comic/comic.cpp

Diff: http://reviewboard.vidsolbach.de/r/310/diff


Testing
-------

Tested that with trunk. Worked fine here -- some glitches left like 
automatically adding/removing scrollbars if the applet is resized.


Thanks,

Matthias

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to