Skip to content

Conversation

crisnicandrei
Copy link
Contributor

Implemented a workaround to allow a custom div/button to toggle the open state of a native by adjusting its size attribute. This mimics dropdown behavior without relying on unsupported programmatic opening. Also handles closing on selection. Steps to test: Go to archive members Go to add Click on the role The dropdown should open

@crisnicandrei crisnicandrei requested a review from slifty July 23, 2025 12:36
Copy link

codecov bot commented Jul 23, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 44.85%. Comparing base (b1f6c08) to head (88fe97e).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #618      +/-   ##
==========================================
+ Coverage   44.77%   44.85%   +0.08%     
==========================================
  Files         371      371              
  Lines       11294    11298       +4     
  Branches     1850     1850              
==========================================
+ Hits         5057     5068      +11     
+ Misses       6067     6062       -5     
+ Partials      170      168       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@crisnicandrei crisnicandrei force-pushed the PER-10229-fix-archive-member-invite-dropdown branch 2 times, most recently from 1209630 to 88fe97e Compare July 23, 2025 12:55
@cecilia-donnelly cecilia-donnelly self-requested a review September 5, 2025 18:10
@cecilia-donnelly
Copy link
Member

I am rebasing on main, which includes a lot of indentation/linting fixes, and I want to keep track of the original commit hash in case I need to get back to it: 88fe97e

Implemented a workaround to allow a custom div/button to toggle the open state of a native <select>
by adjusting its size attribute. This mimics dropdown behavior without relying on unsupported
programmatic opening. Also handles closing on selection.
@cecilia-donnelly cecilia-donnelly force-pushed the PER-10229-fix-archive-member-invite-dropdown branch from 88fe97e to f9ffad2 Compare September 9, 2025 20:53
@cecilia-donnelly cecilia-donnelly requested review from aasandei-vsp and removed request for slifty September 9, 2025 21:13
@cecilia-donnelly
Copy link
Member

@aasandei-vsp, no need to look at this yet - I broke it while rebasing and haven't figured out how to fix yet. :)

@cecilia-donnelly cecilia-donnelly self-assigned this Sep 15, 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