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

Fix initial select value not working properly (#40) #55

Closed
wants to merge 5 commits into from

Conversation

flx-sta
Copy link

@flx-sta flx-sta commented Feb 27, 2024

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):

Fix #40

Test URLs:

URL for testing:

flx-sta and others added 5 commits January 26, 2024 11:48
the `<select>` element does not support the `value` attribute thus the `option.value` needs to be compared to the `fd.Value` instead of ~~`select.value`~~
update mountpoint
update mount-point
@flx-sta flx-sta closed this Feb 27, 2024
@flx-sta flx-sta reopened this Feb 27, 2024
Copy link

aem-code-sync bot commented Feb 27, 2024

Page Scores Audits Google
/form/form-demo PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

@flx-sta flx-sta closed this Feb 27, 2024
@flx-sta flx-sta reopened this Feb 27, 2024
@flx-sta
Copy link
Author

flx-sta commented Feb 27, 2024

I give up. The PSI-Check for the PRs is just reckless in this repository...

@shsteimer shsteimer mentioned this pull request Feb 27, 2024
@shsteimer
Copy link
Collaborator

@felix-staud closing this in favor of #57, which I based on your branch as a starting point.

Thanks for your contribution

@shsteimer shsteimer closed this Feb 27, 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

Successfully merging this pull request may close these issues.

Initial value for selects not working (form block)
2 participants