After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 595299 - ListView column headers are not controllable through keyboard
ListView column headers are not controllable through keyboard
Status: RESOLVED FIXED
Product: banshee
Classification: Other
Component: User Interface
git master
Other Linux
: Normal normal
: 1.x
Assigned To: Banshee Maintainers
Banshee Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-09-15 17:36 UTC by Eitan Isaacson
Modified: 2009-11-12 03:59 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Renamed FocusedIndex to FocusedRowIndex. (12.05 KB, patch)
2009-09-30 17:32 UTC, Eitan Isaacson
none Details | Review
Added FocusedColumnIndex. (1.46 KB, patch)
2009-09-30 17:32 UTC, Eitan Isaacson
none Details | Review
Factored out header left click. (4.69 KB, patch)
2009-09-30 17:32 UTC, Eitan Isaacson
none Details | Review
Added focusable headers, and made them keyboard controllable. (7.28 KB, patch)
2009-09-30 17:32 UTC, Eitan Isaacson
none Details | Review
Draw focused column header. (3.60 KB, patch)
2009-09-30 17:32 UTC, Eitan Isaacson
none Details | Review
Draw focused column header. (4.05 KB, patch)
2009-10-01 19:43 UTC, Eitan Isaacson
none Details | Review
Draw focused column header. (4.05 KB, patch)
2009-10-02 00:07 UTC, Eitan Isaacson
none Details | Review
[a11y] Added focusable headers, and made them keyboard controllable. (8.12 KB, patch)
2009-10-04 22:35 UTC, Eitan Isaacson
none Details | Review
[a11y] Draw focused column header. (4.04 KB, patch)
2009-10-04 22:36 UTC, Eitan Isaacson
none Details | Review

Description Eitan Isaacson 2009-09-15 17:36:55 UTC
Changing the sort order is possible with a pointer, but with keyboard-exclusive access, one cannot put focus on the ListView headers.
Comment 1 Eitan Isaacson 2009-09-25 19:31:43 UTC
This branch addresses this[1]. There are behavior and theming changes involved, so please review. When tabbing into a list view, keyboard focus is on the headers (if they are visible), left or right arrows navigate between the headers. The focus order is not perfect. For example if you tab out of headers and into the list, and then tab entirely out of the list view, shift+tab will bring you back to the headers instead of the list. This could be solved by using a container as the ListView's parent class, this is how GtkTreeView does it.

1. http://github.com/eeejay/banshee-a11y/tree/keyboard_control
Comment 2 Gabriel Burt 2009-09-26 18:36:55 UTC
Hi Eitan, please attach patch(es) to their relevant bug for review.  Thanks!
Comment 3 Eitan Isaacson 2009-09-30 17:32:14 UTC
Created attachment 144418 [details] [review]
Renamed FocusedIndex to FocusedRowIndex.
Comment 4 Eitan Isaacson 2009-09-30 17:32:17 UTC
Created attachment 144419 [details] [review]
Added FocusedColumnIndex.
Comment 5 Eitan Isaacson 2009-09-30 17:32:22 UTC
Created attachment 144420 [details] [review]
Factored out header left click.
Comment 6 Eitan Isaacson 2009-09-30 17:32:27 UTC
Created attachment 144421 [details] [review]
Added focusable headers, and made them keyboard controllable.
Comment 7 Eitan Isaacson 2009-09-30 17:32:31 UTC
Created attachment 144422 [details] [review]
Draw focused column header.
Comment 8 Eitan Isaacson 2009-09-30 17:36:56 UTC
For ease of review, I split the patch into a few simpler ones. Sorry about the spam!
Comment 9 Eitan Isaacson 2009-10-01 19:43:54 UTC
Created attachment 144534 [details] [review]
Draw focused column header.
Comment 10 Eitan Isaacson 2009-10-02 00:07:46 UTC
Created attachment 144554 [details] [review]
Draw focused column header.

A prettier version, works nicer with the updated focus indication patch.
Comment 11 Eitan Isaacson 2009-10-04 22:35:20 UTC
Created attachment 144747 [details] [review]
[a11y] Added focusable headers, and made them keyboard controllable.

Added ListView.ActiveColumn[Changed] instead of Selection.FocusedColumnIndex.
Comment 12 Eitan Isaacson 2009-10-04 22:36:50 UTC
Created attachment 144748 [details] [review]
[a11y] Draw focused column header.

Using new ListView.ActiveColumn.
Comment 13 Gabriel Burt 2009-10-27 20:18:28 UTC
Bulk changing the assignee to banshee-maint@gnome.bugs to make it easier for people to get updated on all banshee bugs by following that address.  It's usually quite apparent who is working on a given bug by the comments and/or patches attached.