Skip to content

Add CSS classname to fix the negative margins not appearing in the Navigation Screen#67825

Merged
t-hamano merged 1 commit into
WordPress:trunkfrom
im3dabasia:css/add-class-to-sidebar-screen
Dec 11, 2024
Merged

Add CSS classname to fix the negative margins not appearing in the Navigation Screen#67825
t-hamano merged 1 commit into
WordPress:trunkfrom
im3dabasia:css/add-class-to-sidebar-screen

Conversation

@im3dabasia

@im3dabasia im3dabasia commented Dec 11, 2024

Copy link
Copy Markdown
Contributor

Issue: #67369

Related PR: #67575

Related comment: comment

What?

Due to regression introduced by #66851. The CSS classname edit-site-sidebar-navigation-screen-main was removed from the <ItemGroup>, this caused negative margins not to be applied

Why?

To apply negative margins.

@github-actions

github-actions Bot commented Dec 11, 2024

Copy link
Copy Markdown

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: im3dabasia <im3dabasia1@git.wordpress.org>
Co-authored-by: jameskoster <jameskoster@git.wordpress.org>
Co-authored-by: t-hamano <wildworks@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@im3dabasia

Copy link
Copy Markdown
Contributor Author

Hi @t-hamano, please check my PR when you have a moment. Thanks!

@jameskoster jameskoster left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing this :)

@t-hamano t-hamano left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks for the quick fix 👍

@t-hamano t-hamano added [Type] Regression Related to a regression in the latest release [Package] Edit Site /packages/edit-site props-bot Manually triggers Props Bot to ensure the list of props is up to date. labels Dec 11, 2024
@github-actions github-actions Bot removed the props-bot Manually triggers Props Bot to ensure the list of props is up to date. label Dec 11, 2024
@t-hamano t-hamano merged commit 6470e09 into WordPress:trunk Dec 11, 2024
@github-actions github-actions Bot added this to the Gutenberg 20.0 milestone Dec 11, 2024
yogeshbhutkar pushed a commit to yogeshbhutkar/gutenberg that referenced this pull request Dec 18, 2024
…tent (WordPress#67825)

Co-authored-by: im3dabasia <im3dabasia1@git.wordpress.org>
Co-authored-by: jameskoster <jameskoster@git.wordpress.org>
Co-authored-by: t-hamano <wildworks@git.wordpress.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Package] Edit Site /packages/edit-site [Type] Regression Related to a regression in the latest release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants