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

Sync icon rotation is uneven (it is wobbling) #216511

Open
zeroskyx opened this issue Jun 18, 2024 · 9 comments · May be fixed by #216512
Open

Sync icon rotation is uneven (it is wobbling) #216511

zeroskyx opened this issue Jun 18, 2024 · 9 comments · May be fixed by #216512
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 icons-product Issues for in-product icons ux User experience issues

Comments

@zeroskyx
Copy link

Greetings,

currently, the sync icon in the bottom bar is not rotating evenly (it is actually wobbling).
The effect can be seen here:

Before.webm

This is caused since the used text icon is not centered properly. The effect has been reproduced on both Windows and Linux versions of VSCode.

  • VS Code Version: 1.90.1

Steps to Reproduce:

  1. Open a project with a remote git repository attached
  2. Click on the sync icon in the bottom bar
  3. Observe the wobbling icon
@bpasero
Copy link
Member

bpasero commented Jun 18, 2024

Yes, I also have seen this a lot of times!

@bpasero bpasero added bug Issue identified by VS Code Team member as probable bug icons-product Issues for in-product icons labels Jun 18, 2024
@bpasero bpasero added the ux User experience issues label Jan 7, 2025
@bpasero
Copy link
Member

bpasero commented Jan 7, 2025

fyi @joaomoreno

@bpasero bpasero added help wanted Issues identified as good community contribution opportunities good first issue Issues identified as good for first-time contributors labels Jan 27, 2025
@kkbrooks kkbrooks assigned kkbrooks and unassigned daviddossett Feb 6, 2025
@Ghady-Abboud
Copy link

First time contributor here, is it cool if I work on it?

@bpasero
Copy link
Member

bpasero commented Feb 16, 2025

Sure, anyone can. But I would think the fix should be in the icon itself, not some CSS rule.

@Ghady-Abboud
Copy link

Ghady-Abboud commented Feb 16, 2025

Ubuntu 22.0.4 on WSL2(Windows 10)
Vscode version 1.90.1

I can't reproduce the error, I see no wobbling

milkcrate-backend.TestResolver.-.Code.-.OSS.Dev.Ubuntu.2025-02-16.17-52-03.online-video-cutter.com.mp4

@roblourens
Copy link
Member

roblourens commented Mar 8, 2025

Was just looking at this and I think it's because the padding is not balanced. It has padding-left: 2px here https://github.com/microsoft/vscode/blob/main/src/vs/workbench/contrib/notebook/browser/media/notebookToolbar.css#L66 all other sides are 3px

But we shouldn't naively change that to 3 and change the spacing of all items. Maybe can make it spin inside the padding instead of spinning the padding. Or just change the padding for rotating elements. cc @rebornix

@bpasero
Copy link
Member

bpasero commented Mar 9, 2025

@roblourens but that would not explain why the icon looks off in the status bar, or does it? The CSS rule seems scoped to notebooks?

@Jeskalana

This comment has been minimized.

@roblourens
Copy link
Member

roblourens commented Mar 10, 2025

Sorry for the noise, I just noticed this in a notebook and was looking around for an issue and didn't even notice that we're talking about a totally different sync icon here...

Opened #243027

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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 icons-product Issues for in-product icons ux User experience issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants