Needs review
Project:
Gin Admin Theme
Version:
8.x-3.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
20 Mar 2022 at 20:05 UTC
Updated:
15 Nov 2024 at 12:46 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
meladawy commentedThis should enable the gin actions back on the taxonomy forms.
Comment #3
meladawy commentedComment #4
saschaeggiI'm changing this into a feature request, as this was until now not part of Gin.
We have to think of that should be placed in the sidebar – or if we need another option to just make the form actions sticky on those forms.
So changing this back to needs work for this reason :)
Comment #6
saschaeggiComment #7
saschaeggiComment #8
volkerk commentedI think we should definitely go for a complete integration including sidebar. We did this for thunder using
hook_gin_content_form_routes(). Additionally, `relations` and `path_settings` details should be moved to the meta sidebar as well, similar to what is done at this thunder branch.The current state of the implementation splits the route determination for sticky action buttons from
isContentForm()tostickyActionButtons(). If we decide to do that we should add another layer ofhook_andhook_alterapi. I would also suggest that we add hooks for adding and removing actions, is there an issue yet?Comment #9
saschaeggi@volkerk maybe you can share a screenshot of how that looks in Thunder at the moment.
Comment #10
volkerk commentedSure
Comment #11
sokru commentedNot most elegant solution, but works for me.
I didn't add
entity.taxonomy_vocabulary.add_formorentity.taxonomy_vocabulary.edit_formroutes to follow same UI as content type edit forms.Comment #14
dieterholvoet commentedThe last patch only works for vocabularies named 'tags'. I created a MR based on the patch and the changes from the Thunder distribution.
Comment #15
santosh_verma commentedI have tested the MR, But it has conflict, @DieterHolvoet can you please check and update the MR.
Thanks in Advance :)
Comment #16
santosh_verma commentedComment #17
dieterholvoet commentedMy bad, I forgot to push my changes. Should work now.
Comment #18
santosh_verma commentedComment #19
santosh_verma commentedI have tested the MR 237 applied successfully. and solution is working. i have shared the screenshot.
Before

after

Thanks
Comment #20
santosh_verma commentedComment #21
iheb.attia commentedThe patch #2
does not work on taxonomy translation pages. I've updated the patch to include the taxonomy translation add and edit route names.
Comment #22
dieterholvoet commented@iheb.attia work is being done in the MR, you should test those changes instead of the patch from #2.
Comment #23
justanothermark commentedA couple of suggestions from reviewing the MR:
1. Should we undo moving the `langcode` field in to the sidebar? The node form doesn't do this so doing it for taxonomy terms makes them inconsistent.
2. I think we should include the `entity.taxonomy_term.content_translation_edit` route as well.
Comment #26
hydra commentedJust wanted to drop that here as an alternative solution to the patch https://www.drupal.org/project/gin_everywhere
Comment #27
volkerk commentedWe need a more in depth review other than posting two screenshots.
Comment #28
hudriHere is a patch file for RC11, suitable for composer patches:
drupal.org/files/issues/2024-06-19/gin_3270626_cherrypick_for_rc11.patch
Just a cherry-pick of commit ee8cb77e refitted for RC11.
(Please do not use this patch file for issue development, use merge requests to resolve the issue)
Comment #29
dieterholvoet commented@justanothermark I cherrypicked your suggestion from MR#305 / comment #23 onto MR#237. I suggest you close MR#305, no need to start a new MR, you can just acquire push access to the existing issue fork and commit there.
Comment #31
dieterholvoet commentedI also rebased MR#305 onto 8.x-3.x.
Comment #35
dieterholvoet commentedI rebased against 3.0-rc13.
Comment #36
anybody@saschaeggi wouldn't it make sense to unify the entity create / edit forms in general, providing the sidebar by default?
I'm not aware of the technical reasons why currently only nodes use the sidebar, but from the user / UX perspective the current state seems inconsistent.
Users wonder why the Create / edit Taxonomy Term, Product, Block, ... entity forms look different from the node forms they mostly work with (and like a lot! <3)
Another option would be to have a settings page where the entity types can be selected that should use the sidebar?
Should we maybe widen the scope of this issue or is there already an existing plan to unify the UX for all entity forms by default?
Thank you!
Comment #37
anybodyComment #38
anybodyComment #39
anna d commentedTested applicable MR https://git.drupalcode.org/project/gin/-/merge_requests/237
Published checkbox isn't part of gin-sticky-form-actions.