-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/980/
-----------------------------------------------------------
(Updated 2009-07-18 17:03:45.122036)


Review request for Plasma.


Changes
-------

Here's a new patch.

I don't know why, but the magic difference between the height of the M and the 
x somehow seems to be the only thing that works all the time...

So in this patch, I'm removing the part forcing the text rectangle to (0, 0), 
which solves the problem with text being cut off for narrow task items, but 
does not change the normal behavior.


Summary
-------

This patch properly aligns text to the drawable rectangle, thus preventing any 
part of the text from being cut off if there is still space available.

It removes two things:

- Code that sets the text at (0, 0) if the "text" is larger than the rectangle 
(which is not correct since there is additional interline spacing added to top 
and bottom which need not be shown)

- The part where it for some strange reason lowers all the text by the 
difference between the height of the 'M' and the 'x', thus pushing it out of 
the drawable area.


This addresses bug 178776.
    https://bugs.kde.org/show_bug.cgi?id=178776


Diffs (updated)
-----

  /trunk/KDE/kdebase/workspace/plasma/applets/tasks/abstracttaskitem.cpp 987462 

Diff: http://reviewboard.kde.org/r/980/diff


Testing
-------

Compiled it on my machine - it works well with task manager of any size.


Screenshots (updated)
-----------

Different sized taskbars with patch 2.
  http://reviewboard.kde.org/r/980/s/150/


Thanks,

Alec

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

Reply via email to