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

Adding missing gap between page intro and the banner. #892

Merged

Conversation

message-dimke
Copy link
Contributor

Changes proposed in this Pull Request:

Puts back the margin removed by Pinterest css rules.

Closes #890

Screenshots:

Pinterest_‹_WordPress_Pinterest_—_WooCommerce

Detailed test instructions:

  1. Being at the develop branch, check the disconnected extension's Home screen.
  2. Checkout the fix/no-margin-above-the-banner branch and build its sources. nvm use && nom run build.
  3. Observe that the Home screen has a margin between the banner and the welcome block above it.

Changelog entry

Fix - Banner top margin missing.

@message-dimke message-dimke added type: bug The issue is a confirmed bug. priority: low The issue/PR is low priority—not many people are affected or there’s a workaround, etc. labels Jan 30, 2024
@message-dimke message-dimke requested a review from a team January 30, 2024 12:13
@message-dimke message-dimke self-assigned this Jan 30, 2024
@budzanowski
Copy link
Collaborator

Can we do this PR against the V5 integration branch so we will not need to do any rebases?

@github-actions github-actions bot added the changelog: fix Took care of something that wasn't working. label Jan 30, 2024
@message-dimke message-dimke changed the base branch from develop to pinterest-v5-integration-branch January 31, 2024 10:23
@message-dimke
Copy link
Contributor Author

Hey, @budzanowski ! It is done!

@message-dimke message-dimke marked this pull request as ready for review January 31, 2024 10:36
Copy link
Collaborator

@budzanowski budzanowski left a comment

Choose a reason for hiding this comment

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

Cool!

@message-dimke message-dimke merged commit 27efdd2 into pinterest-v5-integration-branch Jan 31, 2024
3 checks passed
@message-dimke message-dimke deleted the fix/no-margin-above-the-banner branch January 31, 2024 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog: fix Took care of something that wasn't working. priority: low The issue/PR is low priority—not many people are affected or there’s a workaround, etc. type: bug The issue is a confirmed bug.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Pinterest banner overlaps page content.
2 participants