-
Notifications
You must be signed in to change notification settings - Fork 7
Use a label to highlight the connected block #455
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
Conversation
Test this PR in WordPress Playground. |
title={ sprintf( __( 'Remote Data Block: %s', 'remote-data-blocks' ), remoteDataTitle ) } | ||
> | ||
<PanelBody title={ __( 'Remote Data', 'remote-data-blocks' ) }> | ||
<p style={ { fontSize: 'small', color: 'rgb(117, 117, 117)' } }> |
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.
The fontSize and colour are used to replicate the Gutenberg helper text shown on a lot of their components. Unfortunately, PanelBody
doesn't have help
as an option on it so this is the only way to show it.
I have set the fontSize to be small, and set the colour to be the same within the helper text. If anyone has a better suggestion, I'm happy to update this.
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.
Can this be targeted with a className instead of inline style?
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.
Done in the latest commit 👍🏾
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.
One question, otherwise 👍
Description
This PR is meant to incorporate the feedback from @jarekmorawski, to build on #443. It does the following:
Remote Data