WordPress.org
Get WordPress

Making WordPress.org

Opened 11 years ago

Closed 11 years ago

#1054 closed defect (bug) (fixed)

Theme Directory: Theme navigation is out of place for longer strings

Reported by: ocean90 Owned by: obenland
Priority: normal Milestone:
Component: Theme Directory Keywords: has-patch
Cc:

Description

See attached screenshot.

Attachments (2)

1054.png​ (78.9 KB ) - added by ocean90 11 years ago.
meta-1054.patch​ (832 bytes ) - added by ramiy 11 years ago.

Download all attachments as: .zip

Change History (9)

@ocean90
11 years ago

#1 @Otto42
11 years ago

.theme-navigation needs a min-width to keep this from happening, however, if any other pages have longer translations, then that width will need to be higher.

Alternatively, .theme-navigation .close can be limited to a max-width as well, though it looks weird when it wraps the text.

#2 @jcastaneda
11 years ago

You could put the navigation below the theme's name on smaller screens.

#3 @ocean90
11 years ago

I just noticed that this happens for the English string as well.

@ramiy
11 years ago

#4 @ramiy
11 years ago

  • Keywords has-patch added; needs-patch removed

The patch has an alternative solution: The arrow before the "Return to Themes List" text, under .close, is hidden on small devices - @media only screen and (max-width: 480px)

It gives us few extra pixels. And not requires us to do any layout changes.

I believe it's a simple and elegant solution.

This ticket was mentioned in ​Slack in #meta by ramiy. ​View the logs.
11 years ago

#7 @obenland
11 years ago

  • Owner set to obenland
  • Resolution → fixed
  • Status new → closed

In 2142:

WordPress.org Themes: Prevent theme navigation from wrapping on small screens.

Props ramiy for initial patch.
Fixes #1054.

Note: See TracTickets for help on using tickets.