Navigation Menu

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

Right Mouse Click is broken #100982

Closed
jtsom opened this issue Jun 24, 2020 · 5 comments
Closed

Right Mouse Click is broken #100982

jtsom opened this issue Jun 24, 2020 · 5 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders verified Verification succeeded
Milestone

Comments

@jtsom
Copy link

jtsom commented Jun 24, 2020

Issue Type: Bug

Right click anywhere in a document and the context menu does not appear. An error is reported in the developer tools console.

image

VS Code version: Code - Exploration 1.47.0-exploration (23d15c6, 2020-06-24T17:10:42.684Z)
OS version: Darwin x64 19.5.0

System Info
Item Value
CPUs Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz (12 x 2600)
GPU Status 2d_canvas: enabled
flash_3d: enabled
flash_stage3d: enabled
flash_stage3d_baseline: enabled
gpu_compositing: enabled
metal: disabled_off
multiple_raster_threads: enabled_on
oop_rasterization: enabled
opengl: enabled_on
protected_video_decode: unavailable_off
rasterization: enabled
skia_renderer: disabled_off_ok
video_decode: enabled
webgl: enabled
webgl2: enabled
Load (avg) 2, 2, 2
Memory (System) 32.00GB (5.23GB free)
Process Argv
Screen Reader no
VM 13%
Extensions: none
@jtsom
Copy link
Author

jtsom commented Jun 24, 2020

Also, once the error is triggered, hover tool tips don't show anymore.

@deepak1556
Copy link
Contributor

Thanks for trying out the exploration build

@bpasero this another one of the breaking change from Electron 9 https://github.com/electron/electron/blob/master/docs/breaking-changes.md#behavior-changed-sending-non-js-objects-over-ipc-now-throws-an-exception , ipc serialization will behave like postmessage and it throws if there is an error.

I am yet to make a tracking issue to cover this change.

@deepak1556 deepak1556 added bug Issue identified by VS Code Team member as probable bug electron-9-update labels Jun 25, 2020
@bpasero
Copy link
Member

bpasero commented Jun 25, 2020

@deepak1556 I do not fully understand, the contents we send over are JSON serializable, are you saying we have to send strings now?

The object is of this shape:

export interface ICommonContextMenuItem {
label?: string;
type?: 'normal' | 'separator' | 'submenu' | 'checkbox' | 'radio';
accelerator?: string;
enabled?: boolean;
visible?: boolean;
checked?: boolean;
}
export interface ISerializableContextMenuItem extends ICommonContextMenuItem {
id: number;
submenu?: ISerializableContextMenuItem[];
}

@bpasero
Copy link
Member

bpasero commented Jun 30, 2020

@deepak1556 ping

@bpasero bpasero added this to the July 2020 milestone Jul 1, 2020
@bpasero
Copy link
Member

bpasero commented Jul 1, 2020

Oh I see it now, there is a function sneaking in via options...

@bpasero bpasero modified the milestones: July 2020, June 2020 Jul 1, 2020
@bpasero bpasero closed this as completed in 1f31957 Jul 1, 2020
@lramos15 lramos15 added the verified Verification succeeded label Jul 2, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Aug 15, 2020
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 insiders-released Patch has been released in VS Code Insiders verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

5 participants
@jtsom @bpasero @deepak1556 @lramos15 and others