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

[navigator] some icons & names are not aligned #5580

Closed
fangnx opened this issue Jun 25, 2019 · 4 comments · Fixed by #6973
Closed

[navigator] some icons & names are not aligned #5580

fangnx opened this issue Jun 25, 2019 · 4 comments · Fixed by #6973
Labels
enhancement issues that are enhancements to current functionality - nice to haves navigator issues related to the navigator/explorer ui/ux issues related to user interface / user experience

Comments

@fangnx
Copy link

fangnx commented Jun 25, 2019

As the screenshot shows, icons and labels of package.json and README.md are not aligned properly.

image

@fangnx fangnx added enhancement issues that are enhancements to current functionality - nice to haves navigator issues related to the navigator/explorer ui/ux issues related to user interface / user experience labels Jun 25, 2019
@vince-fugnitto
Copy link
Member

I wonder if we should think about dropping our file-icon.js support for the vscode's icons.
https://marketplace.visualstudio.com/items?itemName=vscode-icons-team.vscode-icons

https://code.visualstudio.com/blogs/2016/09/08/icon-themes

It's not only our vertical spacing that is off but also our horizontal (notice the 'README' label is pushed more to the right than other files).

Screen Shot 2019-06-25 at 9 27 57 PM

@fangnx
Copy link
Author

fangnx commented Jun 26, 2019

I wonder if we should think about dropping our file-icon.js support for the vscode's icons.
https://marketplace.visualstudio.com/items?itemName=vscode-icons-team.vscode-icons

https://code.visualstudio.com/blogs/2016/09/08/icon-themes

It's not only our vertical spacing that is off but also our horizontal (notice the 'README' label is pushed more to the right than other files).

Screen Shot 2019-06-25 at 9 27 57 PM

Seems like the icons all have different width and length. Is it because we used icons from vs code but did not resize them?

@vince-fugnitto
Copy link
Member

I think in general it's how the icons are, they have their own height and width which may differ and we give them all consistent font-sizes, so in the end the icons are not properly normalized to each other.

@vince-fugnitto
Copy link
Member

I think this issue is already fixed.

Unfortunately it is still an issue :(
See how the filenames are not aligned (this happens due to the alignment of icons)

image

vince-fugnitto pushed a commit that referenced this issue Jan 28, 2020
Fixes #5580

- Adjust the spacing between file icons and file names in navigation tree so that they are aligned vertically and horizontally

Signed-off-by: Kaiyue Pan <[email protected]>
akosyakov pushed a commit to akosyakov/theia that referenced this issue Feb 24, 2020
Fixes eclipse-theia#5580

- Adjust the spacing between file icons and file names in navigation tree so that they are aligned vertically and horizontally

Signed-off-by: Kaiyue Pan <[email protected]>
JesterOrNot pushed a commit to JesterOrNot/theia that referenced this issue Mar 12, 2020
Fixes eclipse-theia#5580

- Adjust the spacing between file icons and file names in navigation tree so that they are aligned vertically and horizontally

Signed-off-by: Kaiyue Pan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement issues that are enhancements to current functionality - nice to haves navigator issues related to the navigator/explorer ui/ux issues related to user interface / user experience
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants