Skip to content

Conversation

@moibra05
Copy link
Contributor

@moibra05 moibra05 commented Jan 27, 2026

Because

  • There were several instances in the new results page where collapsible elements were found within modals
  • Having both is superfluous so we can get rid of the nested collapsiblses

This commit

  • Turns all collapsible elements inside modals into static blocks

Fixes #14487

@moibra05
Copy link
Contributor Author

image image image

@yashikakhurana
Copy link
Contributor

@moibra05 when you get a chance can you fix the space here left and right? 🙏
Screenshot 2026-01-29 at 12 12 32 AM

@moibra05
Copy link
Contributor Author

moibra05 commented Jan 29, 2026

@moibra05 when you get a chance can you fix the space here left and right? 🙏 Screenshot 2026-01-29 at 12 12 32 AM

Oh that space is only there because i didnt change the dates on the experiment i was testing with so they didnt get rendered. normally it'd look like this:
image

@moibra05 moibra05 added this pull request to the merge queue Feb 2, 2026
github-merge-queue bot pushed a commit that referenced this pull request Feb 2, 2026
Because

- There were several instances in the new results page where collapsible
elements were found within modals
- Having both is superfluous so we can get rid of the nested
collapsiblses

This commit

- Turns all collapsible elements inside modals into static blocks

Fixes #14487
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 2, 2026
@moibra05 moibra05 removed preview Create Preview Environment pr-preview labels Feb 2, 2026
@moibra05 moibra05 added this pull request to the merge queue Feb 2, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 2, 2026
@moibra05 moibra05 added this pull request to the merge queue Feb 2, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 2, 2026
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.

Remove unnecessary collapsibles within results page sections

4 participants