eraeco / pay

Checkard is a peer-to-peer check payment system for ERA PAY.
MIT License
4 stars 0 forks source link

Stylefix safari #16

Closed jazzyjackson closed 1 month ago

jazzyjackson commented 1 month ago

Safari does some annoying things to prioritize text clarity over consistent scaling behavior

This switch to using a css transform instead of font-size: 25% has the same effect in safari in firefox, probably want to test other browsers before merging, I can't test on mobile right now because my iphone is forcing local IP to redirect to https. annoying.

cloudflare-pages[bot] commented 1 month ago

Deploying pay with  Cloudflare Pages  Cloudflare Pages

Latest commit: 28906c9
Status: ✅  Deploy successful!
Preview URL: https://20d94aea.pay-cwc.pages.dev
Branch Preview URL: https://stylefix-safari.pay-cwc.pages.dev

View logs

amark commented 1 month ago

Screenshot_20240405_041134_Brave Nope, broken on Android.

Cloudflare creates dev preview links

Tho easier is:

amark commented 1 month ago

to run this command ssh srv.us -R 1:localhost:8080 (or whatever port your localhost is on) it will give you back an HTTPS url you can hit from any browser in the world, its an ssh tcp tunnel to your localhost.

amark commented 1 month ago

also note checkard.com the "security features..." pretty much every word is separate line. Prob not actually matter... but I got so annoyed with some checks not processing I decided to try and go almost pixel perfect to my real check :P lol.

jazzyjackson commented 1 month ago

Good ssh trick.

I don't know how you feel about flex box, it's not 1999 but its like, 2014ish. Seems to do the trick.

amark commented 1 month ago

It's still rendering wrong

Screenshot_20240406_173510_Camera Screenshot_20240406_173416_Brave

On both Android and Laptop Brave/Chrome, didn't check others.

So no, this solution doesn't work, it's made things worse so please avoid flex. Try using font size px not pt see if that helps Safari and others, using the same div/css pattern as before. If costs start burning too much on this, I'll step in and do it, let me know.

amark commented 1 month ago

setting .details to width: 5em seemed to fix/work in both my Safari & Chrome.

jazzyjackson commented 1 month ago

I'd rather abandon this pull request then if you have a way to solve it without reworking it.