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

WordPress Trac noreply at wordpress.org
Tue Oct 22 16:22:01 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 dev-feedback  |     Focuses:  accessibility
-------------------------------------------+----------------------------

Comment (by ryelle):

 Replying to [comment:42 SergeyBiryukov]:
 > I agree we should not violate our own coding standards :)

 I was following the [https://developer.wordpress.org/block-
 editor/contributors/develop/coding-guidelines/ Gutenberg css style,] I
 didn't realize it doesn't match the core standards. That said, I'm going
 to argue for an exception for About:

 - The new classes avoid any style bleed (either from other pages or into
 other pages). This about design is so different visually from the rest of
 wp-admin, and probably will continue to be different- **we'll want classes
 that aren't used anywhere else.**
 - We have the ability to create a new "design system" for the about pages-
 **we should try to create a consistent naming pattern.**
 - The BEM-style is already used in the editor, and the double-underscores
 are a way of separating out `[section]__[item]` in a more readable way.

 IMO, this use of underscores is a different kind of use than what's
 described in the core coding standards, where classes are more like just
 item names, rather than scoped to a section. Underscores are still not
 used in the `item` part of the class, they only show up as double-
 underscore separators.

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


More information about the wp-trac mailing list