zed-industries / zed

Code at the speed of thought – Zed is a high-performance, multiplayer code editor from the creators of Atom and Tree-sitter.
https://zed.dev
Other
47.09k stars 2.71k forks source link

Toggled off icons indent #16073

Open agorushkin opened 1 month ago

agorushkin commented 1 month ago

Check for existing issues

Describe the feature

When disabling icons for files through project_panel.file_icons: false, the spaces remaining from the indent is MASSIVE. It would be nice if it worked like applying a display: none property in CSS.

If applicable, add mockups / screenshots to help present your vision of the feature

image image
notpeter commented 1 month ago

Yeah, that's definitely a little ugly. (I was able reproduce on MacOS and Linux)

Screenshot 2024-08-12 at 11 32 51
uladkaminski commented 1 month ago

I think here the goal is to have the same intend with the folders that anyway have icons. Otherwise it is hard to read the structure of the folders.

image
uladkaminski commented 1 month ago

I think it worth to make just a small point as a default icon for the all types of the files to make the layout more user friendly.

Here is the PR for that: https://github.com/zed-industries/zed/pull/16142

image
SomeoneToIgnore commented 1 month ago

image

Looks better now.

SomeoneToIgnore commented 1 month ago

@maxdeviant , could we get some ideas on how that should be implemented?

I do not think the originally proposed, placeholder icon is a good approach. No extra space seems to be bad too due to https://github.com/zed-industries/zed/pull/16167 , so what is a good one?

maxdeviant commented 1 month ago

@maxdeviant , could we get some ideas on how that should be implemented.

I do not think the originally proposed, placeholder icon is a good approach. No extra space seems to be bad too due to #16167 , so what is a good one?

@iamnbutler and I discussed just hiding the folder icon entirely when file icons are disabled, but we're not sure exactly what we want to do here yet.

uladkaminski commented 1 month ago

Happy to help here, just let me know what is the decision.

JosephTLyons commented 1 month ago

This is now available in today's preview build.