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

UI/Input/Container: add JS to retrieve values clientside #8016

Open
wants to merge 6 commits into
base: trunk
Choose a base branch
from

Conversation

nhaagen
Copy link
Contributor

@nhaagen nhaagen commented Sep 5, 2024

This adds some JS to the UI Form to retrieve its values client side.
The JS builds an internal tree over the container's elements by their names rather than their position in the DOM.
Consumers may search for a FormNode and retrieve values from all elements below.
Optional or Switchable Groups will only relay values if selected.

Please note that this is a basic addition without any visible effect, yet, but it will be very handy in folding forms/groups/filters and visualize current selections of otherwise invisible inputs.

(was: #7137)

@nhaagen nhaagen force-pushed the 10/UI/Input/Container/JS2 branch 2 times, most recently from 7ad1145 to 6f7b2af Compare September 9, 2024 10:24
@klees klees self-assigned this Sep 17, 2024
@nhaagen nhaagen force-pushed the 10/UI/Input/Container/JS2 branch from 6f7b2af to ea05fe6 Compare November 12, 2024 08:16
@klees
Copy link
Member

klees commented Jan 14, 2025

Hi @nhaagen,

really looking forward to have this!

Could you please:

  • Rebase this (we have removed novalidate for forms in the meantime...)
  • Join me for me a meeting to talk this over, after you have looked into this again?

Thanks!

@nhaagen nhaagen force-pushed the 10/UI/Input/Container/JS2 branch from 8fe19d3 to bd4240e Compare January 16, 2025 07:41
@nhaagen nhaagen marked this pull request as ready for review January 23, 2025 06:37
@klees klees assigned Amstutz and thibsy and unassigned klees Jan 24, 2025
@klees
Copy link
Member

klees commented Jan 24, 2025

Hi @Amstutz and @thibsy,

please have a look into this. This is a precondition to iterate on filters.

Thanks!

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

Successfully merging this pull request may close these issues.

4 participants