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

PLANET-7744 Fix Media & Text padding in large screens #2525

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mleray
Copy link
Contributor

@mleray mleray commented Feb 7, 2025

Description

See PLANET-7744

Otherwise the content is impossible to read. I also made some design changes following Houssam's recommendations.

Testing

Make sure to check all screen sizes in the following pages:

Please also make sure that the Page Header pattern still look fine, for example on this page

@mleray mleray self-assigned this Feb 7, 2025
planet-4 added a commit to greenpeace/planet4-test-neptune that referenced this pull request Feb 7, 2025
/unhold 411d5ad9-78d0-4fde-a5b1-2fb9846dd84b
@planet-4
Copy link
Contributor

planet-4 commented Feb 7, 2025

Test instance is ready 🚀

🌑 neptune | admin | blocks report | CircleCI | composer-local.json

⌚ 2025.02.12 10:15:37

@mleray mleray added the UAT Needed This PR requires User Acceptance Tests before merge label Feb 7, 2025
@mleray mleray force-pushed the fix-media-text-padding branch from bbf6cad to e68349c Compare February 7, 2025 12:17
planet-4 added a commit to greenpeace/planet4-test-neptune that referenced this pull request Feb 7, 2025
/unhold 64f21931-2aa9-49a7-a3a5-619e25ec0986
@mleray mleray force-pushed the fix-media-text-padding branch from e68349c to ce7aae9 Compare February 7, 2025 12:23
planet-4 added a commit to greenpeace/planet4-test-neptune that referenced this pull request Feb 7, 2025
/unhold f9b874d4-76f5-4fb2-84c1-1653a4b793c8
@mleray mleray marked this pull request as ready for review February 7, 2025 12:44
@mleray mleray requested review from a team, sagarsdeshmukh and GP-Dan-Tovbein and removed request for a team February 7, 2025 12:44
@mleray mleray added UAT Passed User Acceptance Tests passed Review and removed UAT Needed This PR requires User Acceptance Tests before merge labels Feb 7, 2025
Otherwise the content is impossible to read
@mleray mleray force-pushed the fix-media-text-padding branch from ce7aae9 to acf8adb Compare February 12, 2025 10:05
planet-4 added a commit to greenpeace/planet4-test-neptune that referenced this pull request Feb 12, 2025
/unhold b1fb11dd-28b2-4183-a5e3-a65946d07fa8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Review [Test Env] neptune UAT Passed User Acceptance Tests passed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants