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

Clocks and Timers don't apply and persist Style settings properly #3930

Closed
charlesh88 opened this issue Jun 8, 2021 · 5 comments · Fixed by #4372
Closed

Clocks and Timers don't apply and persist Style settings properly #3930

charlesh88 opened this issue Jun 8, 2021 · 5 comments · Fixed by #4372
Assignees
Labels
bug:visual Visual problem. Not a functional issue needs:e2e Needs an e2e test needs:test instructions Missing testing notes severity:trivial type:bug
Milestone

Comments

@charlesh88
Copy link
Contributor

charlesh88 commented Jun 8, 2021

Set styles in Display Layout. In this example the text color is set to a red; should be applied to all text but is only affecting the non-dynamic text:
Screen Shot 2021-06-08 at 11 09 37 AM

Save, nav away and back.
Screen Shot 2021-06-08 at 11 09 59 AM

Clock settings are persisted, but Timers don't retain anything.

@unlikelyzero unlikelyzero added the bug:visual Visual problem. Not a functional issue label Jun 8, 2021
@skylinew
Copy link

Do you have steps to reproduce?

@depperm
Copy link
Contributor

depperm commented Oct 23, 2021

Steps to reproduce:

  1. Create Display Layout
  2. Create child clock and timer
  3. Edit Display Layout and set clock and timer next to each other.
  4. Select each component, click the styles tab on the right
  5. Click set text color, choose a color.

Have a fix for this one too

@charlesh88
Copy link
Contributor Author

Verified fixed Testathon 06-15-22.

@rukmini-bose
Copy link
Contributor

Verified Testathon 06-15-22

@unlikelyzero unlikelyzero added this to the Sprint:2.0.7 milestone Jul 17, 2022
@unlikelyzero
Copy link
Contributor

@unlikelyzero to finish the test.

A lot of scaffolding was required to get to this point in a testsuite

@unlikelyzero unlikelyzero self-assigned this Jul 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug:visual Visual problem. Not a functional issue needs:e2e Needs an e2e test needs:test instructions Missing testing notes severity:trivial type:bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants