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

Bottom left "action" (gear icon) menu not scrolling (MacOS) #40262

Closed
atrn opened this issue Dec 15, 2017 · 16 comments · Fixed by #154092
Closed

Bottom left "action" (gear icon) menu not scrolling (MacOS) #40262

atrn opened this issue Dec 15, 2017 · 16 comments · Fixed by #154092
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug electron Issues and items related to Electron menus Menu items and widget issues papercut 🩸 A particularly annoying issue impacting someone on the team upstream Issue identified as 'upstream' component related (exists outside of VS Code) upstream-issue-linked This is an upstream issue that has been reported upstream verified Verification succeeded
Milestone

Comments

@atrn
Copy link

atrn commented Dec 15, 2017

  • VSCode Version: Code 1.19.0 (816be67, 2017-12-14T09:51:12.123Z)
  • OS Version: Darwin x64 16.7.0
  • Extensions:
Extension Author (truncated) Version
vscode-markdownlint Dav 0.12.0
project-manager ale 0.23.1
alignment ann 0.3.0
code-gnu-global aus 0.2.2
togglehs bbe 0.1.2
gitignore cod 0.5.0
githistory don 0.2.3
git-project-manager fel 1.4.0
auto-close-tag for 0.5.5
Go luk 0.6.69
vscode-clang mit 0.2.2
cpptools ms- 0.14.4
debugger-for-chrome msj 4.0.0
vscode-emacs-improved rkw 0.1.1
cmake twx 0.0.17
vscode-lldb vad 0.7.1
cmake-tools vec 0.10.3
gitblame wad 2.2.0
clang-format xav 1.6.1

(7 theme extensions excluded)


Steps to Reproduce:

  1. Resize VS Code window so it occupies most but not all, of the display.
  2. Click on bottom left "action" (gear icon) menu because it has
    an indicator on it (I'd just updated and it was the restart button).
  3. Menu draws as a drop down but is clipped due to lack of space. See arrow at bottom of menu indicating it can scroll.
  4. Attempt to scroll menu clicking on arrow and usual MacOS pointer motion. Menu doesn't scroll.
  5. Resize window to give more room below (effectively moving the gear icon up a little). Click gear icon and see full menu.

The menu display changes location at different window sizes, which really means available space for the menu's height. With a slightly larger window size the menu renders itself above and to the right of the gear icon. Reducing the window height a little has it render as a drop-down (below) but with insufficient height to display the full menu and adds the non-functioning scroll arrow.

Reproduces without extensions: Yes

@bpasero bpasero added electron Issues and items related to Electron upstream Issue identified as 'upstream' component related (exists outside of VS Code) and removed workbench labels Dec 15, 2017
@ricobbe
Copy link

ricobbe commented Dec 25, 2017

After the most recent VSCode update, the gear-icon menu mentioned in the original submission appears to be fixed; it now displays on-screen in a position that allows the entire menu to display without scrolling.

However, I can still reproduce the original problem by right-clicking on an identifier in a line of an editor window that's close to the bottom of the screen. See attached screenshot for an example; the bottom of the screenshot is the bottom of the screen. As in the original report, neither clicking on nor hovering over the downward arrow in the popup menu has any effect.

image

@bpasero
Copy link
Member

bpasero commented May 26, 2019

This bugs me a lot too, @deepak1556 any clues?

@sbatten sbatten added menus Menu items and widget issues and removed workbench-menu labels Oct 7, 2019
@bpasero bpasero assigned deepak1556 and unassigned bpasero Oct 8, 2019
@bpasero bpasero added the bug Issue identified by VS Code Team member as probable bug label Oct 8, 2019
@bpasero
Copy link
Member

bpasero commented Oct 8, 2019

This is annoying. @deepak1556 we should address this somehow.

@bpasero
Copy link
Member

bpasero commented Dec 17, 2019

This is reported as electron/electron#14261

@bpasero bpasero added the upstream-issue-linked This is an upstream issue that has been reported upstream label Dec 17, 2019
@akaroml
Copy link
Member

akaroml commented Mar 4, 2020

Here's a related case in the bottom left corner. When there is enough space (full screen available) to show the full menu, vscode only uses a smaller portion and shows part of the menu items. I need to scroll to see all the items all the time. Check out this screen recording:

vscode scroll to show full menu

@bpasero
Copy link
Member

bpasero commented Oct 30, 2020

@deepak1556 @sbatten latest insiders from today seems to have fixed the issue for me! did we also move the location of the menu in native? I know we changed it for the custom menu.

recording

@sbatten
Copy link
Member

sbatten commented Oct 30, 2020

well i did change the native one to use the mouse location instead, but I'm not understanding the position in your screen shot.

@sbatten
Copy link
Member

sbatten commented Oct 30, 2020

oh I know, what's happening (I used native for titlebar setting but forgot macos titlebar custom => native menus) so its using the custom menu logic. I mean I can keep it that way if it looks good?

@bpasero
Copy link
Member

bpasero commented Oct 31, 2020

@sbatten since it is not a right-click menu I think the position now is actually better than before and if it fixes the scrolling issue on macOS, even better.

sbatten added a commit that referenced this issue Nov 2, 2020
@sbatten
Copy link
Member

sbatten commented Nov 2, 2020

since the fix was caused by a bug in the logic, i updated the logic to make it intentional. will close this now

@sbatten
Copy link
Member

sbatten commented Nov 12, 2020

I updated how context view lays making alignment a lot better for custom menus. Unfortunately we don't have that level of control for native menus. The "bug" that fixed this has been removed since it is not a reliable fix and it resulted in some strange cases as well.

@deepak1556
Copy link
Contributor

Is this still an issue ? I am unable to repro with latest insiders.

@atrn
Copy link
Author

atrn commented Mar 12, 2021

Seems to have been fixed for a while. Works fine for me these days.

@bpasero bpasero added the papercut 🩸 A particularly annoying issue impacting someone on the team label Mar 12, 2021
@github-actions github-actions bot locked and limited conversation to collaborators May 13, 2021
@rzhao271 rzhao271 modified the milestones: On Deck, May 2021 Jun 4, 2021
@meganrogge meganrogge added the verified Verification succeeded label Jun 7, 2021
@bpasero
Copy link
Member

bpasero commented Jul 30, 2022

This is happening for a long time still:

image

Refs: electron/electron#13279

@bpasero bpasero reopened this Jul 30, 2022
@bpasero bpasero removed this from the May 2021 milestone Jul 30, 2022
@bpasero bpasero removed the verified Verification succeeded label Jul 30, 2022
@VSCodeTriageBot VSCodeTriageBot removed the insiders-released Patch has been released in VS Code Insiders label Jul 30, 2022
@bpasero bpasero added this to the August 2022 milestone Aug 3, 2022
@bpasero
Copy link
Member

bpasero commented Aug 3, 2022

Looks like a fix is in the works: electron/electron#35194

@bpasero
Copy link
Member

bpasero commented Aug 4, 2022

Nice 🎸

@lramos15 lramos15 added the verified Verification succeeded label Aug 24, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug electron Issues and items related to Electron menus Menu items and widget issues papercut 🩸 A particularly annoying issue impacting someone on the team upstream Issue identified as 'upstream' component related (exists outside of VS Code) upstream-issue-linked This is an upstream issue that has been reported upstream verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

11 participants