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

Add Dropout and Nebula default icons #175

Merged
merged 8 commits into from
Jan 22, 2025
Merged

Conversation

ebower42
Copy link
Contributor

These are two icons I created in the svg editor to add buttons for the Nebula and Dropout.tv streaming services to my remote card. I figured I would try creating a PR to add these as defaults so others can use them as well.

Added default icons for the Dropout.tv and Nebula streaming services
@Nerwyn Nerwyn changed the base branch from main to dev January 15, 2025 05:50
@Nerwyn
Copy link
Owner

Nerwyn commented Jan 15, 2025

You should add them to the default sources while you're at it.

@ebower42
Copy link
Contributor Author

You should add them to the default sources while you're at it.

I unfortunately only have an AppleTV so I only know the sources for that platform. Is there somewhere I can look for source names for other platforms?

@ebower42
Copy link
Contributor Author

Ah I missed the links in the default source files. I will look at those sites

@Nerwyn
Copy link
Owner

Nerwyn commented Jan 15, 2025

Do your best and I can try to fill in the gaps later!

@Nerwyn Nerwyn merged commit a236c93 into Nerwyn:dev Jan 22, 2025
1 check passed
@Nerwyn
Copy link
Owner

Nerwyn commented Jan 22, 2025

Thanks for your hard work! These icons and default sources will be added in the next patch version.

@ebower42
Copy link
Contributor Author

Thanks for allowing me to make my first contribution to an open-source project!

@Nerwyn
Copy link
Owner

Nerwyn commented Jan 22, 2025

Added in 4.3.7

@ebower42 ebower42 deleted the ebower42-new-icons branch January 24, 2025 17:03
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.

2 participants