[wp-trac] [WordPress Trac] #64797: Admin Reskin: Primary disabled button should have visible border

[WordPress Trac]

#64797: Admin Reskin: Primary disabled button should have visible border
------------------------------------+---------------------
 Reporter:  wildworks               |       Owner:  (none)
     Type:  defect (bug)            |      Status:  new
 Priority:  normal                  |   Milestone:  7.0
Component:  Administration          |     Version:
 Severity:  normal                  |  Resolution:
 Keywords:  admin-reskin has-patch  |     Focuses:
------------------------------------+---------------------

Comment (by Joen):

 Thanks for the ping, thanks for the effort! Ultimately I'll defer to folks
 involved that are close to the metal, and understand the nuance.

 One of the opportunities we have with this initiative, is to sync up, two-
 ways, with the effort that is going into what we also refer to as the
 WordPress Design System: all the components, tokens, and patterns created
 for the React componentry as documented in
 [[https://wordpress.github.io/gutenberg/?path=/story/docs-introduction--
 page&storybook|Storybook]]. In that sense, Storybook is the source of
 truth, as it represents the shipping code. For the moment, that means a
 disabled primary button maintains the brand color but
 [[https://wordpress.github.io/gutenberg/?path=/story/components-button--
 primary&args=disabled:!true|has lighter text inside]]. The nuance there is
 that I believe @jameskoster and @mciampini is working on a new token
 system that will affect this.
 [[https://github.com/WordPress/gutenberg/pull/70674|This PR]] was a half-
 step towards that, but it was reverted due to some side-effects with
 intent to return to it.

 That means if we want to follow the path the design system is on, we can
 either go with the brand-color with reduced-opacity text inside to match
 **what exists today**, or we can go with the grayed background (and I
 believe dark text) where the design system is moving towards. Though note,
 in none of those situations does the primary button get a border.

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