diff options
| author | Michael Weghorn <m.weghorn@posteo.de> | 2024-05-08 10:17:53 +0200 |
|---|---|---|
| committer | Michael Weghorn <m.weghorn@posteo.de> | 2024-05-08 13:56:10 +0200 |
| commit | addb7abeb8ec1a145d9ba35d3be8518f2502325b (patch) | |
| tree | 43b35746080ce4ff34705ce5042c4a69b2c9b072 /vcl/unx | |
| parent | a469aea9c0b532d928cd41e389c9c51de1af06f0 (diff) | |
gtk3 a11y: No longer manually add focused state for focus object
No longer manually add `ATK_STATE_FOCUSED` to the
ATK focus object.
The focused object should should have the
`css::accessbibility::AccessibleStateType::FOCUSED`
state set on UNO level, in which case this is already
mapped to the focused state in the lines above the
newly removed code, so explicitly addding it
again shouldn't be necessary.
If the state is missing, that most likely needs
to be fixed in the underlying `XAccessibleStateSet`
implementation.
Therefore, remove what looks like a workaround.
If any issues show up after this, let's rather take
a look at the underlying issues and try to fix things
there.
Change-Id: Ie7252c4b2dd2ec15c7a3bbc6a1a5e2830cdd39a7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/167309
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
Tested-by: Jenkins
Diffstat (limited to 'vcl/unx')
| -rw-r--r-- | vcl/unx/gtk3/a11y/atkwrapper.cxx | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/vcl/unx/gtk3/a11y/atkwrapper.cxx b/vcl/unx/gtk3/a11y/atkwrapper.cxx index db0aa1dbc907..a16e8f80834c 100644 --- a/vcl/unx/gtk3/a11y/atkwrapper.cxx +++ b/vcl/unx/gtk3/a11y/atkwrapper.cxx @@ -641,14 +641,6 @@ wrapper_ref_state_set( AtkObject *atk_obj ) if ( (nStateSet & nState) && mapAtkState( nState ) != ATK_STATE_LAST_DEFINED ) atk_state_set_add_state( pSet, mapAtkState( nState ) ); } - - // We need to emulate FOCUS state for menus, menu-items etc. - if( atk_obj == atk_get_focus_object() ) - atk_state_set_add_state( pSet, ATK_STATE_FOCUSED ); -/* FIXME - should we do this ? - else - atk_state_set_remove_state( pSet, ATK_STATE_FOCUSED ); -*/ } } |