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 337382 - Toggle button in main window not accessible.
Toggle button in main window not accessible.
Status: RESOLVED FIXED
Product: planner
Classification: Other
Component: General
unspecified
Other All
: Normal normal
: ---
Assigned To: planner-maint
planner-maint
Depends on:
Blocks: 419136
 
 
Reported: 2006-04-05 14:38 UTC by Rich Burridge
Modified: 2008-02-11 19:35 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch for src/planner-sidebar.diff (6.10 KB, patch)
2007-10-05 15:15 UTC, Igor Vasilcovsky
none Details | Review
Alternative accessibility patch (991 bytes, patch)
2008-02-10 12:38 UTC, Maurice van der Pot
reviewed Details | Review
Orca debug output whilst testing this problem. (28.55 KB, text/plain)
2008-02-11 17:39 UTC, Rich Burridge
  Details

Description Rich Burridge 2006-04-05 14:38:58 UTC
Planner (v0.13) has four toggle buttons in a
panel down the left side:

  Gantt Chart
  Tasks
  Resources
  Resource Usage

These have labels visually beneath them. Unfortunately
there is no accessibility relationship between them.
When one of those toggle buttons has focus, it doesn't
know that the label beneath it is associated with it.

Planner should use the LABEL_FOR relationship to match
up each toggle button with the appropriate label.
Comment 1 Igor Vasilcovsky 2007-10-05 15:15:14 UTC
Created attachment 96708 [details] [review]
patch for src/planner-sidebar.diff
Comment 2 Willie Walker 2007-11-01 17:47:28 UTC
Javier - if I recall, you were interested in planner accessibility at one time.  Can you try this patch and see if it resolves the problem?
Comment 3 Javier 2007-11-05 09:07:47 UTC
This patch works fine for me on latest planner svn trunk.
Except with the toolbar the ribbon toggle button to display more options, that can be found in other apps also.
Would be nice to fix that in default but seems that there is no way to determine that we are on a ribbon toggle button.
Now with this fixed I think that we can retire planner's script until new accessibility bugs requires it.
Comment 4 Willie Walker 2008-01-06 16:32:38 UTC
I tested this patch as well and it fixes the problem.  Note that if this patch is checked in, we need to adjust the Orca planner.py script since this patch will break the workaround we have in planner.py to address this bug.
Comment 5 Rich Burridge 2008-02-01 15:17:51 UTC
Planner maintainers: there is a patch attached to this bug that reported
fixes the problem. Can it please be checked into SVN trunk and this bug 
closed?

Thanks.
Comment 6 Maurice van der Pot 2008-02-10 12:38:28 UTC
Created attachment 104833 [details] [review]
Alternative accessibility patch

I'd like to suggest this alternative, as it does not modify the appearance of the sidebar buttons. Please check if this patch also solves the problem.
Is there an easy way I can check accessibility relations like this myself in a running GUI (a tool like spy++ for gtk+ maybe)?
Comment 7 Rich Burridge 2008-02-10 16:58:47 UTC
You can test patches like this by running Orca and navigating the 
area in question via the keyboard. Orca should hopefully speak
what is required.

I'll try out your patch tomorrow.

Thanks.
Comment 8 Rich Burridge 2008-02-11 17:39:04 UTC
Created attachment 104964 [details]
Orca debug output whilst testing this problem.

I've just tried the version of planner that's in SVN trunk, and the
accessibility problem no present. This is without applying either of
the attached patches.

So I presume the UI has been redesigned since I originally filed this
bug in April 2006, and it's now correctly accessible.

Closing as FIXED.
Comment 9 Rich Burridge 2008-02-11 17:39:36 UTC
Actually I'll close it as OBSOLETE.
Comment 10 Maurice van der Pot 2008-02-11 18:15:02 UTC
Are you sure it's not the work-around in planner.py mentioned by Willie in comment #4? 

This particular file has not been modified since October 2005, so the only way it can work now is because of changes outside Planner (gtk+, orca, ...).
Comment 11 Rich Burridge 2008-02-11 18:42:37 UTC
Ahh, thanks Maurice. This is me trying to do too many things at the
same time. Reopening so I can test your patch.
Comment 12 Rich Burridge 2008-02-11 19:12:53 UTC
Okay, I think I have my act together now. First I applied the
patch attached to bug #419136 to Orca from SVN trunk, to remove 
the workaround that's currently in the planner.py script.

I then tried the second patch below to planner, and rebuilt and
reinstalled.

Orca now nicely speaks those four toggle buttons.
I think this second patch can be committed.

Thanks Maurice. 

Comment 13 Maurice van der Pot 2008-02-11 19:35:57 UTC
Ok, the patch was committed. Thanks for reporting & testing!