[wp-trac] [WordPress Trac] #46901: 5.2 About Page

WordPress Trac noreply at wordpress.org
Tue Apr 30 08:42:35 UTC 2019


#46901: 5.2 About Page
---------------------------------------+-------------------------
 Reporter:  desrosj                    |       Owner:  karmatosed
     Type:  task (blessed)             |      Status:  assigned
 Priority:  normal                     |   Milestone:  5.2
Component:  Help/About                 |     Version:
 Severity:  normal                     |  Resolution:
 Keywords:  has-patch has-screenshots  |     Focuses:
---------------------------------------+-------------------------

Comment (by afercia):

 @SergeyBiryukov [attachment:"46901-tabs.3.diff"] fixes the tabs in the
 "About" pages but also changes the tabs in Appearance > Menus, on the
 multisite pages, or the ones used by plugins. Also, I'm not sure there's
 the need for some of the changes there, unless I'm missing something:
 - not sure there's the need to add `.about-wrap nav` to `.about-wrap h2
 {}` in about.css, as those CSS properties either don't apply to the tabs
 links or are overridden by other rules
 - `line-height: 1.33333333;`: while this fixes the tabs in the About page,
 it makes the other tabs smaller. This `line-height` needs to be the
 equivalent to `24px` with any font-size
 - not sure there's the need for `.wrap nav.nav-tab-wrapper` as there's
 already `.nav-tab-wrapper`
 - not sure there's the need for higher specificity for `.wrap nav .nav-
 tab`
 - we should re-add the fix for the bottom border: the rule should override
 the white border for hover / focus / focus:active.

 I've tried to address these points in [attachment:"46901-tabs.4.diff"]. I
 guess at this point of the release cycle we should try to be very
 conservative and change only what needs to be changed. Some testing would
 be nice :) When testing, please check also the tabs in Appearance > Menus
 and tabs added by plugins, for example Yoast SEO, comparing with the tabs
 in WordPress 5.1.

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


More information about the wp-trac mailing list