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

Develop fix inputgroup text zoom #1878

Merged
merged 2 commits into from
Apr 16, 2024
Merged

Conversation

pethel
Copy link
Contributor

@pethel pethel commented Apr 12, 2024

Provar løsa fonskaleringproplematik på feilmeldninger. Dette er ett problem i alla inputgrupper, inklusive feedback #1835. Problemet er att dagen løsning lager padding under og feilmeldningen blir i prinsip absolut positionert der. Drar man upp fonten så ær paddingen. Det går nok ikke att bruke rem på paddingen under før vad hvis meldninen brekker på flere linjer.

image
image

Det jag gjort istellet er att laga en wrapper med minhøjde som er lika høy som en rad av texten. Dette er ingen perfekt løsning hvis man har zoom og høja meldninger så vill ting flytta seg men trot allt vare fullt synlig.

Jag tror dock dette funker bra før det man i utgangspunktet provade løsa. Korta meldniner som holder seg på en linje?

Hvis dere tester med firefox så funker det muligens ikke trots att nyare versioner skal l funka. Det er egentligen veldig bra support og dette er trots all en progressive enhancement.

Måtte endre markup. Breaking change?

https://caniuse.com/mdn-css_types_length_lh
image

@pethel pethel requested a review from a team as a code owner April 12, 2024 11:21
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-beach-0d62d0d03-1878.westeurope.2.azurestaticapps.net

@pethel pethel force-pushed the develop_fix-inputgroup-text-zoom branch from bcb3570 to 9de7d45 Compare April 16, 2024 11:59
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-beach-0d62d0d03-1878.westeurope.2.azurestaticapps.net

@pethel pethel merged commit 3b8bbab into develop Apr 16, 2024
3 checks passed
@pethel pethel deleted the develop_fix-inputgroup-text-zoom branch April 16, 2024 12:09
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