[wp-trac] [WordPress Trac] #47708: 5.3 about page

WordPress Trac noreply at wordpress.org
Tue Oct 29 20:25:59 UTC 2019


#47708: 5.3 about page
-------------------------------------------------+-------------------------
 Reporter:  karmatosed                           |       Owner:
                                                 |  johnbillion
     Type:  task (blessed)                       |      Status:  reviewing
 Priority:  normal                               |   Milestone:  5.3
Component:  Help/About                           |     Version:
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch commit i18n-change dev-    |     Focuses:
  reviewed                                       |  accessibility
-------------------------------------------------+-------------------------

Comment (by ryelle):

 > Looking further at the CSS class name change from .about-wrap to
 `.about__container`. There are currently 243 occurences of "about-wrap"
 found in 37 files (with 47708.12.diff applied). All of these files are
 CSS, there is no about-wrap anywhere in the HTML.

 Right, this new design changes a lot about the page, so the intention was
 to leave the existing CSS in place for a cycle for any plugins that might
 use the about classes in their own pages. We can remove this all in an
 early patch in 5.4, so folks have more time to update.

 It looks like this selector is found in all the files generated from
 `css/about.css`, the color schemes, and an ie-rtl file that I didn't know
 exists and looks like it hasn't been touched since before 4.8.

 > This will possibly bring other "visual regressions", especially if
 plugins add some content on the About screens.

 Aside from the notice, do you know of any plugins that do this? I don't
 think there are any other places _to_ hook in to add content.

 > Thinking the best course of action at this moment is to revert the
 selector name change, then add it "early" during 5.4

 The whole design of the page has changed, reverting the selector would
 only cascade more problems. Let's fix the notice issue, and keep an eye
 out for other issues.

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


More information about the wp-trac mailing list