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

VeSTige preset left and right arrows are gone #3498

Closed
karmux opened this issue Apr 10, 2017 · 10 comments
Closed

VeSTige preset left and right arrows are gone #3498

karmux opened this issue Apr 10, 2017 · 10 comments

Comments

@karmux
Copy link
Contributor

karmux commented Apr 10, 2017

In VeSTige instrument left and right arrows to browse presets are missing. Instead it shows something that looks like dead pixels. I remember that I was using these arrows about month ago on master branch.

screenshot_20170411_004313

This picture from Wiki shows how it used to look:

LMMS 1.2.0-rc2.144
(Linux x86_64, Qt 5.6.1, GCC 6.2.0 20161005)

@BaraMGB
Copy link
Contributor

BaraMGB commented Apr 10, 2017

Those pixels appears if a pixmap couldn't be found.

@karmux
Copy link
Contributor Author

karmux commented Apr 11, 2017

@lukas-w , when I revert 9f905bc I get icons back.
After looking into it a bit I have no idea why some certain pixmaps always get missing.

@tresf
Copy link
Member

tresf commented Apr 11, 2017

@karmux please see this comment... #1891 (comment)

@karmux
Copy link
Contributor Author

karmux commented Apr 11, 2017

@tresf but vestige has no underscore in the name, nor has stepper-left.png. I can make stepper-left pixmap visible if I use it for "Show/hide GUI" button instead of zoom pixmap.

@tresf
Copy link
Member

tresf commented Apr 11, 2017

@karmux good call. What if you make this line match the others (such as in vibed):

vestige

BUILD_PLUGIN(vestige EMBEDDED_RESOURCES "${CMAKE_CURRENT_SOURCE_DIR}/*.png")

vibed

BUILD_PLUGIN(vibedstrings [...] EMBEDDED_RESOURCES *.png)

@tresf
Copy link
Member

tresf commented Apr 11, 2017

Scratch that... that won't change anything and these don't exist in that directory anyway. These appear to be shared artwork. Still hunting...

@tresf
Copy link
Member

tresf commented Apr 11, 2017

Ok... I think we need to replace PLUGIN_NAME:: with embed:: for shared LMMS (non-plugin) resources. @lukas-w should confirm...

@karmux
Copy link
Contributor Author

karmux commented Apr 11, 2017

I got all pixmaps back by replacing PLUGIN_NAME::getIconPixmap with embed::getIconPixmap in vestige.cpp only for missing pixmaps. Replacing for every pixmap made other pixmaps to disappear.

state_stop icon actually seems like missing (also on Wiki image).

karmux added a commit to karmux/lmms that referenced this issue Apr 12, 2017
@karmux
Copy link
Contributor Author

karmux commented Apr 12, 2017

VeSTige in #3501.

screenshot_20170412_202826

@karmux
Copy link
Contributor Author

karmux commented Apr 28, 2017

Fixed in #3501.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants