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

Adding unique file type icons for different file types #2542

Closed
mhsh312 opened this issue Oct 29, 2023 · 12 comments · Fixed by #2550
Closed

Adding unique file type icons for different file types #2542

mhsh312 opened this issue Oct 29, 2023 · 12 comments · Fixed by #2550

Comments

@mhsh312
Copy link
Contributor

mhsh312 commented Oct 29, 2023

Increasing Access

Adding file type icons makes it quicker to spot files of a certain type. It would makes the UI more aesthetically pleasing.

Feature request details

Right now, the p5 web editor shows a generic file icon for all types of files.
image

I am proposing that we add different icons for different file types supported by the editor.

Example:
image

@welcome
Copy link

welcome bot commented Oct 29, 2023

Welcome! 👋 Thanks for opening your first issue here! And to ensure the community is able to respond to your issue, be sure to follow the issue template if you haven't already.

@mhsh312 mhsh312 changed the title Adding file type icons for html, css and js Adding unique file type icons for different file types Oct 29, 2023
@mhsh312
Copy link
Contributor Author

mhsh312 commented Oct 29, 2023

A good way to do this would be to create a React component and pass the file type as a prop to said component. That way it can work with future supported file types like typescript as well.
I would be down to work on this feature.

@lindapaiste
Copy link
Collaborator

I'm glad that you've opened this for discussion because this is definitely something that I would like to implement! We need to decide what type of icons we would like to use. You can check out a lot of options on the React Icons website.

Something like this BsFileType set is much more subtle than your example (from vscode?). Is that a good thing? Or do we prefer something that's very obvious?
image

@mhsh312
Copy link
Contributor Author

mhsh312 commented Oct 31, 2023

You're right. The VS Code icon pack was just for an example. I don't think it would suit the minimal UI of p5 editor.

And yes React Icons would be good but I think the BSfiletype icons are a little too subtle to the point here they are hard to distinguish. I think the Aioutline icons would look better.
You think I should make a PR draft?

@lindapaiste
Copy link
Collaborator

lindapaiste commented Oct 31, 2023

You're right. The VS Code icon pack was just for an example. I don't think it would suit the minimal UI of p5 editor.

And yes React Icons would be good but I think the BSfiletype icons are a little too subtle to the point here they are hard to distinguish. I think the Aioutline icons would look better. You think I should make a PR draft?

If you want to start working on the code that would be fine. We're probably going to spend some time debating icons, but that will have little impact on code itself. You can look at #2232 as an example of something similar.

@lindapaiste
Copy link
Collaborator

Here's another source for ideas (not sure of the license on these) https://jetbrains.design/intellij/resources/icons_list/

@mhsh312
Copy link
Contributor Author

mhsh312 commented Oct 31, 2023

On it.

@yashpandey06
Copy link
Contributor

@mhsh312 Are you working on this issue ??

@mhsh312
Copy link
Contributor Author

mhsh312 commented Nov 13, 2023

@mhsh312 Are you working on this issue ??

Already posted a PR that closes this issue. Just need further discussion with the maintainers about the icon types.

@yashpandey06
Copy link
Contributor

@mhsh312 have you posted the pr for this particular issue... i can't see one

@mhsh312
Copy link
Contributor Author

mhsh312 commented Nov 15, 2023

@mhsh312 have you posted the pr for this particular issue... i can't see one

image

Click on the blue text.

@yashpandey06
Copy link
Contributor

@mhsh312 got it ..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants