[wp-trac] [WordPress Trac] #38023: Menus screen: simplify the Menu Settings controls

[WordPress Trac]

#38023: Menus screen: simplify the Menu Settings controls
---------------------------------------+--------------------------------
 Reporter:  afercia                    |       Owner:  rianrietveld
     Type:  defect (bug)               |      Status:  assigned
 Priority:  normal                     |   Milestone:  4.7
Component:  Menus                      |     Version:
 Severity:  normal                     |  Resolution:
 Keywords:  has-screenshots has-patch  |     Focuses:  ui, accessibility
---------------------------------------+--------------------------------

Comment (by xavortm):

 Thanks, @rianrietveld

 I think the problem is that I am selecting by HTML element, not a class,
 which I understand, though I wasn't sure if I had to introduce another
 classes or if we're using similar ones somewhere else. Also it would be
 nice to know if such a component is being used elsewhere before rewriting
 the wrappers.

 Having classes for selecting this component is a plus for sure in my
 opinion. Also another solution would be to change a bit the markup ( I
 only changed the html elements ) by wrapping the left and right part in
 divs and setting display:table, just like we have everywhere else around
 the dashboard, just without the table elements.

 I hope I'm not overcomplicating things here If it works as it is, then I'm
 happy ^^

--
Ticket URL: <https://core.trac.wordpress.org/ticket/38023#comment:11>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform