This currently overlaps with the D7 navbar module and I'm guessing has a similar integration issue with D8. It would be nice to have better placement somehow.

Screenshot of Masquerade Float Block overlapping navbar

Otherwise, this is a nice solution to work across public/admin themes.

CommentFileSizeAuthor
#8 masquerade-switcher-gripper-2671536-7.patch1.37 KBamanire
#6 masqueradeswitcher-gripper-2671536-6.patch1.37 KBamanire
Screen Shot 2016-02-18 at 6.25.28 PM.png18.25 KBamanire

Comments

amanire created an issue. See original summary.

amanire’s picture

amanire commented
Issue summary: View changes
dstorozhuk’s picture

dstorozhuk
Location Chicago 🇺🇸, Toronto 🇨🇦, Kyiv 🇺🇦
commented

It is very depend on screen resolution and naw bar type your are using (admin menu, core toolbar).
But the good news here is that you can drag the item where you think it is best for you for now and position will be remembered.

amanire’s picture

amanire commented

Thanks @dsorozhuk. That is good news! but that affordance has no signifier. I could write a patch that adds a grippy edge and shows the "move" cursor pointer.

BTW the text also wraps before "block" on my public theme. I would recommend changing the width to use em units so that it scales proportionally to the text.

dstorozhuk’s picture

dstorozhuk
Location Chicago 🇺🇸, Toronto 🇨🇦, Kyiv 🇺🇦
commented

amanire, please create a patch if you can for your recommendation if you want improve the module.

Thanks.

amanire’s picture

amanire commented
StatusFileSize
new masqueradeswitcher-gripper-2671536-6.patch1.37 KB

Cool, this patch does a few things:
1. it prepends a span to the switcher containing a couple of vertical ellipses which happen to look like a gripper.
2. It adjusts the spacing to use em units to adjust with the theme's font size
3. It matches the navbar module's font stack

Make of it what you will

amanire’s picture

amanire commented
Title: Overlap with navbars » Expose moveability of switcher and improve compatibility with theme fonts
amanire’s picture

amanire commented
StatusFileSize
new masquerade-switcher-gripper-2671536-7.patch1.37 KB

Small modification to the vertical ellipsis spacing.

amanire’s picture

amanire commented
Status: Active » Needs review
amanire’s picture

amanire commented

@dstorozhuk, did you have a chance to try out the patch? I think it improves the UX. Please let me know what you think.

dstorozhuk’s picture

dstorozhuk
Location Chicago 🇺🇸, Toronto 🇨🇦, Kyiv 🇺🇦
commented

@amanire, I am a bit busy right now. Hope will have have some time for module next week.
thanks for your patience.

  • dstorozhuk committed d888ea0 on 7.x-1.x authored by amanire 
    Issue #2671536 by amanire: Expose moveability of switcher and improve...
dstorozhuk’s picture

dstorozhuk
Location Chicago 🇺🇸, Toronto 🇨🇦, Kyiv 🇺🇦
commented

No time to wait! @amanire, your code in 7.x-dev!
Good, job, module more user friendly ).

dstorozhuk’s picture

dstorozhuk
Location Chicago 🇺🇸, Toronto 🇨🇦, Kyiv 🇺🇦
commented
Status: Needs review » Closed (fixed)