Skip to content

Conversation

@william-valencia
Copy link
Contributor

@william-valencia william-valencia commented Sep 4, 2025

Overview

What is the feature?

Fixing the dropdown issue in groups after searching a second time. Also enlarging the "more actions" icon

What is the Solution?

Adjusted the z-index of the dropdown and enlarged the icon.

What areas of the application does this impact?

React MMT

Testing

Reproduction steps

  • Environment for testing:
  • Collection to test with:

Go to Groups and select a provider then select submit.
Now go to select a provider again and notice that the drop-down menu is not obscured by the table.

Select a collection and view the more actions icon. MAke sure its size is inline with with Edit/Delete/Clone buttons

Attachments

Screenshot 2025-09-04 at 6 16 01 PM Screenshot 2025-09-04 at 6 12 48 PM

Checklist

  • I have added automated tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings

@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.99%. Comparing base (dd094bb) to head (5b0a5dd).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1405   +/-   ##
=======================================
  Coverage   97.99%   97.99%           
=======================================
  Files         421      421           
  Lines        6737     6737           
  Branches     1417     1418    +1     
=======================================
  Hits         6602     6602           
  Misses        134      134           
  Partials        1        1           

☔ 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.

@eudoroolivares2016 eudoroolivares2016 merged commit 93d0f1e into main Sep 5, 2025
12 checks passed
@eudoroolivares2016 eudoroolivares2016 deleted the MMT-4069 branch September 5, 2025 15:41
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.

6 participants