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

List:       kde-bugs-dist
Subject:    [Bug 83053] accesskeys conflict with normal shortcuts
From:       Germain Garand <germain () ebooksfrance ! com>
Date:       2004-07-12 1:07:06
Message-ID: 20040712010706.22479.qmail () ktown ! kde ! org
[Download RAW message or body]

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
      
http://bugs.kde.org/show_bug.cgi?id=83053      




------- Additional Comments From germain ebooksfrance com  2004-07-12 03:07 -------
Hi Jean-baptiste,
I think your patch looks good, but that it doesn't solve the main problem : \
advertisement. 

Accessibility is not an advanced feature, so people should be able to find the \
accessor intuitively.

Every HTML doc out there on the web advertise the accesskey combination as being \
Alt+<key>, so why not just use that?

If you have noticed, the normal Alt+key behaviour is very sane in case of conflict:
it simply cycle between all identical top level menu accelerators, popping them on \
the way.

So what would work nicely is as follow:

Given a keypress of Alt+Key_X

1) if no element has focus or currently focused element doesn't have a matching \
accesskey  => look for the first element with access key "X"

- not found? let the event slip to the application for top level menu handling
- found ? => focus

2) else if an element has focus and has a matching accesskey,
  => look for next element with accesskey "X" 
  
- no more? let the event slip to the application
- found ? => focus

and then you have a good, standard compliant,  accessor cycling and no conflict.
What do you think?

Otherwise, your idea of popping all accessors at once is all fine, but again, people \
will have to know beforehand about the key... not sure they'll find out ;-( Anyway, \
that's rather a bonus feature for die-hard accessibility users, so that's OK if it's \
not obvious - provided it's documented somewhere.


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

Configure | About | News | Add a list | Sponsored by KoreLogic