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

Make BB-tracks themeable #817

Merged
merged 1 commit into from
Jun 9, 2014
Merged

Make BB-tracks themeable #817

merged 1 commit into from
Jun 9, 2014

Conversation

diizy
Copy link
Contributor

@diizy diizy commented Jun 6, 2014

  • This allows defining a default colour for BB-track patterns in the CSS
  • The default colour is used for all bb-patterns which don't have a custom colour set by the user: in other words, the colour of a pattern can be any rgb-value OR "style colour"
  • By default, all created bb-patterns use the style colour
  • You can also reset colourized patterns to use style colour again
  • Backwards compatibility: old projects will be loaded so that any pattern using either of the old default colours will be converted to use style colour

TODO: add a settings option that can disable custom colours (ie. always use style colour), and/or an option to reset all patterns in a project to style colour. This is needed, since themes can now change the song editor background, which can lead to unfortunate colour combinations with custom colours...

- This allows defining a default colour for BB-track patterns in the CSS
- The default colour is used for all bb-patterns which don't have a custom colour set by the user: in other words, the colour of a pattern can be any rgb-value OR "style colour"
- By default, all created bb-patterns use the style colour
- You can also reset colourized patterns to use style colour again
- Backwards compatibility: old projects will be loaded so that any pattern using either of the old default colours will be converted to use style colour

TODO: add a settings option that can disable custom colours (ie. always use style colour), and/or an option to reset all patterns in a project to style colour. This is needed, since themes can now change the song editor background, which can lead to unfortunate colour combinations with custom colours...
@Sti2nd
Copy link
Contributor

Sti2nd commented Jun 6, 2014

Good work! :godmode: So was this the finale code for full theme support? ❤️

@diizy
Copy link
Contributor Author

diizy commented Jun 6, 2014

On 06/06/2014 01:57 PM, Stian Jørgensrud wrote:

Good work! :godmode: So was this the finale code for full theme
support? ❤️

There's probably some small stuff left here and there.

tobydox added a commit that referenced this pull request Jun 9, 2014
@tobydox tobydox merged commit 019c438 into LMMS:stable-1.1 Jun 9, 2014
@Sti2nd Sti2nd mentioned this pull request Jan 24, 2015
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.

3 participants