Review Request: Scrollbars for the comic applet

Matthias Fuchs mat69 at gmx.net
Mon Dec 15 19:54:40 CET 2008


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

(Updated 2008-12-15 10:54:40.133333)


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



More information about the Plasma-devel mailing list