Skip to:
Content

bbPress.org

Opened 5 years ago

Closed 5 years ago

#2637 closed defect (fixed)

Accessibility: Template Notices

Reported by: netweb Owned by: johnjamesjacoby
Milestone: 2.6 Priority: normal
Severity: normal Version:
Component: General - Accessibility Keywords:
Cc:

Description

via Karl Groves bbPress Accessibility Feedback https://gist.github.com/ntwb/c8f99df8a714ed028e2d

Validation error messaging. The use of a list allows the user to know how many errors were found.

Be sure to shift focus directly to this div $('.error:first').focus();

Proposed:

<div class="bbp-template-notice error" role="alert" tabindex="-1">
    <ul>
        <li><strong>ERROR</strong>: Your topic needs a title.</li>
        <li><strong>ERROR</strong>: Your topic cannot be empty.</li>
    </ul>
</div>

Current:

<div class="bbp-template-notice error">
        <p><strong>ERROR</strong>: Your topic needs a title.</p>
        <p><strong>ERROR</strong>: Your topic cannot be empty.</p>
</div>

Attachments (1)

2637.diff (24.9 KB) - added by netweb 5 years ago.

Download all attachments as: .zip

Change History (6)

@netweb
5 years ago

#1 @johnjamesjacoby
5 years ago

  • Component changed from Theme Compatability to Accessibility
  • Milestone changed from Awaiting Review to 2.7

Bumping to 2.7 so we can clear out Awaiting Review and 2.6 milestones.

#2 @johnjamesjacoby
5 years ago

  • Milestone changed from 2.7 to 2.8

Bumping all 2.7 to 2.8 milestone.

This ticket was mentioned in Slack in #bbpress by netweb. View the logs.


5 years ago

#4 @johnjamesjacoby
5 years ago

  • Milestone changed from 2.8 to 2.6

#5 @johnjamesjacoby
5 years ago

  • Owner set to johnjamesjacoby
  • Resolution set to fixed
  • Status changed from new to closed

In 5563:

Convert template notices from paragraph tags to unordered lists.

Improves accessibility by hinting there may be several notices on the page.

Props Karl Groves, netweb. FIxes #2637.

Note: See TracTickets for help on using tickets.