Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.The facet add form can be much simpler to improve site builder UX. I suggest following changes:
- Move weight and url_alias to the display tab
- Auto fill name using js with the selected field name.
- Change field order (source => field => name)
- Some small copy changes in the facet add form (field names & description)
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | clean_up_facet_add_form-2712505-7.patch | 20.87 KB | StryKaizer |
Comments
Comment #2
StryKaizerComment #5
borisson_This makes the form look a lot better, such awesome. Tests still fail though.
Comment #6
borisson_Changing component.
Comment #7
StryKaizerForgot core_search_facet tests (lets move those in the same group as facets, so I dont need 2 testgroups to check).
Now with those fixed too.
Comment #8
borisson_@StryKaizer: created an issue for that over at #2712519: Change the @group annotation for core search tests. If tests are green, I'm really happy with this patch.
Comment #10
borisson_Fixed a small coding style thing on commit in the .js file, overall very happy with the changes. Thank you very much!