Flytt visning av helpLabel til over <input> i InputGroup #3885
Conversation
|
✅ Forhåndsvisning: https://jokul.fremtind.no/preview/3876-move-label-2/ Forhåndsvisningen blir tilgjengelig innen et par minutter. Den fjernes automatisk når pull requesten lukkes. |
The deprecated props `helpLabel` and `errorLabel` on the `SupportLabel` component has been removed. Use `label` and `labelType` instead. BREAKING CHANGE: errorLabel and helpLabel props will now be ignored and are removed from the exported types
Removed the `supportLabelProps` prop from `InputGroup`, if you need to control the label's props you can use a `SupportLabel` element for either `helpLabel` or `errorLabel`. BREAKING CHANGE: `supportLabelProps` no longer exists
Previously, providing both props would mean only showing the error ISSUES CLOSED: #3876
|
Alt ser bra ut (annet enn at jeg fant en eksisterende bug, men tar ikke den her)! Men jeg lurer på om vi bør oppdatere eksempler og dokumentasjon litt. Nå er det mye dobbelt info i feilmeldinger i eksemplene, ettersom hjelpeteksten ikke lenger skjules, og vi kan fjerne/endre avsnittet som handler om nettopp repetisjon av innhold :) I tillegg må vi sørge for å få koordinert release av dette med design på en god måte! (tagger @fremtindjonas ) |
|
Det ble protester på Designsystem forum så denne må vi bare legge litt på is |
piofinn
added
the
⛔️ Blokkert
|
Hva enn vi ender med å gjøre her så tenker jeg denne branchen er såpass gammel nå at det er like greit å ta endringene fra scratch i ny monopakke dersom det skal gjøres noe med dette. |
closes: #3876
🎯 Sjekkliste
pnpm buildogpnpm ci:testgir ingen feil