- Issue created by @donquixote
- 🇩🇪Germany donquixote
The change seems simple, but the conditionality is more tricky.
We should have some kind of feature detection support in core.
- 🇩🇪Germany donquixote
This patch is a first proof of concept, definitely not to be committed as-is.
It lacks the conditionality, and it lacks tests.It is ok for people who want to patch both core and field_group to get the machine name column.
- Status changed to Needs work
7 months ago 7:22am 8 September 2024 - 🇩🇪Germany Anybody Porta Westfalica
Nice @donquixote! :) Can you please turn this into a MR?
- First commit to issue fork.
- Merge request !97Issue #3471822: Add "machine_name" column in field_ui to support Drupal 11.x. → (Open) created by duaelfr
- 🇫🇷France duaelfr Montpellier, France
@anybody There you go.
Now ✨ Show machine name in "Manage form display" and "Manage display" table row Fixed has been released. This causes an issue where the weight field for the field_group line is displayed. Dragging the field_group around is still functional but it can be troubling for end users. - 🇧🇪Belgium nils.destoop
The fix will need to be smarter. As the column only exists for 11.1. We don't want to break the screens of sites running an older version.
- 🇫🇷France duaelfr Montpellier, France
I don't know why I was convinced that 4.X was D11 only. My bad.
Is it better like this? - 🇩🇪Germany donquixote
I would suggest to use some kind of feature detection instead of version detection.
This way we also support sites that apply a patch.
Not sure yet how that would work.. - First commit to issue fork.
- 🇬🇧United Kingdom scott_euser
Not the cleanest as there is no array key, but would this be okay?
- 🇧🇪Belgium joevagyok
The solution is looking good for me! Thanks @scott_euser
I've tried the patch https://www.drupal.org/project/field_group/issues/3471822#comment-15755143 📌 Add "Machine name" column in field_ui to align with latest core Active and it works for me (using Drupal 11.1.5 and Firefox 136.0)