Skip to content

Make arrow "stepper" buttons themeable and use SVG assets#8123

Merged
messmerd merged 10 commits intoLMMS:masterfrom
rubiefawn:svg-arrow-btn
Nov 21, 2025
Merged

Make arrow "stepper" buttons themeable and use SVG assets#8123
messmerd merged 10 commits intoLMMS:masterfrom
rubiefawn:svg-arrow-btn

Conversation

@rubiefawn
Copy link
Contributor

@rubiefawn rubiefawn commented Nov 5, 2025

This upgrades the arrow buttons used in the instrument windows, VST effects, and Vestige from PNG images to CSS utilizing SVG assets for the icons inside.

Before

image image

After

image image

Note

Part of #7767

These buttons are used in the instrument window, Vestige, and VST
effects. Separate versions of the arrow icons are used for the classic
theme.
@rubiefawn rubiefawn added enhancement gui needs code review A functional code review is currently required for this PR needs testing This pull request needs more testing labels Nov 5, 2025
LMMS renders these SVGs just fine, but apparently the removal of the
XML declaration completely breaks Github's ability to render the image,
so I am adding these back for the sake of those who want to actually
look at the diff on the website lmao
You may ask, "what does this have to do with the arrow buttons?" and you
would be right to assume this is unrelated. However, I'm already
touching the relevant lines of the stylesheet so I may as well sneak it
in there.
And fix mixed indentation in headphones.svg
The originals were right angle chevrons
@bratpeki
Copy link
Member

Tested, looks great!

@bratpeki
Copy link
Member

Please merge ASAP

@rubiefawn
Copy link
Contributor Author

Please merge ASAP

image

🤷‍♀️

@rubiefawn rubiefawn removed the needs testing This pull request needs more testing label Nov 21, 2025
@messmerd messmerd merged commit eb41051 into LMMS:master Nov 21, 2025
11 checks passed
@rubiefawn rubiefawn deleted the svg-arrow-btn branch November 21, 2025 07:23
@rubiefawn rubiefawn removed the needs code review A functional code review is currently required for this PR label Nov 21, 2025
@bratpeki bratpeki self-assigned this Nov 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants