-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Consistent admin bar pattern #76
Conversation
Signed-off-by: Joe Fusco <joe.fusco@wpengine.com>
Signed-off-by: Joe Fusco <joe.fusco@wpengine.com>
Signed-off-by: Joe Fusco <joe.fusco@wpengine.com>
Signed-off-by: Joe Fusco <joe.fusco@wpengine.com>
@@ -55,7 +55,7 @@ | |||
<rule ref="WordPress.WP.I18n"> | |||
<properties> | |||
<property name="text_domain" type="array"> | |||
<element value="wp-graphql" /> | |||
<element value="wpgraphql-ide" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👌🏻
@@ -3,7 +3,7 @@ import { Drawer as VaulDrawer } from 'vaul'; | |||
import { useDispatch, useSelect } from '@wordpress/data'; | |||
|
|||
export function EditorDrawer( { children } ) { | |||
const buttonLabel = 'GraphQL IDE'; | |||
const buttonLabel = '🚀'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm cool with this as a short-term solution, but I anticipate re-visiting this at some point as I don't think the 🚀 really conveys to anyone other than you an I what this button really is meant for 🤔
But, at the moment I don't have another suggestion. Just thinking we might revisit this at some point. 🤷🏻♂️
These changes help to align the UX with what is currently expected, moving the current drawer button to be a 🚀 (for now)