- Issue created by @Harlor
- @harlor opened merge request.
- π©πͺGermany Christian.wiedemann
Hi, this is intentional. Maybe we make this configurarble.
- Status changed to Needs review
almost 2 years ago 2:54pm 24 January 2023 - π©πͺGermany Harlor Berlin
Ah ok I tried to introduce such a setting.
Btw. is there a particular reason why the gin_lb settings are not part of the theme settings?
-
Christian.wiedemann β
committed b1358ac4 on 1.0.x authored by
Harlor β
Issue #3336094: edit-discard-changes and edit-revert buttons hidden
-
Christian.wiedemann β
committed b1358ac4 on 1.0.x authored by
Harlor β
- π©πͺGermany Christian.wiedemann
Thanks! And yes the config should be moved to the gin settings.
- Status changed to Fixed
almost 2 years ago 1:12pm 3 February 2023 Automatically closed - issue fixed for 2 weeks with no activity.
- πΊπΈUnited States jessehs Taos, New Mexico
There's a typo here that breaks the feature. The setting is hide_discard_revert_button where the .module file checks for hide_discard_revert_buttons (plural).
https://git.drupalcode.org/project/gin_lb/-/blob/1.0.x/gin_lb.module#L347
compare that with
https://git.drupalcode.org/project/gin_lb/-/blob/1.0.x/src/Form/Settings...
It's also missing a line here: https://git.drupalcode.org/project/gin_lb/-/blob/1.0.x/src/Form/Settings...
->set('hide_discard_revert_buttons', $form_state->getValue('hide_discard_revert_buttons'))
- Status changed to Fixed
over 1 year ago 10:08am 2 May 2023 - π©πͺGermany design.er
I can confirm that the setting
Hide "Discard changes" and "Revert to defaults" button
on the page/admin/config/gin_lb/settings
doesn't work at the moment.
It's supposed to be enabled by default, if I understand correctly. It's always disabled, though.If I enable the setting and save the form, this setting appears as disabled again. Even if it's disabled, the buttons do not appear in the Layout Builder.
The
Discard changes
button and theRevert to defaults
button do entirely different things and should be treated independently. Therefore, there should be a separate setting for each button.The possibility to discard changes that may have gone wrong is crucial for every editor.
The possibility to reset the entire page back to defaults, on the other hand, is super dangerous and I always hide it to prevent accidents. - π©πͺGermany design.er
Oops, I've seen the referenced issue that tries to solve the problem just now. π