Opened 3 years ago

Closed 2 months ago

#14799 closed defect (bug) (wontfix)

Twenty Ten Theme - adding a back to top link

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

Description

One for accessibility pundits, a back to top link situated in the footer.

Attachments (2)

back-link.patch (1.2 KB) - added by elfin 3 years ago.
back-link-dre-v1.patch (2.4 KB) - added by dremeda 2 years ago.
Moved backlink left and adjusted styles.

Download all attachments as: .zip

Change History (12)

  • Keywords twentyten-a11y added

#14872 parent ticket.

comment:3 follow-up: ↓ 4   elfin3 years ago

this one wasn't in that ticket...

comment:4 in reply to: ↑ 3   nacin3 years ago

Replying to elfin:

this one wasn't in that ticket...

That's fine. I'm just putting them all in one place so any feedback on accessibility or from the designers can be targeted. They're now all linked in once place.

+1 to the idea, but:

  • The color should be grey, like the "Proudly powered by WordPress" link
  • It should sit inline, to the right of the homepage link, inside the site-info div in the footer.

elfin3 years ago

Patch updated, although I'm a little unhappy about having 2 links next to each other.

This looks better, but as I mentioned above I think it'd make more sense to the right of the homepage link, not the powered by link.

  • Keywords twenty-eleven added
  • Milestone changed from Awaiting Review to WordPress.org

dremeda2 years ago

Moved backlink left and adjusted styles.

  • Component changed from Accessibility to Bundled Theme
  • Keywords twenty-eleven removed
  • Milestone WordPress.org deleted
  • Resolution set to wontfix
  • Status changed from new to closed

I think it's OK to "wontfix" this one.

Note: See TracTickets for help on using tickets.