That's a quite common case that theme layout has not right place for Masquerade block, especially at development time. Therefore default admin theme (Seven) is built without sidebars and the block is forced to be in main content region. I propose attaching a simple floating button for masquerading / unmasquerading.

CommentFileSizeAuthor
#6 2448271-6.patch547 bytesandypost
#6 masquerade_menu.png5.74 KBandypost
#5 masquerade-modal.gif67.63 KBchi
#3 masquerade-ui_improvement-2448271-2.patch3.79 KBchi
#1 masquerade-ui_improvement-2448271-0.patch3.77 KBchi

Comments

chi’s picture

chi commented
Status: Active » Needs review
StatusFileSize
new masquerade-ui_improvement-2448271-0.patch3.77 KB

Status: Needs review » Needs work

The last submitted patch, 1: masquerade-ui_improvement-2448271-0.patch, failed testing.

chi’s picture

chi commented
Status: Needs work » Needs review
StatusFileSize
new masquerade-ui_improvement-2448271-2.patch3.79 KB

Cleanup

andypost’s picture

andypost
he/him
Primary language Russian
commented
Issue tags: +Needs screenshots

Thanx for idea, technical implementation suppose should be a bit different
This approach solves at last current regression of absence unmasquerade link (seems menu changes)

Also I thought about svg icon for right area of toolbar but toolbar is not enabled for authenticated users in standard profile so we need default menu link in any standard menus.
Having a dialog form to masquerade require to add setting for redirect or front route because modal have no referer to redirect

chi’s picture

chi commented
Issue tags: -Needs screenshots
StatusFileSize
new masquerade-modal.gif67.63 KB

so we need default menu link in any standard menus

Many sites do not use standard menus.

andypost’s picture

andypost
he/him
Primary language Russian
commented
StatusFileSize
new masquerade_menu.png5.74 KB
new 2448271-6.patch547 bytes

Take a loot at comment and user module *.links.menu.yml files they provide links for standard profile. I just can't get why access for them is not checked right now

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Related issues: +#2463659: Regression test coverage: integration test for an uncacheable menu link that depends on session data, +#1805054: Cache localized, access filtered, URL resolved, and rendered menu trees

So, #6 is answered by #2463659: Regression test coverage: integration test for an uncacheable menu link that depends on session data, and is fixed by #1805054: Cache localized, access filtered, URL resolved, and rendered menu trees. Access now is correctly checked, even when menu blocks are render cached.

andypost’s picture

andypost
he/him
Primary language Russian
commented

It makes sense to use the same approach as \Drupal\user\Plugin\Menu\LoginLogoutMenuLink does in core now

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented

That makes sense :)

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Needs review » Postponed
Parent issue: » #2695779: Unmasquerade link appears for users with the permission: 'link to any page', regardless of masquerade status.

Postponed on #2695779: Unmasquerade link appears for users with the permission: 'link to any page', regardless of masquerade status.

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Postponed » Active
Parent issue: #2695779: Unmasquerade link appears for users with the permission: 'link to any page', regardless of masquerade status. » #2982514: Masquerade 8.x-2.1 stable release plan