Skip to content

Conversation

rosier
Copy link
Contributor

@rosier rosier commented Sep 20, 2025

Also fixes the alignment of the input fields and the add-ons

@stof
Copy link
Member

stof commented Sep 22, 2025

is the icon still a trigger to open the flatpickr picker ?

@rosier
Copy link
Contributor Author

rosier commented Sep 22, 2025

is the icon still a trigger to open the flatpickr picker ?

No, the button does nothing that's why I opted to remove the button and make the calendar icon the same as the tags icon

The flatpickr calendar opens when the input field gets focus or is clicked.

@stof
Copy link
Member

stof commented Sep 22, 2025

I think the intent was that the button would also trigger the picker (see the aria-label for instance), but the implementation was broken.
I think it would make sense to update assets/js/flatpicker.js to also use the button as trigger.

@rosier
Copy link
Contributor Author

rosier commented Sep 23, 2025

Hmm, I see. It may take some time until I get to that.

@rosier rosier force-pushed the bs5-form-fields branch 2 times, most recently from 43a7cc6 to 31840dc Compare October 1, 2025 15:25
@rosier rosier changed the title [Bootstrap v5] Update form fields [flatpickr] Fix flatpickr calendar buttons Oct 1, 2025
@rosier rosier changed the title [flatpickr] Fix flatpickr calendar buttons [flatpickr] Fix flatpickr calendar buttons and localization Oct 1, 2025
@rosier
Copy link
Contributor Author

rosier commented Oct 3, 2025

I did add a toggle and a clear button

@javiereguiluz
Copy link
Member

Jan, thanks for fixing this and for the effort you put on it 🙇

@javiereguiluz javiereguiluz merged commit dfd1c37 into symfony:main Oct 3, 2025
8 checks passed
@rosier rosier deleted the bs5-form-fields branch October 5, 2025 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants