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

layout somehow messed up #126

Open
whysthatso opened this issue Jan 28, 2023 · 3 comments
Open

layout somehow messed up #126

whysthatso opened this issue Jan 28, 2023 · 3 comments

Comments

@whysthatso
Copy link

image

not sure what's happening here

running latest version of the gem in rails 7

any idea?

@whysthatso
Copy link
Author

whysthatso commented Apr 28, 2023

turns out replacing "Letter" with "Mails" was already sufficient to prevent the buttons "Clear" and "Refresh" from overflowing onto the next line. Not sure how better to fix it, probably buttons could go a bit smaller? This might be a font rendering issue (firefox on linux here, i've seen font issues like this before)

@fgrehm
Copy link
Owner

fgrehm commented May 14, 2024

@whysthatso sorry for the long silence here, is this still a problem for you?

@whysthatso
Copy link
Author

no worries, @fgrehm i appreciate your gift, and there's no need to be sorry :)

the issues indeed still exists. i think it stems from the css value .col width=350px; and the resulting fixed width. the font might be slightly differently rendered in different browser, hence 'letters' being slightly too wide to accommodate the two buttons. i'd suggest to exchange 'letters' for 'mail' and that should fix this edge case.

whysthatso added a commit to whysthatso/letter_opener_web that referenced this issue May 19, 2024
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

No branches or pull requests

2 participants