Comment on attachment 581750
Updated patch.

[Approval Request Comment]
Regression caused by (bug #): 
User impact if declined: 
Accessibility will not be enabled on a GNOME 3 system, unless the user manually 
sets the accessibility gconf key (used by GNOME 2 but not 3, and thus not 
generally set under GNOME 3).

Testing completed (on m-c, etc.): 
Risk to taking this patch (and alternatives if risky):
The old behavior of checking gconf will be used if the dbus call does not 
succeed (ie, if AT-SPI2 is not installed), so I think that the risk is small.
An alternative approach would be to modify the start-up script to call 
dbus-send and set GNOME_ACCESSIBILITY=1 if the call indicates that 
accessibility is enabled, but this would likely affect start-up time more than 
applying the patch would, so applying the patch seems preferable.

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to firefox in Ubuntu.
https://bugs.launchpad.net/bugs/857153

Title:
  Needs to get accessibility settings from GSettings

Status in The Mozilla Firefox Browser:
  Fix Released
Status in “firefox” package in Ubuntu:
  Fix Committed
Status in “firefox” source package in Oneiric:
  Fix Committed
Status in “firefox” source package in Precise:
  Fix Committed

Bug description:
  Luke mentioned to me in Dublin that the screen reader doesn't work
  with Firefox when accessibility is enabled. This is because Firefox is
  still using GConf to check if accessibility is enabled, when it really
  needs to be using GSettings. I've just realized this is still the
  case, so I assume that the screen reader still won't work by default

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/857153/+subscriptions

-- 
Mailing list: https://launchpad.net/~desktop-packages
Post to     : desktop-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~desktop-packages
More help   : https://help.launchpad.net/ListHelp

Reply via email to