Bug 118418 - Should implemented scrollto, scrolltopoint, scrollsubstringto, scrollsubstringtopoint
Summary: Should implemented scrollto, scrolltopoint, scrollsubstringto, scrollsubstrin...
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: sdk (show other bugs)
Version:
(earliest affected)
Inherited From OOo
Hardware: All All
: medium enhancement
Assignee: Not Assigned
URL:
Whiteboard: target:7.0.0
Keywords: accessibility
Depends on:
Blocks: a11y
  Show dependency treegraph
 
Reported: 2018-06-27 15:52 UTC by Samuel Thibault
Modified: 2020-05-08 22:18 UTC (History)
2 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 Samuel Thibault 2018-06-27 15:52:59 UTC
Description:
For better accessibility, libreoffice should implement the scroll*to* accessibility calls, which allow the screen reader to fix what is shown, so that a blind user and a sighted user can understand each other when they are working together on a document.


Actual Results:
iaccessible2 and atspi scrollto calls have no effect

Expected Results:
iaccessible2 and atspi scrollto calls should scroll the document


Reproducible: Always


User Profile Reset: No



Additional Info:
More precisely, for instance winaccessibility/source/UAccCOM/MAccessible.cxx reads:

STDMETHODIMP CMAccessible:: scrollTo(enum IA2ScrollType)
{
    return E_NOTIMPL;
}

It should support the requested scroll instead :)
(and similar for scrollToPoint, and in winaccessibility/source/UAccCOM/AccTextBase.cxx for scrollSubstringTo and scrollSubstringToPoint)

The code mentioned above is for Windows, Linux' atspi2 has just gotten scrollTo and scrollToPoint in atk 2.29.1, and scrollSubstringTo and scrollSubstringToPoint will probably be added before 2.30 too.
Comment 2 Commit Notification 2020-05-07 10:11:59 UTC
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.
Comment 3 cwendling 2020-05-07 13:23:46 UTC
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.
Comment 4 Commit Notification 2020-05-08 22:18:57 UTC
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.