[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kfm-devel
Subject:    Re: [PATCH] konqueror keyboard link navigation
From:       Robert Ewald <robewald () gmx ! net>
Date:       2004-12-01 23:11:38
Message-ID: 200412020011.38793.robewald () gmx ! net
[Download RAW message or body]

On Wednesday 01 December 2004 18:10, Lubos Lunak wrote:
> On Wednesday 01 of December 2004 15:18, Robert Ewald wrote:
> > Hi,
> >
> > I made a patch implementing my idea of efficient link navigation in an
> > html view.
> >
> > It works as follows:
> >
> > In an HTML view you hit 'd' and a tooltips over each visible link is
> > displayed. The tooltip contains a keysequence to access that link.
> > Entering the keysequence focusses the link and upon hitting enter the
> > link is followed (although that does not work reliably yet).
>
>  'd' doesn't feel quite right, especially hardcoded. 

That is true and it can be easily changed. I just took this one to implement 
this feature. I think it would be best to have it configurable, but a 
sensible default is also needed. Maybe you have a suggestion?

>  Besides, this looks a 
> lot like accesskeys. Wouldn't it be simpler if you only extended accesskeys
> support to also assign some keys to elements which don't have any accesskey
> assigned directly in the html page?

It does indeed sound much like accesskeys, and in fact I have been "inspired" 
by the code dealing with access keys. 

The problem I see here: the access keys occupy a certain range, and in the 
worst case occupy all 26 characters, so if there are more elements visible 
there are not any characters left to span more elements with a second 
character. To give an example: 25 access keys are defined: 'a-y' then we 
could use 'za' for the first link with no access key, 'zb' for the second and 
so on. If 'z' would have been also defined as an access key by the webpage, 
no space would be left for links without access keys. 

That is why I think a seperate function  would be better.

Greetings

Robert Ewald
[prev in list] [next in list] [prev in thread] [next in thread]