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

UI BUG in activity bar #193304

Closed
whoizNiKHiL opened this issue Sep 17, 2023 · 9 comments
Closed

UI BUG in activity bar #193304

whoizNiKHiL opened this issue Sep 17, 2023 · 9 comments
Assignees
Labels
under-discussion Issue is under discussion for relevance, priority, approach workbench-auxsidebar Issues for secondary sidebar

Comments

@whoizNiKHiL
Copy link

whoizNiKHiL commented Sep 17, 2023

Type: Bug

When primary bar is set to right and we click on bottom options (account / settings), the popup/ modal closes itself by choosing one option. Overall the two options are hard to operate when on right side.

VS Code version: Code 1.82.2 (abd2f3d, 2023-09-14T05:51:20.981Z)
OS version: Linux x64 5.15.0-83-generic
Modes:

System Info
Item Value
CPUs 11th Gen Intel(R) Core(TM) i7-1165G7 @ 2.80GHz (8 x 2800)
GPU Status 2d_canvas: enabled
canvas_oop_rasterization: disabled_off
direct_rendering_display_compositor: disabled_off_ok
gpu_compositing: enabled
multiple_raster_threads: enabled_on
opengl: enabled_on
rasterization: enabled
raw_draw: disabled_off_ok
video_decode: enabled
video_encode: disabled_software
vulkan: disabled_off
webgl: enabled
webgl2: enabled
webgpu: disabled_off
Load (avg) 2, 2, 1
Memory (System) 15.18GB (9.04GB free)
Process Argv --unity-launch --crash-reporter-id 0bc5accc-6ab2-40f9-a02c-999162a42d2e
Screen Reader no
VM 0%
DESKTOP_SESSION ubuntu
XDG_CURRENT_DESKTOP Unity
XDG_SESSION_DESKTOP ubuntu
XDG_SESSION_TYPE x11
A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
vscorecescf:30445987
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
pythonvs932:30410667
vsclangdc:30486549
c4g48928:30535728
dsvsc012cf:30540253
pynewext54:30695312
azure-dev_surveyone:30548225
vscccc:30803845
2e4cg342:30602488
89544117:30613380
vscrpc:30673769
showlangstatbar:30737416
0bi6i642:30835152
03d35959:30757346
pythonfmttext:30731395
fixshowwlkth:30771522
showindicator:30805244
pythongtdpath:30769146
i26e3531:30792625
pythonnosmt12:30797651
pythonidxpt:30805730
pythonnoceb:30805159
dsvsc013:30795093
dsvsc014:30804076
diffeditorv2:30821572
dsvsc015:30829745

@whoizNiKHiL
Copy link
Author

It works fine on left as there is some margin. But on right there is no margin of popup / modal. So it just overlaps on your click.

@sbatten
Copy link
Member

sbatten commented Sep 19, 2023

@deepak1556 right now, native anchored menus (dropdowns or button-triggered menus) have this issue because electron will not compensate for mouse location. @bpasero's fix for context menus won't apply here because we pass an anchor over. should we consider if native menus are enabled for linux/windows that we ignore the anchor and use mouse position instead.

alternatively, we could ditch native menus for the context menu service on non-macos desktop platforms regardless of titlebar style

@deepak1556
Copy link
Collaborator

native anchored menus (dropdowns or button-triggered menus) have this issue because electron will not compensate for mouse location

Is there an upstream issue for this I can look at ?

@sbatten
Copy link
Member

sbatten commented Sep 27, 2023

It's essentially this one #113175

@deepak1556
Copy link
Collaborator

Ah okay, I don't have way to move that issue forward since I lack repro on my setup. If I am not mistaken, @bpasero also made the change to native context menus to use mouse position #113175 (comment) and given the issue persists, I like the alternative to make custom context menus as default regardless of titlebar style.

@sbatten
Copy link
Member

sbatten commented Sep 28, 2023

I assume this means for Windows, Linux, and Web

@deepak1556
Copy link
Collaborator

I was 👍 for Linux, is there a strong reason to make the change on Windows as well ?

@sbatten
Copy link
Member

sbatten commented Sep 29, 2023

I think the same applies for those that use the native titlebar on WIndows

@deepak1556 deepak1556 removed their assignment Dec 11, 2023
@rzhao271 rzhao271 added under-discussion Issue is under discussion for relevance, priority, approach workbench-auxsidebar Issues for secondary sidebar labels Dec 13, 2024
@rzhao271
Copy link
Contributor

Closing as duplicate of #167941

@rzhao271 rzhao271 closed this as not planned Won't fix, can't repro, duplicate, stale Dec 13, 2024
@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators Jan 29, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
under-discussion Issue is under discussion for relevance, priority, approach workbench-auxsidebar Issues for secondary sidebar
Projects
None yet
Development

No branches or pull requests

6 participants