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

Summary:	Can not permanently change the default shortcut for 'Create Folder' in Konqueror
Product:	TDE	Reporter:	Michele Calgaro <michele.calgaro>
Component:	tdebase	Assignee:	Michele Calgaro <michele.calgaro>
Status:	RESOLVED FIXED
Severity:	normal	CC:	bugwatch, michele.calgaro, slavek.banko
Priority:	P5
Version:	R14.0.0 [Trinity]
Hardware:	All
OS:	Linux
Compiler Version:		TDE Version String:
Application Version:		Application Name:
Attachments:	tdebase konqueror patch for action shortcuts Multiplication of menu items New patch without the problem of menu item replication

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.