- Issue created by @SirClickALot
- ๐ฌ๐งUnited Kingdom SirClickALot Somerset
I have just raised this to major which I think is fair given that the issue blocks all site editing using the Front End Editing (FEE) module and that might cause some anxiety to content editors who rely on the FEE module and who are not in a position to quickly roll back to an earlier gin theme version.
- First commit to issue fork.
- Merge request !453Issue #3417719: Refine User Experience: Addressing Size, Position, and Hover... โ (Merged) created by saschaeggi
- Status changed to Needs review
5 months ago 10:41am 9 July 2024 - ๐ฌ๐งUnited Kingdom SirClickALot Somerset
I can confirm that after applying the MR via the patch file, we can now use the Front End Editing module again โ the Save button is back ;-)
Super fast work there, thank you!
- Status changed to RTBC
4 months ago 10:47am 12 July 2024 - ๐ฎ๐ณIndia Tirupati_Singh
Hi @saschaeggi, I've applied the MR as patch and the patch applied successfully. The issue of
Save
button missing while usingfrontend_editing
module has been resolved successfully through patch. Now using frontend_editing module the quick edit done can be saved successfully through the save button. Attaching before and after fixes screenshots for reference. Hence moving the issue status to Reviewed & tested. - Status changed to Fixed
4 months ago 8:46pm 12 July 2024 - Status changed to Needs work
4 months ago 3:15pm 14 July 2024 - ๐ฌ๐งUnited Kingdom SirClickALot Somerset
I have just realised that although this is fixed for the EDIT form, the Save button is still missing when you try to insert a brand new Paragraph - i.e. I' don't think the ADD form has been taken into account too.
Thanks all.
- ๐จ๐ญSwitzerland saschaeggi Zurich
@SirClickalot can you please open an MR with the additional forms we need to check for? Thanks
- ๐ฌ๐งUnited Kingdom SirClickALot Somerset
@saschaeggi I'm afraid I do not know how to open an MR - sorry!
However, the form id appears to be paragraph_text_entity_add_form and looking into the last patch from MR: 454, I can see the addition ofโฆ
strpos($form_id, '_entity_add_form') !== FALSE ||
โฆwould do it.
- ๐ฎ๐ณIndia nayana_mvr
Hi @SirClickalot, I have been trying to reproduce the issue which is mentioned in #11 ๐ Save button missing in modal for editing using the Front End Editing module Needs work but I couldn't find an option to add a new paragraph to a page using Frontend Editing module. Could you please specify the steps to reproduce?
Note:- I'm able to add new paragraph to a new/existing page and save it using normal editing. - ๐ฌ๐งUnited Kingdom SirClickALot Somerset
Sure,
It's a fundamental whizzy feature of the Front End Editing module...
What you will find โ when using Gin as the backend theme โ is that having chosen one of the available Paragraphs types, you can happily edit away but cannot Save it.
I hope that helps.
- ๐ฎ๐ณIndia nayana_mvr
Due to some reason, I'm not getting that '+' icon to add new paragraph. That's why I'm not able to reproduce the issue. I tried by re-installing the module multiple times but still facing this issue.
- ๐ฌ๐งUnited Kingdom SirClickALot Somerset
That could possibly be because you have not enable the particular Paragraph type at:
/admin/config/frontend-editing/entity-bundle-restrictions
- ๐ฎ๐ณIndia nayana_mvr
It's already enabled. Because then only the edit button will be available in the frontend for the paragraphs.
- Assigned to Tirupati_Singh
- Issue was unassigned.
- Status changed to Needs review
4 months ago 12:35pm 16 July 2024 - ๐ฎ๐ณIndia Tirupati_Singh
Hi, I've added the Save button for adding a new paragraph using frontend_editing module. Attaching the before and after video clips for reference. Please review the MRs.
- ๐ฌ๐งUnited Kingdom SirClickALot Somerset
I can confirm that MR:341817 fixes the issue for me.
Thank you @tirupati_singh โ
- ๐จ๐ญSwitzerland saschaeggi Zurich
The MR !473 seems to contain untelated changes to
dist/js/more_actions.js
Also can you check if the MR in https://www.drupal.org/project/gin/issues/3463796 ๐ Ensure sticky action buttons to work with modals and ajax refresh-calls Needs review will solve this issue?
- ๐ฎ๐ณIndia Tirupati_Singh
@saschaeggi, I've removed the unrelated changes to
dist/js/more_actions.js
in MR !473. Please review the changes.I've applied the MR !479 as a patch and it applied with no errors. After applying the patch, the MR didn't resolve the issue.The Save button missing issue persists when
Enable sticky action buttons
is enabled. -
saschaeggi โ
committed 2c6bfc0e on 8.x-3.x authored by
Tirupati_Singh โ
Resolve #3460111 "Save button missing 3x"
-
saschaeggi โ
committed 2c6bfc0e on 8.x-3.x authored by
Tirupati_Singh โ
-
saschaeggi โ
committed 12463593 on 4.0.x authored by
Tirupati_Singh โ
Resolve #3460111 "Save button missing"
-
saschaeggi โ
committed 12463593 on 4.0.x authored by
Tirupati_Singh โ
- Status changed to Fixed
4 months ago 11:44am 1 August 2024 - ๐จ๐ญSwitzerland saschaeggi Zurich
Thanks for testing, pushed the fix ๐
Automatically closed - issue fixed for 2 weeks with no activity.