Skip to content

a11y: make ins/del diff markup accessible to screen readers - #39

Merged
ddtobiasz merged 2 commits into
appian:masterfrom
hiteshdugar:a11y/ins-del-screen-reader-support
Jun 16, 2026
Merged

a11y: make ins/del diff markup accessible to screen readers#39
ddtobiasz merged 2 commits into
appian:masterfrom
hiteshdugar:a11y/ins-del-screen-reader-support

Conversation

@hiteshdugar

@hiteshdugar hiteshdugar commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Problem

When HTML diff output is displayed in richTextFieldWithTables in readOnly mode, added text (green) and removed text (red strikethrough) are conveyed through visual styling only. Screen readers read all text as plain content with no indication of what was added or removed — violating WCAG 2.1 SC 1.3.1.

Changes

index.js

  1. added "ins", "del" to ALLOWED_TAGS;
  2. added makeInsDelAccessible() and escapeAttr() functions; called makeInsDelAccessible() in readOnly path of setEditorContents()

custom.css

  1. added ins { text-decoration: none; } and del { text-decoration: none; }

appian-component-plugin.xml

  1. version 1.19.0 to 1.19.1

Backward compatible. No impact on editing, pasting, saving, or existing readOnly content.

@hiteshdugar
hiteshdugar force-pushed the a11y/ins-del-screen-reader-support branch from 16be2f9 to 86d69ce Compare June 15, 2026 19:19
@hiteshdugar
hiteshdugar force-pushed the a11y/ins-del-screen-reader-support branch from 86d69ce to 4d19d03 Compare June 15, 2026 19:29
@ddtobiasz
ddtobiasz force-pushed the a11y/ins-del-screen-reader-support branch from fb3a71e to 9447e89 Compare June 16, 2026 18:28
@ddtobiasz
ddtobiasz merged commit 78c537f into appian:master Jun 16, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants