Bug 106280 - khtml adds accesskeys not declared in HTML source
Summary: khtml adds accesskeys not declared in HTML source
Status: RESOLVED WORKSFORME
Alias: None
Product: konqueror
Classification: Applications
Component: khtml (other bugs)
Version First Reported In: unspecified
Platform: Compiled Sources Linux
: NOR wishlist
Target Milestone: ---
Assignee: Konqueror Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-05-25 17:29 UTC by Andreas Leuner
Modified: 2005-05-28 10:34 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:

Attachments
Testcase for khtml-added access keys (3.25 KB, text/html)
2005-05-25 17:36 UTC, Andreas Leuner 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Andreas Leuner 2005-05-25 17:29:08 UTC 
Version:            (using KDE Devel)
Installed from:    Compiled sources

I have created a patch for the man kioslave which makes it provide access keys for the sections list, for the section lists and for the page indicating there were several manpages for a command. See Bug #99384.
It generates html output like that of the page I'll attach shortly. If you try the access keys there you'll find ones that weren't assigned in the HTML source code.

This looks really bad in a manpage section list.
Comment 1 Maksim Orlovich 2005-05-25 17:32:51 UTC 
That's intentional
Comment 2 Andreas Leuner 2005-05-25 17:36:14 UTC 
Created attachment 11197 [details]
Testcase for khtml-added access keys

This is the output of a modified manpage kioslave. Except a missing doctype
declaration it's valid html (other than what's normally brought by kio_man).

As you can see from the html source, there is no manpage starting with e.g.
"p". Nevertheless, "p" shows up as an access key.

Is that a bug or a feature?
Comment 3 Andreas Leuner 2005-05-25 17:38:26 UTC 
@Maksim
Ah, I see. How does that work?
Comment 4 Maksim Orlovich 2005-05-25 17:54:14 UTC 
http://lists.kde.org/?l=kfm-devel&m=111296590628785&w=2
Comment 5 Andreas Leuner 2005-05-28 10:34:37 UTC 
Well I think this thing is quite reasonable so I'm closing this BR