Summary: | Should implemented scrollto, scrolltopoint, scrollsubstringto, scrollsubstringtopoint | ||
---|---|---|---|
Product: | LibreOffice | Reporter: | Samuel Thibault <samuel.thibault> |
Component: | sdk | Assignee: | Not Assigned <libreoffice-bugs> |
Status: | NEW --- | ||
Severity: | enhancement | CC: | cwendling, foss, jean-philippe.mengual, vsfoote |
Priority: | medium | Keywords: | accessibility |
Version: | Inherited From OOo | ||
Hardware: | All | ||
OS: | All | ||
See Also: |
https://bugs.documentfoundation.org/show_bug.cgi?id=105610 https://bugs.documentfoundation.org/show_bug.cgi?id=155352 |
||
Whiteboard: | target:7.0.0 | ||
Crash report or crash signature: | Regression By: | ||
Bug Depends on: | |||
Bug Blocks: | 101912 |
Description
Samuel Thibault
2018-06-27 15:52:59 UTC
The methods are documented on https://gitlab.gnome.org/GNOME/atk/blob/master/atk/atkcomponent.h#L204 and https://gitlab.gnome.org/GNOME/atk/blob/master/atk/atktext.h#L337 Colomban Wendling committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/b46a0a7f9f88a0213b0a5ca2712b6eb93b74d088 tdf#118418 implement scrollSubstringTo() for gtk It will be available in 7.0.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. https://git.libreoffice.org/core/commit/b46a0a7f9f88a0213b0a5ca2712b6eb93b74d088 implemented `scrollSubstringTo(*, *, SCROLL_ANYWHERE)` in a subset of LO. This issue is however still relevant for the other queries and types of scrolling. Samuel Thibault committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/6aef5a1e880262354cf739e461a12a0ef9573fef tdf#118418 implement scrollSubstringTo() for Qt & windows It will be available in 7.0.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. Is there any reason to keep this issue open? The support is enough for most screen reader use, but it's far from complete. (In reply to Samuel Thibault from comment #6) > The support is enough for most screen reader use, but it's far from complete. Wouldn't be better to rename this one as initial support and report specifics issues if needed? Otherwise, I suspect this issue would stay opened forever. > > The support is enough for most screen reader use, but it's far from complete.
>
> Wouldn't be better to rename this one as initial support and report specifics issues if needed? Otherwise, I suspect this issue would stay opened forever.
Possibly, yes.
Hello, Just to make it clear: I would say that before spending money on this issue, the real remaining needs should be assessed. Ideally we'd have scroll* support on all scrollable widgets of course, but in practice we don't really need it on all of them, far from it. In orca for instance, it is essentially used when requesting to speak a whole text. So it is really needed only on the document itself and in textview pieces of dialog boxes. The current support is already enough for various use cases, notably it works for the document. Before planning to invest money on completing the support for other widgets, I would say to first check which pieces of the interface the users are still lacking support for. Possibly the current support is already really fine, and it's better to prioritize other tasks. Restoring the accessibility tests comes to my mind as a high priority. Samuel Giving more technical details. By "it's far from complete", I mean that the scrollSubstringTo method is usually not implemented by widgets of libreoffice, it's really implemented only in sw/source/core/access/accpara.cxx, and only for the SCROLL_ANYWHERE request case. But in the current screen reader usage (show the progress of the speech synthesis for a long reading), the widgets for which scrollto* support is useful is the widgets which represent text inside a scrollable area. The support in accpara is thus already covering the main usage: reading the text of the document. Possibly some other document elements would welcome adding the support, when for instance the document is composed of a series of images with descriptions, without any text in between. That's however quite low priority. One thing I can think of that would be useful is for textviews in dialog boxes, for instance in lowriter's File->Properties, in the Description tab, the Comments textview. That widget shows up from times to times, e.g. in the spelling dialog box, in the Envelope dialog box, etc. Quite often the textview is tall enough for the text to fit and no scrolling is required, but adding the support there would probably be useful. |