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

MoreCast: Refactor weather parameter tab state into an array #3509

Open
dgboss opened this issue Apr 3, 2024 · 0 comments
Open

MoreCast: Refactor weather parameter tab state into an array #3509

dgboss opened this issue Apr 3, 2024 · 0 comments

Comments

@dgboss
Copy link
Collaborator

dgboss commented Apr 3, 2024

We're running into a race condition when toggling the visibility of weather parameter tabs and the Forecast Summary tab. To reproduce:

  1. Launch MoreCast and click the RH tab so both temp and rh are visible.
  2. Click the Forecast Summary tab
  3. Click the RH tab.

Actual Result: Temp and RH are both visible
Expected Result: Only the RH tab should be visible.

I believe this is happening due to each tab having independent state for managing its visibility and that we can workaround the problem by using a single object to manage the state of all the tabs.

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

No branches or pull requests

1 participant