Bug 121144

Summary:	short keys activated by CTRL: Many duplicates
Product:	[Applications] konqueror	Reporter:	Wolfgang Rohdewald <wolfgang>
Component:	khtml	Assignee:	Konqueror Bugs <konqueror-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	l.lunak
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	unspecified
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description Wolfgang Rohdewald 2006-02-01 11:52:33 UTC

Version:           3.5.1 (using KDE 3.5.1, Debian Package 4:3.5.1-1 (testing/unstable))
Compiler:          Target: i486-linux-gnu
OS:                Linux (i686) release 2.6.15

duplicates are generated if the underlying link is identical. On www.sbroker.de, many links say "javascript: void(0);". They all get the same short key although they do very different things.

If konqueror cannot differentiate them by looking at the script, it would be better to define a short key only for the first of several javascript calls.

Comment 1 Lubos Lunak 2006-02-07 14:50:25 UTC

SVN commit 506683 by lunakl:

Never consider two javascript links to be the same.
BUG: 121144



 M  +1 -1      khtmlview.cpp  


--- branches/KDE/3.5/kdelibs/khtml/khtmlview.cpp #506682:506683
@@ -2640,7 +2640,7 @@
             QString url = (*it).url;
             it = data.remove( it );
             // assign the same accesskey also to other elements pointing to the same url
-            if( !url.isEmpty()) {
+            if( !url.isEmpty() && !url.startsWith( "javascript:", false )) {
                 for( QValueList< AccessKeyData >::Iterator it2 = data.begin();
                      it2 != data.end();
                      ) {