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

[Bug]: AppButton colours don't change while switching themes #1356

Closed
PWagner1 opened this issue Feb 26, 2024 · 9 comments
Closed

[Bug]: AppButton colours don't change while switching themes #1356

PWagner1 opened this issue Feb 26, 2024 · 9 comments
Labels
area:ribbon All issues to do with the ribbon. area:themes All things to do with themes. bug Something isn't working completed This issue has been completed.
Milestone

Comments

@PWagner1
Copy link
Contributor

While testing #1343, it has been discovered that AppButton colours don't change while switching themes. Looks as if it's not repainting somewhere.

AppButton Colours

@PWagner1 PWagner1 added bug Something isn't working area:themes All things to do with themes. area:ribbon All issues to do with the ribbon. labels Feb 26, 2024
@Smurf-IV
Copy link
Member

Smurf-IV commented Apr 2, 2024

I would expect the following to have an input:
image

But even when they are empty a colour is still being found from somewhere.

@Smurf-IV Smurf-IV self-assigned this Apr 2, 2024
@Smurf-IV Smurf-IV added the under investigation This bug/issue is currently under investigation. label Apr 2, 2024
@Smurf-IV
Copy link
Member

Smurf-IV commented Apr 3, 2024

Narrowed it down to these Palette values:
image

Nothing to do with the designer values above?!?

@Smurf-IV
Copy link
Member

Smurf-IV commented Apr 3, 2024

To get this:
image

Set these things in the designer:
image

@Smurf-IV
Copy link
Member

Smurf-IV commented Apr 3, 2024

To get this:
image

Set the following in the designer:
image

Which means that the RibbonAppButton is really the QATButton !
So the Explanation text needs to change because changing the Designer text will break all the serialisation of that object !!

@Smurf-IV
Copy link
Member

Smurf-IV commented Apr 3, 2024

@Wagnerp Proposal:
Because the ##Dark and ##Light values are currently not overridden (Used) in the serialisation of the Palettes:

  • Remove from their current "General" location in the designer
  • Add new Area and values to the State### area with an appropriate named Key: i.e. RibbonFileAppButton
    • Use the existing values as the defaults, but not "Hard Code" them !
  • Wire up the usage of Disabled etc usages to use the ##Dark and ##Light overrides to follow the correct pattern
  • Update the Breaking Changes area to state correct usages of the RibbonFileAppButton values
  • Fix Examples
  • Fix Theme Template editor
    Thoughts ?

@PWagner1
Copy link
Contributor Author

PWagner1 commented Apr 3, 2024

@Wagnerp Proposal: Because the ##Dark and ##Light values are currently not overridden (Used) in the serialisation of the Palettes:

  • [] Remove from their current "General" location in the designer

  • [] Add new Area and values to the State### area with an appropriate named Key: i.e. RibbonFileAppButton

    • Use the existing values as the defaults, but not "Hard Code" them !
  • [] Wire up the usage of Disabled etc usages to use the ##Dark and ##Light overrides to follow the correct pattern

  • [] Update the Breaking Changes area to state correct usages of the RibbonFileAppButton values

Thoughts ?

@Smurf-IV Seems fine to me.

@Smurf-IV Smurf-IV added this to the Version 90 milestone Apr 3, 2024
@Smurf-IV
Copy link
Member

Smurf-IV commented Apr 5, 2024

This is happening:
image

And a new entry for the text (To make it clear what it is!)
image

@Smurf-IV
Copy link
Member

Smurf-IV commented Apr 5, 2024

Done so far:

  • Remove from their current "General" location in the designer
  • Add new Area and values to the State### area with an appropriate named Key: i.e. RibbonFileAppButton
    • Use the existing values as the defaults, but not "Hard Code" them !
  • Wire up the usage of Disabled etc usages to use the ##Dark and ##Light overrides to follow the correct pattern
  • Update the Breaking Changes area to state correct usages of the RibbonFileAppButton values
  • Fix Examples
  • Fix Theme Template editor

Smurf-IV added a commit that referenced this issue Apr 5, 2024
- [x] Remove from their current "General" location in the designer
- [x] Add new Area and values to the `State###` area with an appropriate named Key: i.e. `RibbonFileAppButton`
  - [ ] Use the existing values as the defaults, but not "Hard Code" them !
- [x] Wire up the usage of Disabled etc usages to use the `##Dark` and `##Light` overrides to follow the correct pattern
- [x] Update the Breaking Changes area to state correct usages of the `RibbonFileAppButton` values
- [ ] Fix Examples
- [ ] Fix Theme Template editor

#1356
@Smurf-IV Smurf-IV mentioned this issue Apr 5, 2024
7 tasks
Smurf-IV added a commit to Krypton-Suite/Standard-Toolkit-Demos that referenced this issue Apr 5, 2024
Smurf-IV added a commit to Krypton-Suite/Theme-Palettes that referenced this issue Apr 5, 2024
@Smurf-IV
Copy link
Member

Smurf-IV commented Apr 5, 2024

  • Remove from their current "General" location in the designer
  • Add new Area and values to the State### area with an appropriate named Key: i.e. RibbonFileAppButton
    • Use the existing values as the defaults, but not "Hard Code" them !
  • Wire up the usage of Disabled etc usages to use the ##Dark and ##Light overrides to follow the correct pattern
  • Update the Breaking Changes area to state correct usages of the RibbonFileAppButton values
  • Fix Examples
  • Fix Theme Template editor
  • [ ]
    SwitchFileTabColours

@Smurf-IV Smurf-IV closed this as completed Apr 5, 2024
@Smurf-IV Smurf-IV added completed This issue has been completed. and removed under investigation This bug/issue is currently under investigation. labels Apr 5, 2024
@Smurf-IV Smurf-IV removed their assignment Apr 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:ribbon All issues to do with the ribbon. area:themes All things to do with themes. bug Something isn't working completed This issue has been completed.
Projects
None yet
Development

When branches are created from issues, their pull requests are automatically linked.

2 participants