Problem/Motivation

The lazy load formatter uses a div for the button, which doesn't receive tab focus
For a11y sake it should be a button as this will receive focus and is more correct semantically

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#2 fix-markup-2998670-1.patch1.07 KBflxa

Comments

larowlan created an issue. See original summary.

flxa’s picture

flxa commented
Status: Active » Needs review
StatusFileSize
new fix-markup-2998670-1.patch1.07 KB
sam152’s picture

sam152 commented
Version: 8.x-1.x-dev » 8.x-2.x-dev

I assume it looks the same? Will commit against 2.x.

sam152’s picture

sam152 commented
Status: Needs review » Fixed

  • Sam152 committed 2bedf2b on 8.x-2.x authored by flxa 
    Issue #2998670 by flxa, larowlan: Lazyload formatter should use a button...
larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
commented
Version: 8.x-2.x-dev » 8.x-1.x-dev
Status: Fixed » Reviewed & tested by the community

Chance of a commit against 8.1 too, we're using media_entity

sam152’s picture

sam152 commented
Status: Reviewed & tested by the community » Fixed

Sure thing.

  • Sam152 committed 9e201b3 on 8.x-1.x authored by flxa 
    Issue #2998670 by flxa, larowlan: Lazyload formatter should use a button...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.