Skip to content

sound: Rebrand as URL Playback to avoid confusion with the built-in sound category #2121

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

Merged
merged 6 commits into from
May 21, 2025

Conversation

ScratchFakemon
Copy link
Contributor

Includes revamped menu icon and banner (both made in Figma, inspired by @WAYLIVES's designs, font is Lohit Devanagari)
URL Playback (Sound) Icon
URL Playback (Sound)

I also changed the color scheme of the extension. The existing code was pretty bare-bones so adding an icon and a color wasn't too hard.

Other than rebrands, the extension's code is unchanged. The blocks still function the way they used to and the ID remains the same.

@github-actions github-actions bot added the pr: change existing extension Pull requests that change an existing extension label May 20, 2025
@ScratchFakemon
Copy link
Contributor Author

If you want me to change something, feel free too :)

@GarboMuffin
Copy link
Member

good idea

@GarboMuffin GarboMuffin changed the title Rebrand Sound (id: notSound) as URL Playback to avoid confusion with the built-in sound category. sound: Rebrand as URL Playback to avoid confusion with the built-in sound category May 21, 2025
@CubesterYT CubesterYT merged commit 44d4297 into TurboWarp:master May 21, 2025
3 checks passed
@Brackets-Coder
Copy link
Contributor

As a musician, it's really going to bother me that the note heads are centered with the stems instead of adjacent to the stems in the new banner. Eighth notes are supposed to look like this:
Image

@GarboMuffin
Copy link
Member

Can you fix it?

@Brackets-Coder
Copy link
Contributor

Can you fix it?

I'll probably open a PR sooner or later, there's probably other music familiar people that are bothered by it

@ScratchFakemon
Copy link
Contributor Author

@GarboMuffin was I supposed to update the image attribution or will you do that

@GarboMuffin
Copy link
Member

It would happen more quickly if you can submit a pull request

@ScratchFakemon
Copy link
Contributor Author

It would happen more quickly if you can submit a pull request

Alright I'll do that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: change existing extension Pull requests that change an existing extension
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants