Opened 3 years ago

Last modified 8 months ago

#14792 new enhancement

Twenty Ten Theme - gallery changes

Reported by: elfin Owned by:
Priority: normal Milestone: WordPress.org
Component: Bundled Theme Version:
Severity: normal Keywords: twentyten-a11y
Cc:

Description

With reference to #14791 those changes will produce an ugly 10px wide border in red on focus/hover etc. This patch does fix this, at the expensive of top/bottom margins. This therefore may need an additional patch before it can be applied.

Attachments (1)

galleryfix.patch​ (510 bytes) - added by elfin 3 years ago.

Download all attachments as: .zip

Change History (7)

elfin — 3 years ago

comment:1   elfin — 3 years ago

ref previous ticket #14782

comment:2   nacin — 3 years ago

  • Keywords twentyten-a11y added

comment:3   iammattthomas — 3 years ago

See my comment on #14791; highlighting the linked image borders is great but we shouldn't have to sacrifice the large image border.

Consider that with each of these, we have both an <a> and an <img> that can have borders, backgrounds, and padding; so we've got lots of options for how to style those borders (and which parts to change on hover).

comment:4   nacin — 3 years ago

  • Milestone changed from Awaiting Review to Future Release

comment:5   SergeyBiryukov — 10 months ago

  • Component changed from Accessibility to Bundled Theme

comment:6   SergeyBiryukov — 8 months ago

  • Milestone changed from Future Release to WordPress.org
Note: See TracTickets for help on using tickets.