By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.
Bug 1689 - Can not permanently change the default shortcut for 'Create Folder' in Konqueror
Summary: Can not permanently change the default shortcut for 'Create Folder' in Konqueror
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: tdebase (show other bugs)
Version: R14.0.0 [Trinity]
Hardware: All Linux
: P5 normal
Assignee: Michele Calgaro
URL:
Depends on:
Blocks:
 
Reported: 2013-10-28 04:29 CDT by Michele Calgaro
Modified: 2013-11-08 18:40 CST (History)
3 users (show)

See Also:
Compiler Version:
TDE Version String:
Application Version:
Application Name:

Attachments
tdebase konqueror patch for action shortcuts (4.02 KB, patch)
2013-11-07 02:00 CST, Michele Calgaro 		Details | Diff
Multiplication of menu items (52.98 KB, image/png)
2013-11-07 20:00 CST, Slávek Banko 		Details
New patch without the problem of menu item replication (4.72 KB, patch)
2013-11-08 08:38 CST, Michele Calgaro 		Details | Diff
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Michele Calgaro 2013-10-28 04:29:17 CDT 
In Konqueror I can not permanently change the default shortcut for 'Create Folder'. Even if I change it, after closing and reopening Konqueror, the shortcut is back to F10.
The same shortcut associated with another option (es. Activate tab 1) is preserved when closing and opening Konqueror again.
Perhaps there may be more options that are not permanently changeble
Comment 1 Michele Calgaro 2013-10-28 05:10:02 CDT 
As a reminder (to myself) for future analysis, search code for

m_paNewDir = new TDEAction( i18n("Create Folder..." ), Key_F10

in konq_mainwindow.cc
Comment 2 Michele Calgaro 2013-11-07 01:38:49 CST 
This also extends to the actions "Copy files..." and "Move files..."
Comment 3 Michele Calgaro 2013-11-07 02:00:06 CST 
Created attachment 1595 [details]
tdebase konqueror patch for action shortcuts

Proposed patch that fixes the problem for "Create folder", "Copy Files" and "Move files" shortcuts
Comment 4 Slávek Banko 2013-11-07 20:00:03 CST 
Created attachment 1605 [details]
Multiplication of menu items

The patch needs to fix! After several view mode switching menu appears, as shown in the attached picture.

Sorry for the Czech, but I assume that the problem is still clearly visible :)
Comment 5 Michele Calgaro 2013-11-08 01:30:40 CST 
(In reply to comment #4)
> Created attachment 1605 [details]
> Multiplication of menu items
> The patch needs to fix! After several view mode switching menu appears, as
> shown in the attached picture. 
> Sorry for the Czech, but I assume that the problem is still clearly visible :)

Interesting behavior. Obviously there is something I missed :)
I will have a new look at this and revert, probably during the weekend.
Comment 6 Michele Calgaro 2013-11-08 08:01:02 CST 
(In reply to comment #5)
> (In reply to comment #4)
> > Created attachment 1605 [details] [details]
> > Multiplication of menu items
> > The patch needs to fix! After several view mode switching menu appears, as
> > shown in the attached picture. 
> > Sorry for the Czech, but I assume that the problem is still clearly visible :)

I can reproduce the problem also on my computer. I will work on this.
Comment 7 Michele Calgaro 2013-11-08 08:38:00 CST 
Created attachment 1607 [details]
New patch without the problem of menu item replication

Reworked patch. I tested locally and I have no more any menu item replication.
Comment 8 Slávek Banko 2013-11-08 18:40:22 CST 
Great, thank you.
Pushed to GIT in hash 6525a4c1.