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

Icon and Icon.sys Improvements #35

Open
wants to merge 21 commits into
base: master
Choose a base branch
from

Conversation

NathanNeurotic
Copy link

@NathanNeurotic NathanNeurotic commented Aug 24, 2024

This PR adds more detailed icon.sys lines, adds a "Do Not Delete" delete icon to the sets to discourage users from deleting important files in the browser.
BOOT icon was updated to include a mini-lock and mini-gears icons.
SYS-CONF icon was updated to include mini-lock and wrench icons.
FMCB icons were updated to have mini-lock and mini-gears icons.
image
image
Graphics created/modified by BlackNinja

@NathanNeurotic
Copy link
Author

FMCB-1 953  _ _FMCB-1 953_20240824130142
FMCB-1 953  _ _FMCB-1 953_20240824130139
FMCB-1 953  _ _FMCB-1 953_20240824130134
FMCB-1 953  _ _FMCB-1 953_20240824130130
FMCB-1 953  _ _FMCB-1 953_20240824130126
FMCB-1 953  _ _FMCB-1 953_20240824130121

@TnA-Plastic
Copy link

TnA-Plastic commented Aug 26, 2024

Nice work! @NathanNeurotic (Ripto) and @koraxial (BlackNinja)!

@israpps
Copy link
Owner

israpps commented Sep 18, 2024

It's nice but it will eat up more space...

not sure if all users will like that

@NathanNeurotic
Copy link
Author

The per-install difference isn't much, while offering a uniform design and indicators for the users who delete/don't understand their setups.

We can adapt and tweak the results to fit your vision.
I'll see if we can optimize anything any further, maybe sway you.
image
image

@israpps
Copy link
Owner

israpps commented Sep 18, 2024

I wish there was a way for icon.sys to reference another icon with absolute path
that way we could have

mc?;/SYS-CONF/DONTDELETEME.ICN

and reference it from all needed folders

@TnA-Plastic

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

Successfully merging this pull request may close these issues.

3 participants