Review Request: Handle focus in KUrlNavigator

José Millán Soto fid at gpul.org
Wed Aug 17 18:37:27 BST 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102345/
-----------------------------------------------------------

(Updated Aug. 17, 2011, 5:37 p.m.)


Review request for kdelibs.


Changes
-------

This new version of the patch uses drawTest with the flag to handle accelerators.


Summary
-------

This patch makes KUrlNavigator focusable.
Also, QStyle::drawControl is used instead of QPainter::drawText in KUrlNavigatorButton, because accelerators are set in the buttons and drawText did not display them correctly.


Diffs (updated)
-----

  kfile/kurlnavigatorbuttonbase.cpp 45f8eee 
  kfile/kurlnavigatorbuttonbase_p.h 70aacb3 
  kfile/kurlnavigator.cpp e71c979 
  kfile/kurlnavigatorbutton.cpp 5d38e56 

Diff: http://git.reviewboard.kde.org/r/102345/diff


Testing
-------


Thanks,

José

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20110817/a911dc36/attachment.htm>


More information about the kde-core-devel mailing list