Skip to:
Content

BuddyPress.org

Opened 3 years ago

Closed 3 years ago

#6882 closed enhancement (fixed)

Accessibility: Bump heading in groups/single/admin.php

Reported by: mercime Owned by: mercime
Milestone: 2.7 Priority: normal
Severity: normal Version:
Component: Core Keywords: has-patch has-screenshots
Cc:

Description (last modified by mercime)

Basic rationale: see #6871

Current headings:
<h1>Group Name</h1>
<h2>Group Admins/Mods</h2>
admin.php
<h4>Privacy Options</h4> => <h2> ... </h2>
<h4>Group Invitations</h4> => <h2> ... </h2>

This also includes grouping of radio buttons to improve accessibility without using fieldsets.

Attachments (9)

6882.patch (6.7 KB) - added by mercime 3 years ago.
6882-plus-css.patch (12.3 KB) - added by mercime 3 years ago.
2010.png (31.4 KB) - added by mercime 3 years ago.
2016.png (47.4 KB) - added by mercime 3 years ago.
2015.png (37.9 KB) - added by mercime 3 years ago.
2014.png (32.8 KB) - added by mercime 3 years ago.
2013.png (42.6 KB) - added by mercime 3 years ago.
2012.png (37.4 KB) - added by mercime 3 years ago.
6882-fieldset.patch (6.7 KB) - added by mercime 3 years ago.
Use fieldsets to group radio buttons

Download all attachments as: .zip

Change History (16)

#1 @mercime
3 years ago

  • Description modified (diff)

@mercime
3 years ago

@mercime
3 years ago

@mercime
3 years ago

@mercime
3 years ago

@mercime
3 years ago

@mercime
3 years ago

@mercime
3 years ago

#2 @mercime
3 years ago

  • Keywords has-patch has-screenshots added

Attached patches bump headings of the screen in Group Manage > Settings and groups two radio sets on screen by using aria-labelledby" and aria role="group" in lieu of fieldsets

  • 6882.patch​ - noted above
  • 6882-plus-css.patch - noted above plus adds minimum style support

Attached before & after screenshots in the following themes:

  • Twenty Ten
  • Twenty Sixteen
  • Twenty Fifteen
  • Twenty Fourteen
  • Twenty Thirteen
  • Twenty Twelve

#3 @DJPaul
3 years ago

  • Keywords changed from has-patch has-screenshots to has-patch, has-screenshots
  • Milestone changed from Awaiting Review to 2.6

#4 @mercime
3 years ago

  • Keywords changed from has-patch, has-screenshots to has-patch has-screenshots
  • Milestone changed from 2.6 to Future Release

#5 @DJPaul
3 years ago

  • Component changed from General - UX/UI to Core

#6 @mercime
3 years ago

  • Milestone changed from Future Release to 2.7
  • Owner set to mercime
  • Status changed from new to assigned

@mercime
3 years ago

Use fieldsets to group radio buttons

#7 @mercime
3 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 10969:

Accessibility: Use fieldsets to group radio buttons in Manage > Settings.

Fixes #6882.

Note: See TracTickets for help on using tickets.