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

Cosmetic Changes #28

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

SecularSteve
Copy link

I decided to make it look a bit more modern with my design proposals for the new versions.

newicon1000

You are free to modify it how you please.

I also have the raw PSD files:
NewRaw1000.zip

Note: Only the options.css and ui.css files are modified in the ui folder.

I decided to make it look a bit more modern with my design proposals for the new versions.

You are free to modify it how you please.

I also have the raw PSD files:
@Znrl
Copy link

Znrl commented Oct 6, 2018

Since I just saw that there finally is a repository and this PR for cosmetic changes I would like to add what I did a while ago.
I already sent those to the developer a while back but never heard anything about it.
There were some threads in the forum like mine (https://forums.informaction.com/viewtopic.php?f=10&t=23751).
So if there is the time to look at the UI you can take my approaches into account.
My thought was to replace every icon with SVGs.
(SVGs done with Inkscape)
noscript-svg-icons.zip

@hackademix
Copy link
Owner

Thank you very much for your contribution.
Unfortunately I cannot just merge it as it is, and not just because someone may prefer the "old" icons and therefore this should be made a preference, but also because I'm working to port it to Chromium, which doesn't support SVG icons.
Therefore this "alternate" icons set must be both optional and automatically disabled where not supported.
I'll be glad to add it, eventually, though, when I (or some other contributor) can submit a patch satisfying both these requirements.
Thanks again.

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