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

🐛 Fixed feedback buttons' layout breaking in different languages, in newsletters #21752

Merged
merged 5 commits into from
Dec 2, 2024

Conversation

dvdwinden
Copy link
Contributor

@dvdwinden dvdwinden commented Nov 28, 2024

fixes https://linear.app/ghost/issue/DES-974/feedback-and-comment-icons-when-translated-cause-overlapping-in-email

  • Since we added i18n, the feedback/comment button layout would break in languages where the text is longer.
  • We've revised the button layout to be centred. On smaller (mobile) resolutions, we hide the labels and rely solely on the icons.

Before
french-before


After
newsletter-buttons-desktop-french


newsletter-buttons-mobile

@dvdwinden dvdwinden force-pushed the feedback-comment-email-fix branch 2 times, most recently from db12297 to 55ead35 Compare November 28, 2024 14:20
@dvdwinden dvdwinden force-pushed the feedback-comment-email-fix branch from b1b29d9 to a884ac3 Compare November 28, 2024 14:27
@cathysarisky
Copy link
Contributor

closes #21700 .

@dvdwinden
Copy link
Contributor Author

closes #21700 .

Thank you, @cathysarisky. We had another issue for this internally, and I hadn't yet flagged it here.

Fix is awaiting review, and will go out early this week.

@dvdwinden dvdwinden merged commit df53eb0 into main Dec 2, 2024
21 checks passed
@dvdwinden dvdwinden deleted the feedback-comment-email-fix branch December 2, 2024 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants