Review Request: Adding Shortcut F2 and Delete also F5 on Project Manager View

Przemek Czekaj xcojack at gmail.com
Tue Oct 16 19:40:35 UTC 2012


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

(Updated Oct. 16, 2012, 7:40 p.m.)


Review request for KDevelop.


Changes
-------

Beacause I have no idea how to solve F5 issue on file tree, i decided to remove this feature from the code, and update the diff to actual version of code from master. Got an suggestion on irc:

[16:48:33] <apaku|work> cojack: IIRC thats a bug with the katepart, the same happens for one or two other shortcuts occupied by katepart. It should work if you close all editor tabs.

And apaku have right, if everything is closed, F5 on file tree worsk as well, but I can't force it to work other way.

Regards.


Description
-------

It's a feature request to add to kdevelop, pressing F2 in the Project Tree Widget on the folder or file, will trigger the rename window, pressing Delete will trigger remove dialog, pressing F5 should trigger reload but doesn't get the focus on the widget :(

I do it, becasue a lot of editors have this shortcuts by default on the project tree view fe: eclipse, netbeans, jetbrains editors etc...

Regards.


Diffs (updated)
-----

  plugins/projectmanagerview/projectmanagerviewplugin.cpp 601dbc4 

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


Testing
-------

compiled and tested, but the problem is with F5 on the Project Tree, it doesn't get the focus on this widget, just refresh the file view widget (have no idea how should it be writed... so Im very sorry)


Screenshots
-----------

Context menu, show what patch add
  http://git.reviewboard.kde.org/r/106374/s/720/


Thanks,

Przemek Czekaj

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20121016/9761cd43/attachment.html>


More information about the KDevelop-devel mailing list