Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Give menus dropshadows #6657

Open
ninavizz opened this issue Jun 1, 2021 · 7 comments · May be fixed by QubesOS/qubes-desktop-linux-xfce4#25
Open

Give menus dropshadows #6657

ninavizz opened this issue Jun 1, 2021 · 7 comments · May be fixed by QubesOS/qubes-desktop-linux-xfce4#25
Assignees
Labels
C: gui-virtualization P: minor Priority: minor. The lowest priority, below "default." pr submitted A pull request has been submitted for this issue. T: enhancement Type: enhancement. A new feature that does not yet exist or improvement of existing functionality. ux User experience

Comments

@ninavizz
Copy link
Member

ninavizz commented Jun 1, 2021

The problem you're addressing (if any)
When I open a menu in an app, it typically has an outline—but no dropshadow. This presents a usability problem, with making it difficult to disambiguate what is "in front" vs what is "behind." For reference, I'm on Qubes 4.0.

Describe the solution you'd like
Just a dropshadow.
The mockup below has a .7 alpha #000000 10px dropshadow, w/o any x/y offsets.

Screen Shot 2021-05-31 at 7 47 31 PM

Where is the value to a user, and who might that user be?
Sighted users desiring clearer visual hierarchies and focus areas in their UI.

Related, non-duplicate issues
Maybe #6414, but nothing else I was able to see.

@ninavizz ninavizz added T: enhancement Type: enhancement. A new feature that does not yet exist or improvement of existing functionality. P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. ux User experience labels Jun 1, 2021
@ninavizz ninavizz added this to the TBD milestone Jun 1, 2021
@andrewdavidwong andrewdavidwong added C: gui-virtualization P: minor Priority: minor. The lowest priority, below "default." and removed P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. labels Jun 1, 2021
@GWeck
Copy link

GWeck commented Jun 2, 2021

This is just what MIcrosft has wrecked in theier design since Windows 8. Lack of a clear separation between fore- and backgroung makes these systems nearly unusable. Qubes is allready far better, and getting still beter with your efforts!

@p1xxxel
Copy link

p1xxxel commented Dec 30, 2021

@ninavizz Do you mean something like this to be turned on by default?
Screenshot_2021-12-30_15-47-58

@GWeck
Copy link

GWeck commented Dec 30, 2021

That's it - exactly!

@ninavizz
Copy link
Member Author

YES!! That, as well as the OS menus—but for folks working in apps in qubes, DEFINITELY THAT would be a hyooge "general clarity" usability help to keep brains from cramping! TY! <3

@p1xxxel
Copy link

p1xxxel commented Dec 31, 2021

actually this was very easy to do. I have done a pull request to enable it by default but this was an option in xfce.
To enable it in your version, click on the top left corner then go to System Tools --> Window Manager Tweaks --> Compositor and then make sure Show shadows under popup windows is checked.

If you want to do it through command line, open a dom0 terminal and then paste this :

xfconf-query -c xfwm4 -p /general/show_popup_shadow -s true

This uses xfconf-query tool to change the show_popup_shadow to true.

@andrewdavidwong andrewdavidwong added the pr submitted A pull request has been submitted for this issue. label Dec 31, 2021
@ninavizz
Copy link
Member Author

@DemiMarie Was there a reason this hadn't been done before? I seem to remember there being one and am curious, now that this PR has been submitted.

Thank you @p1xxxel!! I will go do that to my own QubesOS machine, now. :D

@DemiMarie
Copy link

@ninavizz I am honestly not certain. Personally I was not aware that this option existed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: gui-virtualization P: minor Priority: minor. The lowest priority, below "default." pr submitted A pull request has been submitted for this issue. T: enhancement Type: enhancement. A new feature that does not yet exist or improvement of existing functionality. ux User experience
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants