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

#1150 updated solution to align with the existing structure #1178

Open
wants to merge 2 commits into
base: beta
Choose a base branch
from

Conversation

Amelie1253
Copy link
Collaborator

No description provided.

@Amelie1253 Amelie1253 changed the base branch from milka-1150 to beta February 25, 2025 00:39
@Amelie1253 Amelie1253 marked this pull request as ready for review February 25, 2025 00:39
@MilkaZek
Copy link
Collaborator

We created a new property in grnstate.js (demoDropdownValue) to hold the value of the Demo Dropdown and in update-app.js we check to see if demoDropdownValue is equal to null. If it is, we reset the dropdown selection. When a custom workbook is being displayed, demoDropdownValue is set to null. If a demo dropdown value is selected, the value of the new grnstate property is changed so that it matches what was selected and isn't null anymore.

We think we need to find out how the three demo dropdowns are connected in their values to move forward because with the addition of our new code, when the top dropdown is changed, the one on the sidebar changes to match but then is set to null.

@Amelie1253 Amelie1253 changed the title updated solution to align with the existing structure #1150 updated solution to align with the existing structure Feb 25, 2025
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.

2 participants