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

Offset of top level tree items changes with icon them change #43084

Closed
sandy081 opened this issue Feb 7, 2018 · 9 comments
Closed

Offset of top level tree items changes with icon them change #43084

sandy081 opened this issue Feb 7, 2018 · 9 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug tree-views Extension tree view issues verified Verification succeeded
Milestone

Comments

@sandy081
Copy link
Member

sandy081 commented Feb 7, 2018

From here by @LazarusX

kapture 2018-02-07 at 10 10 20

@sandy081 sandy081 added bug Issue identified by VS Code Team member as probable bug tree-views Extension tree view issues layout General VS Code workbench layout issues labels Feb 7, 2018
@aeschli
Copy link
Contributor

aeschli commented Feb 7, 2018

I think that is the expected result. The whitespace you see is there for twisties.

@aeschli aeschli closed this as completed Feb 7, 2018
@LazarusX
Copy link

LazarusX commented Feb 7, 2018

@aeschli Why is the twisty behavior different when switching the file icon theme between Seti and Minimal? In addition, I think the file icon theme setting is not supposed to affect the appearance of extension tree views (see the screen cast in comment: #42097 (comment))

@aeschli
Copy link
Contributor

aeschli commented Feb 7, 2018

In general, the goal is to have all labels of the same hierarchy level aligned.
The VSCode Set theme does not have any folder icons. In that case, the file icons are shifted to the left, and align with the twisties of folders at the same level. Best is to try it out with files and folders at the same level.
@sandy081 is looking into the custom trees. which can have user-provided icons as well as file icons from the file icon theme.
IMO, again, the custom tree should make sure, that all labels are aligned.
To do so, the custom tree provider will have to provide placeholder icons for file and folders in case that the current file icon theme does not provide these.
The custom tree provider should also be able to indicate whether

  • the tree should always show icons for all items: Placeholders icons have to be used for file/folder/items that don't provide any
  • the tree should follow the style of the current file icon theme: No icons, icons only for leafs ('seti' style) or show icons for all items. Again placeholders icons are required for non-file/folder items.

@LazarusX
Copy link

LazarusX commented Feb 8, 2018

Your point regarding the icons in the file explorer makes sense to me.

But for custom trees, it seems that currently there is no way for extensions to respond to file icon theme style changes. In addition, naming it "explorer icon theme" rather than "file icon theme" might better reflect this setting's scope.

@sandy081
Copy link
Member Author

sandy081 commented Feb 9, 2018

Agreed that custom views should not be concerned about file icon themes. Reopening it

@sandy081 sandy081 reopened this Feb 9, 2018
@sandy081 sandy081 removed the layout General VS Code workbench layout issues label Feb 9, 2018
@sandy081 sandy081 added this to the February 2018 milestone Feb 9, 2018
@sandy081
Copy link
Member Author

sandy081 commented Feb 9, 2018

Fixed by ea78776

@sandy081 sandy081 closed this as completed Feb 9, 2018
@sandy081
Copy link
Member Author

sandy081 commented Feb 9, 2018

kapture 2018-02-09 at 18 52 12

@sandy081
Copy link
Member Author

sandy081 commented Mar 1, 2018

Reopening it because #28974 (ea78776) is reverted

@sandy081 sandy081 modified the milestones: February 2018, March 2018 Mar 1, 2018
@sandy081 sandy081 reopened this Mar 13, 2018
@sandy081
Copy link
Member Author

Fixed by 11a262e

@roblourens roblourens added the verified Verification succeeded label Mar 30, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators Apr 27, 2018
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 tree-views Extension tree view issues verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

4 participants