Skip to content

Initial support for NSO applet (by ELY3M) #105

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

Closed
wants to merge 1 commit into from
Closed

Initial support for NSO applet (by ELY3M) #105

wants to merge 1 commit into from

Conversation

ShadowOne333
Copy link

Based on the fork by @ELY3M mentioned in issue #73.
This should add standard support for editing and dumping the NSO applet.

Needs extra work (according to ELY3M) to be able to change the background color of the icon.

@exelix11
Copy link
Owner

It doesn't seem to add much aside from some needed definitions and from what i understand it doesn't actually work.
I'm not currently interested in developing new features, i'm closing this unless you're actually willing to finish implementing it.

@exelix11 exelix11 closed this Nov 30, 2021
@ELY3M
Copy link

ELY3M commented Dec 1, 2021

actually, it is working for me. I have to do tweaking with layout json and stuff. I ended up using black background and it look better on my switch. this is one way to get rid of red background is this layout json at https://pastebin.com/c3HFRRQS

@ELY3M
Copy link

ELY3M commented Dec 1, 2021

here is layout json with all black background. I am using this layout right now.
https://pastebin.com/4JhN3UGV

@exelix11
Copy link
Owner

exelix11 commented Dec 1, 2021

Are you sure there is no way of to make it look like the other buttons like by copying over the properties ? Maybe the code behind doesn't theme it at all so it is impossible, idk.
Changing all of them so they match feels hacky and probably breaks the system color "themes".

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