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

Widgets left not themeable #2770

Open
3 of 7 tasks
IvanMaldonado opened this issue May 11, 2016 · 14 comments
Open
3 of 7 tasks

Widgets left not themeable #2770

IvanMaldonado opened this issue May 11, 2016 · 14 comments

Comments

@IvanMaldonado
Copy link
Contributor

IvanMaldonado commented May 11, 2016

Based on a post made in #1839 I have made a list of the things that are not still themable at all or individually.

List of currently unthemeable items

Themeable, but need improvement

  • Qt Tooltips (Radius)

Screenshots:
Mixer background color #2704
b+b editor background #2634
QTooltip radius
qtooltip
General settings/Buffer size section
generalsettings
Buttons in general
buttons
Help textbox
help
Sidemenu
sidemenu_border
The sidebar gradient is vertical
sidebar
Border style of some elements.
boxes
And plugins in general, not sure how viable this is, but skins for lmms plugins could be a good idea.

These are the ones I have found so far, I have removed some elements on my list because I decided they would be Fixes rather than editables. (Some elements that are not well centred, misused space, elements that are related in the .css file, etc...) As an example, the "sidebar gradient is vertical" could be considered something to fix because it is already editable, same with the QTooltip radius.

I can post the rest of the issues here or I can open another Issue asking for fixes and I can move the examples given there, whatever is better.

@tresf
Copy link
Member

tresf commented May 11, 2016

Thanks @IvanMaldonado.

Can you please change this to checklist format and either hyperlink the images or organize them a bit better so it's easier to read/track. Also, it may be worth mentioning (related because it's for styling, although not a duplicate):

#2510 (comment)

@liushuyu
Copy link
Member

@tresf I'm confirming this...

@tresf
Copy link
Member

tresf commented May 11, 2016

@liushuyu Please feel free to amend the original post as needed (since you have access to). 👍

@liushuyu
Copy link
Member

@IvanMaldonado Sorry, but I can't figure out what are the problems with General settings/Buffer size section and General Buttons ? What do you want to change about these widgets?

@liushuyu
Copy link
Member

liushuyu commented May 11, 2016

@tresf Okay

@IvanMaldonado I have edited your issue per @tresf 's request, please feel free to correct it if I was wrong XD

@IvanMaldonado
Copy link
Contributor Author

It's ok, I'm fine with any modification to my post in order to make it more redeable .
@liushuyu The problem of the general settings/Buffer size section is the scroll bar.
And one can not change the form of the general buttons, nor their borders or the aspect they have when you hover on them.

@liushuyu
Copy link
Member

@IvanMaldonado That makes more sense now, thanks for your clarification!

@RebeccaDeField
Copy link
Contributor

Thanks for getting this list together @IvanMaldonado 👍

@michaelgregorius
Copy link
Contributor

@IvanMaldonado It seems that the top-bottom sidebar gradient is rather a problem with the style sheet that you use. In the original style sheet the gradient goes from left to right. Here's a screenshot with adjusted colors to make it more clear:
2770-sidebar gradient
The original style sheet looks as follows:

background: qlineargradient(x1:0, y1:0, x2:1, y2:0, stop: 0 #98a2a7, stop: 1.0 #5b646f);

I guess you have it set to something like (0, 0) -> (0, 1) instead of (0, 0) -> (1, 0).

@Umcaruje
Copy link
Member

skins for lmms plugins could be a good idea.

You can already do that. You just put the files necessary into a plugins folder. Here's an example: https://github.com/StakeoutPunch/LMMS_Themes/tree/master/uiX/plugins

@IvanMaldonado
Copy link
Contributor Author

IvanMaldonado commented May 13, 2016

@michaelgregorius You're right, it was my bad 🌵
@Umcaruje I see, so if I make a folder called plugins, will lmms scan for the skins automatically? also, how do I know how to name each element of each plugin? and... currently the LSP just allows you to upload 1MB files, which I don't think will be enough for a theme with skins for every plugin.

EDIT: I have removed the vertical sidebar theming problem since it was a mistake of mine and not an actual problem 👍 I will post the other stuff I found later, I want to check them out deeply this time.

@Umcaruje
Copy link
Member

how do I know how to name each element of each plugin?

Look at its source code. Example: https://github.com/LMMS/lmms/tree/master/plugins/audio_file_processor

You can see all the necessary png's there. You just need to rename them to be like PluginName_ImageName.png. Note that this will change with #1891, where you will have seperate folders and not prefixes. That PR is not going to go into 1.2 so you shouldn't worry about it.

currently the LSP just allows you to upload 1MB files, which I don't think will be enough for a theme with skins for every plugin.

That's a valid concern. @tresf should we perhaps make larger files uploadable to LSP? Do we have enough server room?

@tresf
Copy link
Member

tresf commented May 18, 2016

That's a valid concern. @tresf should we perhaps make larger files uploadable to LSP? Do we have enough server room?

👍 LMMS/lmms.io@df05bf8

@Umcaruje
Copy link
Member

👍 awesome!

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

6 participants