WordPress.org

Make WordPress Core

Opened 8 days ago

Closed 6 days ago

Last modified 6 days ago

#42491 closed defect (bug) (fixed)

Theme Editor: Styling skewed for notice displayed in file list when editing a child theme

Reported by: ndiego Owned by: westonruter
Milestone: 4.9 Priority: normal
Severity: normal Version: trunk
Component: General Keywords: has-patch dev-reviewed has-screenshots commit
Focuses: ui Cc:

Description

When editing files for child theme in the new editor, the styling is a bit skewed because there is an alert added letting the user know white are editing a child theme. See a screenshot here https://www.dropbox.com/s/yp0zi2egkcd8uep/wp-file-editor.png?dl=0. I suggest the alert be moved into the <ul> that makes up the file list and then all that is needed is some additional padding tweets.

Attachments (6)

wp-file-editor_preview.png (31.2 KB) - added by westonruter 8 days ago.
42491.0.diff (2.1 KB) - added by westonruter 7 days ago.
applied-42491.0.png (85.1 KB) - added by westonruter 7 days ago.
42491.1.diff (2.3 KB) - added by westonruter 7 days ago.
applied-42491.1.png (86.1 KB) - added by westonruter 7 days ago.
Fix applied on mobile
42491.2.diff (2.4 KB) - added by melchoyce 7 days ago.

Download all attachments as: .zip

Change History (15)

#1 @melchoyce
8 days ago

Thanks for catching. Sounds like a good solution 👍

Are you interested in making a patch, or should we open it up to anyone who wants to?

#2 @johnbillion
8 days ago

  • Component changed from Editor to Plugins
  • Focuses ui added
  • Keywords needs-patch added
  • Milestone changed from Awaiting Review to 4.9

@westonruter
7 days ago

#3 @westonruter
7 days ago

  • Keywords has-patch dev-feedback added; needs-patch removed
  • Owner set to melchoyce
  • Status changed from new to reviewing

@melchoyce What do you think of applied-42491.0.png?

@westonruter
7 days ago

@westonruter
7 days ago

Fix applied on mobile

#4 @westonruter
7 days ago

  • Keywords has-screenshots added

#5 @westonruter
7 days ago

  • Component changed from Plugins to General
  • Summary changed from File editor styling skewed when editing a child theme to Theme Editor: Styling skewed for notice displayed in file list when editing a child theme

@melchoyce
7 days ago

#6 @melchoyce
7 days ago

Made a super tiny padding tweak in 42491.2.diff. I think it's good to go now!

#7 @westonruter
6 days ago

  • Keywords dev-reviewed commit added; dev-feedback removed
  • Owner changed from melchoyce to westonruter
  • Status changed from reviewing to accepted

#8 @westonruter
6 days ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In 42155:

Theme Editor: Fix styling of child theme notice.

Props westonruter, melchoyce, ndiego.
See #24048.
Fixes #42491 for trunk.

#9 @westonruter
6 days ago

In 42156:

Theme Editor: Fix styling of child theme notice.

Props westonruter, melchoyce, ndiego.
See #24048.
Fixes #42491 for 4.9.

Note: See TracTickets for help on using tickets.