-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Description
There are several flaws with the new admin table filter and search implementation.
- One can only choose one filter at the time
- The fulltext search submits on key stroke
- Scope selection disappear when a filter or search is happening
- You cannot see a filter being enabled without opening a filter dropdown again
Most of that because there is no submit button, lacking any semantics of this form. This is why the code needs to be overly complex, which makes it very hard to maintain or to fix bugs and contribute.
This makes it necessary to run all admin feature specs in full JS enabled browsers, making them slow and flaky. As we are all aware of (nearly every single PR has flaky admin specs).
I like JS to enhance UX, but for simple things like search forms we need to make sure it works without it. Also we have Turbo, wich make forms submits very fast. Stimulus is not necessary for this at all.
Since we are currently laying the foundation for the new admin and encourage the community to contribute, we need to give good examples on how we want to build a maintainable admin interface. The current admin does not give such a good example at the moment.
I am a bit alarming here as we approach to finalize the admin and we as the core team will have to maintain this code for a very long time. Which I am not sure I am willing to.
Metadata
Metadata
Assignees
Labels
Type
Projects
Status