Skip to content
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

Removed the bottom-margin for the radio-control #19340

Merged
merged 1 commit into from Dec 29, 2019

Conversation

@mapk
Copy link
Contributor

mapk commented Dec 27, 2019

Description

Fixes #17929.
There was an extra bottom-margin on the radio-control as indicated in #17929. I edited the radio-control to remove this margin.

How has this been tested?

Tested locally. I checked through some settings, although the Latest Posts block may be the only block settings that utilize this particular combination of cascaded styles.

Screenshots

radio-control 2019-12-26 22_06_53

Types of changes

Non-breaking CSS changes.

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows the accessibility standards.
  • My code has proper inline documentation.
  • I've included developer documentation if appropriate.
  • I've updated all React Native files affected by any refactorings/renamings in this PR. .
@mkaz
mkaz approved these changes Dec 29, 2019
Copy link
Member

mkaz left a comment

Confirmed fix, looks good 👍

@mkaz mkaz merged commit 1c14c2a into master Dec 29, 2019
2 checks passed
2 checks passed
pull-request-automation
Details
Travis CI - Pull Request Build Passed
Details
@mkaz mkaz deleted the update/radio-control-bottom-margin branch Dec 29, 2019
@youknowriad youknowriad added this to the Gutenberg 7.2 milestone Jan 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.