Bug 123864 - No feedback for screen reader when radio button for underline attribute changes
Summary: No feedback for screen reader when radio button for underline attribute changes
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
6.0.6.2 release
Hardware: All All
: high major
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: accessibility, bibisected, bisected, regression
Depends on:
Blocks: a11y
  Show dependency treegraph
 
Reported: 2019-03-05 10:12 UTC by Jean-Philippe MENGUAL
Modified: 2019-04-05 16:29 UTC (History)
10 users (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jean-Philippe MENGUAL 2019-03-05 10:12:29 UTC
Hi,

Using Orca master:
1. So far, ctrl-u enabled underline and said "Underline on/off".
2. Now, underline is still enabled, but Orca says only "Underline button". It is no longer able to describe it status, unlikely it does for italic, bold and other buttons.

Regards
Comment 1 Alex ARNAUD 2019-03-15 09:07:25 UTC
@Caolan: Could it be possible to request you help on this ?

I aware you're very well familiar of UI :).

Best regards,
Alex.
Comment 2 Alex ARNAUD 2019-03-15 09:14:02 UTC
I can reproduce this with:
Version: 6.3.0.0.alpha0+
Build ID: 4aa8a6ab08b755e3d82860e8dbc294f854336477
CPU threads: 4; OS: Linux 4.9; UI render: default; VCL: gtk3; 
TinderBox: Linux-rpm_deb-x86_64@86-TDF, Branch:master, Time: 2019-03-14_17:51:01
Locale: fr-FR (fr_FR.UTF-8); UI-Language: en-US
Calc: threaded
Comment 3 Alex ARNAUD 2019-04-01 09:37:11 UTC
I can reproduce this on LibreOffice 6.0.6.2 but not on earliest release of LibreOffice 6.0 release cycle (5.4.0.0.alpha1+).

I'll do a bibisect ASAP.

Best regards,
Alex.
Comment 4 Alex ARNAUD 2019-04-01 16:39:02 UTC
This is the result of my bibisection on repo https://git.libreoffice.org/bibisect-linux-64-6.0:

39cd2b9108fd2b536bea08f5fb7ce97377bcd292 is the first bad commit
commit 39cd2b9108fd2b536bea08f5fb7ce97377bcd292
Author: Jenkins Build User <tdf@pollux.tdf>
Date:   Fri Jul 14 15:05:21 2017 +0200

    source sha:52dfefec8da5d7f25c39218fd890cad6491728ab
    
    source sha:52dfefec8da5d7f25c39218fd890cad6491728ab

:040000 040000 b2764b8359a7dadf360e7a6f377194e609ad9c7e 737a8c5bdefa4b67c32018b11be9ce838608fe45 M	instdir

Best regards,
Alex.
Comment 5 Alex ARNAUD 2019-04-02 08:39:37 UTC
@Jan-Marek Glogowski: it looks like it's your commit that has introduced this issue.

I'm not a LibreOffice Developper, I've redo the bibisect a second time to be sure and it's your commit 52dfefec8da5d7f25c39218fd890cad6491728ab that has introduced this regression.

Best regards,
Alex.
Comment 6 Alex ARNAUD 2019-04-02 08:54:10 UTC
I suggest to increase the severity of this issue at nowadays a blind person couldn't know if underline is enabled or disable when he does the shortcut.

Is there a rule about severity of issues ?

Best regards,
Alex.
Comment 7 Xisco Faulí 2019-04-04 15:17:50 UTC
Hi Alex,
Does it happen if you run LibreOffice with SAL_USE_VCLPLUGIN=gen ?
Comment 8 Alex ARNAUD 2019-04-05 09:30:25 UTC
(In reply to Xisco Faulí from comment #7)
> Hi Alex,
> Does it happen if you run LibreOffice with SAL_USE_VCLPLUGIN=gen ?

The gen backend seems completely inaccessible so I can't tell you.

Otherwise, the issue appears both with GTK2 and GTK3 backends on my Debian 9 "Stretch".

Maybe Patrick or Vstuart could confirm the issue on Windows.

Best regards,
Alex.
Comment 9 Alex ARNAUD 2019-04-05 09:56:02 UTC
> Maybe Patrick or Vstuart could confirm the issue on Windows.

It seems worse on Windows 10 with NVDA 2018.4.1, I obtain completely no feedback when activate/deactivate bold, italic or underline with LibreOfficeDev 6.3 built the 2019-04-05.

Best regards,
Alex.
Comment 10 V Stuart Foote 2019-04-05 12:57:02 UTC
On Windows 10 Home 64-bit en-US (1809) with NVDA 2019.1 and
Version: 6.3.0.0.alpha0+ (x64)
Build ID: 4ca9db953d59d93ce8e3a54a36d23ed52b9c62a9
CPU threads: 4; OS: Windows 10.0; UI render: GL; VCL: win; 
TinderBox: Win-x86_64@42, Branch:master, Time: 2019-03-26_03:34:16
Locale: en-US (en_US); UI-Language: en-US
Calc: threaded

On the Standard toolbar, the Underline button does report its status.

Either with navigation via keyboard, or on mouseover.

Program emits a "pressed" status that NVDA picks up and sounds. The same as for the other text attribute button actions visible by default--Bold, Italic and Strike-through.
Comment 11 Alex ARNAUD 2019-04-05 13:22:02 UTC
(In reply to V Stuart Foote from comment #10)
> On Windows 10 Home 64-bit en-US (1809) with NVDA 2019.1 and
> Version: 6.3.0.0.alpha0+ (x64)
> Build ID: 4ca9db953d59d93ce8e3a54a36d23ed52b9c62a9
> CPU threads: 4; OS: Windows 10.0; UI render: GL; VCL: win; 
> TinderBox: Win-x86_64@42, Branch:master, Time: 2019-03-26_03:34:16
> Locale: en-US (en_US); UI-Language: en-US
> Calc: threaded
> 
> On the Standard toolbar, the Underline button does report its status.
> 
> Either with navigation via keyboard, or on mouseover.
> 
> Program emits a "pressed" status that NVDA picks up and sounds. The same as
> for the other text attribute button actions visible by default--Bold, Italic
> and Strike-through.

Do you confirm that it works when you select a text, keep the cursor into the document content and press control + u ?

Best regards,
Alex.
Comment 12 V Stuart Foote 2019-04-05 14:52:57 UTC
On Windows 10 Pro 64-bit en-US (1803) with NVDA 2018.4 and
Version: 6.2.3.1 (x64)
Build ID: 9ba025bafb03b962c34687cf87806cc03a3a7436
CPU threads: 8; OS: Windows 10.0; UI render: GL; VCL: win; 
Locale: en-US (en_US); UI-Language: en-US
Calc: threaded

In Writer _none_ of the text attribute controls (Bold, Italic, Underline, Strikethrough) sound when their shortcuts are used while in document canvas.

But they all sound when applied as "pressed" when formatting is applied from the Formatting toolbar (F10 - F6 - F6 and TAB to desired attribute), and then placing focus back on text cursor in document canvas (ESC).

I'm not sure about prior releases, did this ever sound from document canvas? I'll have to play with it a bit.
Comment 13 V Stuart Foote 2019-04-05 15:18:00 UTC
(In reply to V Stuart Foote from comment #12)

> In Writer _none_ of the text attribute controls (Bold, Italic, Underline,
> Strikethrough) sound when their shortcuts are used while in document canvas.
> 

And should have noted, the actual formatting applied is read back in NVDA when I have set NVDA preferences for Document Formatting from the Font block of types.

So, events are consistently emitted ("pressed", or silent) for attributes set via the Toolbar input mechanisim, and resulting formatting is readback from document canvas.  And events are consistently not emitted for text attributes set via keyboard accelerators.  So, assume there is no linkage to the button status?

Should it sound the button (formatting or style) state (pressed/released), or the  state of the text attribute (applied/removed)?

On Toolbar -> <Tab> onto button widget; "Bold pressed", "Underline released"

While in document at text cursor -> via Keyboard <Ctrl+I> "Italic applied", <Ctrl+U> "Underline removed"
Comment 14 V Stuart Foote 2019-04-05 16:18:36 UTC
For 5.3 and at 5.3 the Underline command went through some changes in creating a Split button for use on Sidebar.

Noticed when testing this against a 5.3 build that the Formatting toolbar was listed as a "Dropdown button" and was "checked" unlike the other "Button"(s) which were "pressed".

At 5.3 we got two UNO controls (bug 101672):

.uno:Underline -- which is now used on the Sidebar -> Properties deck (from 5.2.0 to current)

.uno:UnderlineSimple -- which is now used on the Formatting toolbar (from 5.3.0 to current)

At 5.3 Windows and NVDA still do not report the attribute changes with any of the shortcut/accelerators. Maybe should, but they were consistent.

But is the disconnect in Orca somehow related to work on the Underline UNO control?
Comment 15 V Stuart Foote 2019-04-05 16:29:04 UTC
s/For 5.3 and at 5.3/For 5.2 and at 5.3/

And, at some point prior to 6.2 and master/6.3.0 (probably at 5.4.0) the split button is removed from the Formatting toolbar in favor of the UnderlineSimple control.

I don't have a load of 5.4, 6.0 or 6.1 available to check against. But I'll poke at it later.