Make WordPress Core

Opened 7 weeks ago

Last modified 7 weeks ago

#62238 accepted defect (bug)

Update the screen-reader-text CSS class and its local implementations

Reported by: afercia's profile afercia Owned by: joedolson's profile joedolson
Milestone: 6.8 Priority: normal
Severity: normal Version:
Component: General Keywords: has-patch
Focuses: accessibility Cc:

Description

In a recent conversation on the block editor repository at https://github.com/WordPress/gutenberg/pull/65409 it was agreed to update and simplify the screen-reader-text CSS class by:

  • Removing the now unnecessary clip: rect(1px, 1px, 1px, 1px);
  • Removing any prefixed version of clip-path e.g. -webkit-clip-path: inset(50%);

This should be done through the entire codebase and the documentation , including bundled themes.

Change History (6)

#1 @joedolson
7 weeks ago

  • Owner set to joedolson
  • Status changed from new to accepted

#2 @joedolson
7 weeks ago

  • Milestone changed from Awaiting Review to 6.8

This ticket was mentioned in PR #7579 on WordPress/wordpress-develop by @hbhalodia.


7 weeks ago
#3

  • Keywords has-patch added

#4 @hbhalodia
7 weeks ago

Hi @afercia @joedolson, I have raised the PR by removing the required CSS for screen-reader-text. I have checked the entire codebase for the same.

Thank You,

@hbhalodia commented on PR #7579:


7 weeks ago
#5

Thanks @hbhalodia A couple first point for feedback:

  • Some stylesheet meant to support old browsers like Internet Explorer 7 must keep the old clip.
  • In other cases where clip is removed and there's no clip-path, then clip-path should be added.

Hi @afercia, thanks for the feedback. I would resolve this.

@hbhalodia commented on PR #7579:


7 weeks ago
#6

Hello @afercia, I have updated the PR with the requested changes.

Thank You,

Note: See TracTickets for help on using tickets.