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

Update: Increase bottom padding on gallery image caption #8459

Closed
wants to merge 2 commits into from
Closed

Update: Increase bottom padding on gallery image caption #8459

wants to merge 2 commits into from

Conversation

johnwatkins0
Copy link
Member

@johnwatkins0 johnwatkins0 commented Aug 3, 2018

Description

This resolves #8018 by increasing the bottom padding of gallery image captions by 4px.

How has this been tested?

Visually verified the change in the editor (in English and in an RTL language) and in the published post.

Types of changes

One character change in an SCSS file.

@aduth aduth added [Feature] Blocks Overall functionality of blocks Needs Design Feedback Needs general design feedback. labels Sep 13, 2018
@karmatosed
Copy link
Member

In theory this could be a good addition but the PR needs refreshing. Could you work on that please @johnwatkins0?

@karmatosed karmatosed added the [Status] Stale Gives the original author opportunity to update before closing. Can be reopened as needed. label Oct 8, 2018
@johnwatkins0
Copy link
Member Author

Updated to address staleness, @karmatosed.

@karmatosed karmatosed removed the [Status] Stale Gives the original author opportunity to update before closing. Can be reopened as needed. label Oct 11, 2018
@jasmussen
Copy link
Contributor

Be sure to check the contrast is sufficient when moving the text upwards.

@karmatosed karmatosed added this to the WordPress 5.0.1 milestone Nov 22, 2018
@karmatosed karmatosed removed the Needs Design Feedback Needs general design feedback. label Nov 22, 2018
@youknowriad youknowriad modified the milestones: WordPress 5.0.1, 4.8 Dec 4, 2018
@youknowriad youknowriad modified the milestones: 4.8, 4.9 Dec 19, 2018
@gziolo
Copy link
Member

gziolo commented Jan 31, 2019

What is missing to land this one?

@kjellr
Copy link
Contributor

kjellr commented Jan 31, 2019

This works well, but just needs a tiny adjustment to the gradient. I was unable to pull the branch down to test (I think the source repo has been deleted), so I created a fresh PR with both changes: #13623

I'll close this issue and we can migrate over there instead.

Thanks for getting this going, @johnwatkins0!

@kjellr kjellr closed this Jan 31, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Blocks Overall functionality of blocks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Multi-line gallery captions are visually imbalanced.
7 participants