microsoft / vscode

Visual Studio Code
https://code.visualstudio.com
MIT License
163.44k stars 28.95k forks source link

Allow a touchbar that overflows #68214

Closed shpontex closed 5 years ago

shpontex commented 5 years ago
vscodebot[bot] commented 5 years ago

(Experimental duplicate detection) Thanks for submitting this issue. Please also check if it is already covered by an existing one, like:

bpasero commented 5 years ago

Likely caused by: https://github.com/electron/electron/issues/16473

bpasero commented 5 years ago

@shpontex I cannot reproduce what you describe with the following steps. Unfortunately I do not own a MacBookPro with native touchbar, so I have installed the touchbar simulator (https://sindresorhus.com/touch-bar-simulator/)

The change with Electron 3 that we had to work on was a change in how large icons would appear in the touchbar. They showed up much larger causing the touchbar to drop all the debug items at the end because space got little. It is possible that extensions have to update their icon sizes to account for that, however Nasc VSCode Touchbar seems to work just fine for me.

image

bpasero commented 5 years ago

One thing that might help is to disable "Show control strip" from:

image

bpasero commented 5 years ago

@isidorn @misolori you seem to have added a new entry to the debug touchbar (Start without Debug) which could also have an impact. By that you reduced the size of the touchbar by one item, and as such if an extension adds a lot of other items there is a chance that those items get removed.

I am hijacking this issue to have the real proper fix which is to implement a touchbar that simply scrolls left and right.

If someone wants to look into this, be my guest. Touchbar is created in https://github.com/Microsoft/vscode/blob/58fb98d/src/vs/code/electron-main/window.ts#L988

shpontex commented 5 years ago

@bpasero thank you for your explanation i confirm Nasc VSCode Touchbar work if i hide the control strip other plugins too i tried before to remove two buttons on the Touch Bar Display Commands extension "which contains only 5 buttons" but it didnt work

justinchan23 commented 5 years ago

@bpasero thank you for your explanation i confirm Nasc VSCode Touchbar work if i hide the control strip other plugins too i tried before to remove two buttons on the Touch Bar Display Commands extension "which contains only 5 buttons" but it didnt work

i submitted a pr to update the nasc vscode touchbar to work with vscode v1.31.0 with or without the control strip.

alistairholt commented 5 years ago

@isidorn @misolori you seem to have added a new entry to the debug touchbar (Start without Debug) which could also have an impact. By that you reduced the size of the touchbar by one item, and as such if an extension adds a lot of other items there is a chance that those items get removed.

I am hijacking this issue to have the real proper fix which is to implement a touchbar that simply scrolls left and right.

If someone wants to look into this, be my guest. Touchbar is created in https://github.com/Microsoft/vscode/blob/58fb98d/src/vs/code/electron-main/window.ts#L988

This is how it should function IMO. Being restricted to a fixe-width space is not a great long-term solution.

pelmers commented 5 years ago

I'd like to give this issue a shot by using the TouchBarScrubber (https://electronjs.org/docs/api/touch-bar-scrubber).

pelmers commented 5 years ago

I wasn't able to solve this with current Electron APIs, I filed an issue upstream at https://github.com/electron/electron/issues/16936.

albelin commented 5 years ago

Hi, I am a new user here looking to contribute to the repo. Is this task still available?

hasimyerlikaya commented 5 years ago

Hello to everyone. I was looking for a solution because I couldn't use the Touch Bar when debugging. I am developing a mobile application with Flutter and found a solution in my own way. I wanted to share with you the idea that it can be useful as well. I solved the problem by customizing the Touch Bar. I just added the tools I use most to the "Control Strip" field. In this way, the area there was reduced and the debug buttons were opened. I lost two of the Debug buttons, but I didn't use it much. Now I am very happy with this use. You can use it that way too.

Touch Bar Shot 2019-08-03 at 07 22 39

image

vscodebot[bot] commented 5 years ago

This issue is being closed to keep the number of issues in our inbox on a manageable level, we are closing issues that are not going to be addressed in the foreseeable future: We look at the number of votes the issue has received and the number of duplicate issues filed. More details here. If you disagree and feel that this issue is crucial: We are happy to listen and to reconsider.

If you wonder what we are up to, please see our roadmap and issue reporting guidelines.

Thanks for your understanding and happy coding!