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

Shortcut background broken with centered editor layout #59952

Closed
joaomoreno opened this issue Oct 4, 2018 · 13 comments · Fixed by #170403 or GulajavaMinistudio/vscode-1#471
Closed

Shortcut background broken with centered editor layout #59952

joaomoreno opened this issue Oct 4, 2018 · 13 comments · Fixed by #170403 or GulajavaMinistudio/vscode-1#471
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug good first issue Issues identified as good for first-time contributors help wanted Issues identified as good community contribution opportunities insiders-released Patch has been released in VS Code Insiders verified Verification succeeded workbench-zen Full screen 'Zen' view issues
Milestone

Comments

@joaomoreno
Copy link
Member

joaomoreno commented Oct 4, 2018

Steps:

  1. Open new Window
  2. Toggle Centered Editor layout
  3. Resize the editor while holding Alt, so the other sash doesn't resize

screenshot 2018-10-04 at 09 24 08

@isidorn
Copy link
Contributor

isidorn commented Oct 4, 2018

The issue here is that the watermark-box element is not belonging to the editor-area split view, thus it does not get centered.
IMHO it should be creataed inside the editor area split view just like the background image is.
forwarding to @chrmarti

@isidorn isidorn assigned chrmarti and unassigned isidorn Oct 4, 2018
@chrmarti chrmarti added bug Issue identified by VS Code Team member as probable bug workbench-welcome Welcome page issues labels Oct 4, 2018
@joaomoreno joaomoreno added good first issue Issues identified as good for first-time contributors help wanted Issues identified as good community contribution opportunities labels Apr 22, 2021
@joaomoreno joaomoreno added this to the Backlog milestone Apr 22, 2021
@joaomoreno
Copy link
Member Author

Just came here to file this issue once again. 😆

@isidorn
Copy link
Contributor

isidorn commented Apr 23, 2021

Would be cool if we fix this. Workaround:

	"workbench.tips.enabled": false

@timfenney
Copy link

What are the steps to reproduce? I opened a split view editor, and even on a 4k monitor with low scaling couldn't repro it.

@joaomoreno
Copy link
Member Author

joaomoreno commented Jun 14, 2021

Added steps to first comment. 👌

@JacksonKearl JacksonKearl added workbench-zen Full screen 'Zen' view issues and removed workbench-welcome Welcome page issues labels Oct 22, 2021
@LSViana
Copy link

LSViana commented May 9, 2022

@joaomoreno I'm trying to start contributing and I noticed this could be a possible first issue to solve. After a few hours, I did solve the issue and the solution is working consistently, but I'm pretty sure it's not optimal (and it doesn't have tests 😅).

Is it okay if I submit a PR for this to get some feedback and try polishing it?

@JacksonKearl
Copy link
Contributor

@LSViana yes feel free to submit a PR and we can take it from there

@LSViana
Copy link

LSViana commented May 10, 2022

@JacksonKearl Thanks for your prompt response!

I just submitted #149132 and I need help because I have little (to no) knowledge about the VS Code codebase.

@lramos15 lramos15 assigned sbatten and unassigned JacksonKearl May 25, 2022
@p4nu
Copy link

p4nu commented Jun 5, 2022

If it's okay to @LSViana I could take a look at this

@p4nu
Copy link

p4nu commented Jun 6, 2022

Do we want a css change or should we move the creation of watermark to editor area split view?

I have found the place where the background is created but I need to figure out how to move the creation of watermark there. Any examples in the codebase would be appreciated!

@joaomoreno
Copy link
Member Author

Simple CSS should do here.

@p4nu
Copy link

p4nu commented Jun 8, 2022

The position and width elements are not going to be connected between watermark-box and editor-group-letterpress. I'm not sure how the css can be implemented here because they are in different places in the dom hierarchy. The watermark-box is not getting any information from separator-borders.

@p4nu
Copy link

p4nu commented Jun 8, 2022

image

image

@joaomoreno joaomoreno modified the milestones: Backlog, January 2023 Jan 2, 2023
@joaomoreno joaomoreno assigned joaomoreno and unassigned sbatten Jan 2, 2023
joaomoreno added a commit that referenced this issue Jan 2, 2023
* move watermark shortcuts closer to letterpress

fixes #59952

* fix layout in firefox
@VSCodeTriageBot VSCodeTriageBot added the unreleased Patch has not yet been released in VS Code Insiders label Jan 2, 2023
@VSCodeTriageBot VSCodeTriageBot added insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels Jan 3, 2023
@bpasero bpasero added the verified Verification succeeded label Jan 26, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Feb 16, 2023
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 good first issue Issues identified as good for first-time contributors help wanted Issues identified as good community contribution opportunities insiders-released Patch has been released in VS Code Insiders verified Verification succeeded workbench-zen Full screen 'Zen' view issues
Projects
None yet
10 participants