Skip to content

Conversation

rickirunge
Copy link
Collaborator

fds-input og xfds-form-input havde properties der var beskrevet og brugt i eksempler som ikke virkede som det lovede

…et og brugt i eksempler som ikke virkede som det lovede
@rickirunge rickirunge added the bug Something isn't working label Jul 3, 2023
@rickirunge rickirunge self-assigned this Jul 3, 2023
@rickirunge rickirunge requested a review from whitewillow July 3, 2023 19:10
…oblemet med properties, og så opdagede jeg også lige eksempler der brugte "readonly" property som heller ikke virkede
@rickirunge
Copy link
Collaborator Author

rickirunge commented Jul 4, 2023

Jeg har rent bevidst fjernet flere af de "v-bind=attrs" da jeg syntes det i brug-scenarierne blev lidt uklart hvor en attribute så betød hvis man sætte den på, fordi det ikke altid gav helt mening at den så havnede på det component.

Eksempel
<xfds-form-input class="something">...</xfds-form-input>
Ville man her så forvente at class "something" blev sat på input-tag eller på form-group, eller måske sat på fds-input component (og blive sat på den wrapper-div der også indeholder prefix/suffix)

Jeg mener at hvis vi skal bruge den direkte "forward" af attributes så skal det være helt klar og intuitivt hvor de attributter bliver "forward'et" til.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant