Review Request: Add textbox to calendar to copy paste current date

Beat Wolf asraniel at fryx.ch
Thu Dec 4 14:37:25 CET 2008


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

Review request for Plasma.


Summary
-------

This patch brings back the possibility to copy paste the current selected date out of the calendar.
I also had to adapt the scaling, because the textbox uses vertical space, so the calendar had to use more horizontal space.
If you resize the calendar really badly, you will see something like in the second screenshot. But this can't be fixed before 4.3.
No new string are needed for this, and it brings back a feature which was lost since 4.1, hope this can get in even with feature freeze.

Here it how it looks:
http://www.fryx.ch/Asraniel/calendar1.png
Scaling issue because the way the calendar draws the numbers (they are images):
http://www.fryx.ch/Asraniel/calendar.png


Diffs
-----

  trunk/KDE/kdebase/workspace/libs/libplasmaclock/calendar.h
  trunk/KDE/kdebase/workspace/libs/libplasmaclock/calendar.cpp
  trunk/KDE/kdebase/workspace/libs/libplasmaclock/calendartable.cpp

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


Testing
-------


Thanks,

Beat



More information about the Plasma-devel mailing list