Problem/Motivation
Going into click-edit mode requires just one click, but returning to the normal mode with the Manage or Shortcut menu visible requires three clicks - on different corners of the screen window, and with unclear labels.
A user can enter the quick edit mode by clicking on the "Quick edit" contextual link on a block - without using or even being aware of the "Edit" link at the top left corner. They then save their changes at the bottom of settings tray at the right hand side. To leave the "Quick edit" mode they then need to click on the "Editing" link at the top left corner, and then they need to enable the manage menu again.
The step to click "editing" to get out of the quick edit mode seems to be unnecessary because the user did not contumaciously enter that mode. Even more so because when you click "Quick edit" to edit content, then you don't get into a different mode.
Also the label "Editing" is confusing because "editing" is not what the user wants to do. In general links in Drupal describe what the user will do when they click on that link or something that they will go to. I can't think of any other button that describes what the user is currently doing. Clearer would be "Edit/Stop editing" or "Quick-edit mode/Leave edit mode".
However, if a user clicks on a link in the systems tray (for example the Advanced options to edit more block configuration, or to edit a menu item) then saving that returns them to the normal mode.
If a user edits a menu link then they are either returned to the site (if they edit custom link) or brought to the configuration page of the menu (if they edit a menu item provide by code, such as the 'Home').
Proposed resolution
(1) If a user uses the quick edit mode by just clicking on a contextual link, then saving the changes, or closing the settings tray should return them to the normal mode.
(2) If one of the menus (Manage, Shortcuts, User) are enabled when a users clicks on a quick-edit link, then they need to be enabled again when the user leaves that mode.
(3) The "Editing" label should be replaced with a label that describes what will happen when the link is clicked to be consistent with other labels.
Comments
Comment #2
ifrikComment #3
tkoleary CreditAttribution: tkoleary at Acquia commentedVery good points.
Rather than add the overhead of maintaining a solution that attempts to guess at the users intent, why don't we just offer two submits?
This adds less code and also permits the user more explicit control.
Comment #4
ifrikThanks, that sounds like a good approach.
When writing up the issue, I wasn't quite sure what was intended behaviour or what might be unintended. And it's probably the worst issue title I ever came up with, so I would welcome a better title.
Comment #5
tkoleary CreditAttribution: tkoleary at Acquia commented@ifrik Thx!
Comment #6
Wim LeersComment #7
Wim LeersComment #9
tedbowChanging to new settings_tray.module component. @drpal thanks for script help! :)
Comment #20
mstrelan CreditAttribution: mstrelan at PreviousNext commentedIs this still relevant for core now that Quick Edit module is in contrib?
This sounds like it should be addressed in Quick Edit module.
As above
I can't find the "Editing" label in Drupal 10.1 with or without Quick Edit contrib, is this perhaps outdated?
Comment #22
smustgrave CreditAttribution: smustgrave at Mobomo commentedSince there hasn't been a follow up to #20 going to close out for now.
If still a valid bug please reopen addressing #20.
Thanks all!