diff options
author | Michael Weghorn <m.weghorn@posteo.de> | 2023-10-27 14:53:59 +0200 |
---|---|---|
committer | Michael Weghorn <m.weghorn@posteo.de> | 2023-10-28 05:45:15 +0200 |
commit | f44eae4912e8f2ba34d93a71a609473df976a2cb (patch) | |
tree | b9b7147fbf4020bc27c322b2a4ab07e28745f464 /winaccessibility | |
parent | 33061be5a7538409839dcca3116b9fa72629013f (diff) |
tdf#135586 a11y: Add a new BLOCK_QUOTE a11y role
A block quote role exists in all of WAI-ARIA 1.3
(role "blockquote", [1]), IAccessible2
(`IA2_ROLE_BLOCK_QUOTE`, [2]) and AT-SPI
(`ROLE_BLOCK_QUOTE`, [3]).
Take over the definition that is the same in WAI-ARIA
and IAccessible2:
> A section of content that is quoted from another source.
The intended use for now is for a Writer paragraph using
the "Block Quotation" paragraph style, similar to how the
HEADING role is used for paragraphs using a corresponding
paragraph style.
For gtk3 (ATK) and winaccessibility (IAccessible2),
map the new role to the equivalant roles.
For macOS and the gtk4 as well as the Qt-based VCL plugins
on Linux which currently don't have an equivalent role,
fall back to the same role that the PARAGRAPH role is
mapped to.
This way, the behavior there will remain unchanged
once the BLOCK_QUOTE role is used for Writer paragraphs
with the corresponding style.
In general, treat BLOCK_QUOTE like PARAGRAPH
in code applying special handling for the PARAGRAPH
role.
[1] https://w3c.github.io/aria/#blockquote
[2] https://github.com/LinuxA11y/IAccessible2/blob/3d8c7f0b833453f761ded6b12d8be431507bfe0b/api/AccessibleRole.idl#L318
[3] https://gitlab.gnome.org/GNOME/at-spi2-core/-/blob/7cc4cee53ddbd22631fd110f0e5ce045dec2e411/xml/Accessible.xml#L615-616
Change-Id: I248c183a2e7ec5d6f0a89bf3cb4829bbd8588c77
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/158573
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
Diffstat (limited to 'winaccessibility')
-rw-r--r-- | winaccessibility/inc/AccParagraphEventListener.hxx | 4 | ||||
-rw-r--r-- | winaccessibility/source/service/AccObject.cxx | 6 | ||||
-rw-r--r-- | winaccessibility/source/service/AccObjectWinManager.cxx | 1 |
3 files changed, 8 insertions, 3 deletions
diff --git a/winaccessibility/inc/AccParagraphEventListener.hxx b/winaccessibility/inc/AccParagraphEventListener.hxx index daa07603bc19..d2f5457e1b0c 100644 --- a/winaccessibility/inc/AccParagraphEventListener.hxx +++ b/winaccessibility/inc/AccParagraphEventListener.hxx @@ -26,8 +26,8 @@ /** * AccParagraphEventListener is inherited from AccContainerEventListener. It handles the events - * generated by container controls. The accessible roles are: PARAGRAPH and HEADING. - * It defines the procedure of specific event handling related with text containsers and provides + * generated by container controls. The accessible roles are: PARAGRAPH, HEADING and BLOCK_QUOTE. + * It defines the procedure of specific event handling related with text containers and provides * the detailed support for some related methods. */ class AccParagraphEventListener : public AccContainerEventListener diff --git a/winaccessibility/source/service/AccObject.cxx b/winaccessibility/source/service/AccObject.cxx index ad0f65d8cc09..96e8cfa37b5e 100644 --- a/winaccessibility/source/service/AccObject.cxx +++ b/winaccessibility/source/service/AccObject.cxx @@ -67,6 +67,8 @@ short lcl_mapToIAccessible2Role(sal_Int16 nUnoRole) return IA2_ROLE_UNKNOWN; case css::accessibility::AccessibleRole::ALERT: return ROLE_SYSTEM_DIALOG; + case css::accessibility::AccessibleRole::BLOCK_QUOTE: + return IA2_ROLE_BLOCK_QUOTE; case css::accessibility::AccessibleRole::COLUMN_HEADER: return ROLE_SYSTEM_COLUMNHEADER; case css::accessibility::AccessibleRole::CANVAS: @@ -369,7 +371,7 @@ void AccObject::UpdateName( ) if( ( TEXT_FRAME == m_accRole ) && ( m_pParentObj !=nullptr )&& ( SCROLL_PANE == m_pParentObj -> m_accRole ) ) m_pIMAcc->Put_XAccName( o3tl::toW(m_pParentObj->m_xAccContextRef->getAccessibleName().getStr()) ); //IAccessibility2 Implementation 2009----- - if ( PARAGRAPH == m_accRole) + if (m_accRole == AccessibleRole::PARAGRAPH || m_accRole == AccessibleRole::BLOCK_QUOTE) { m_pIMAcc->Put_XAccName(L""); } @@ -470,6 +472,7 @@ void AccObject::SetValue( Any pAny ) // 3. date editor's msaa value should be the same as spinbox case DATE_EDITOR: case TEXT: + case BLOCK_QUOTE: case PARAGRAPH: case HEADING: case TABLE_CELL: @@ -924,6 +927,7 @@ void AccObject::UpdateState() case HEADING: //Image Map + case BLOCK_QUOTE: case PARAGRAPH: case PASSWORD_TEXT: case SHAPE: diff --git a/winaccessibility/source/service/AccObjectWinManager.cxx b/winaccessibility/source/service/AccObjectWinManager.cxx index 04f81875cd88..24deb8150c51 100644 --- a/winaccessibility/source/service/AccObjectWinManager.cxx +++ b/winaccessibility/source/service/AccObjectWinManager.cxx @@ -749,6 +749,7 @@ AccObjectWinManager::CreateAccEventListener(XAccessible* pXAcc) case AccessibleRole::VIEW_PORT: pRet = new AccContainerEventListener(pXAcc, this); break; + case AccessibleRole::BLOCK_QUOTE: case AccessibleRole::PARAGRAPH: case AccessibleRole::HEADING: pRet = new AccParagraphEventListener(pXAcc, this); |