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

fix(dialog): adjust the handling of the hasCloseButton flag to avoid a potential infinite loop with React 19 #2261

Merged
merged 1 commit into from
Jun 21, 2024

Conversation

acd02
Copy link
Contributor

@acd02 acd02 commented Jun 20, 2024

fix(dialog)

Description, Motivation and Context

Adjust the handling of the hasCloseButton flag to avoid a potential infinite loop with React 19

Types of changes

  • 🛠️ Tool
  • 🪲 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 🧾 Documentation
  • 📷 Demo
  • 🧪 Test
  • 🧠 Refactor
  • 💄 Styles

@acd02 acd02 added Type: Bug A bug to fix Component Related to a component Component: dialog Related to dialog component Component: popover Related to label component labels Jun 20, 2024
@acd02 acd02 self-assigned this Jun 20, 2024
Copy link

codecov bot commented Jun 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.56%. Comparing base (c957f90) to head (9a6db95).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2261      +/-   ##
==========================================
- Coverage   97.56%   97.56%   -0.01%     
==========================================
  Files         760      760              
  Lines        6008     6001       -7     
  Branches     2138     2137       -1     
==========================================
- Hits         5862     5855       -7     
  Misses        144      144              
  Partials        2        2              

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

@acd02 acd02 force-pushed the fix-dialog-popover-hasbutton branch from 0a634f1 to a56ac6d Compare June 20, 2024 18:09
@acd02 acd02 changed the title fix: Adjust the handling of the hasCloseButton flag to avoid a potential infinite loop with React 19 fix(dialog): adjust the handling of the hasCloseButton flag to avoid a potential infinite loop with React 19 Jun 20, 2024
@acd02 acd02 removed the Component: popover Related to label component label Jun 20, 2024
@acd02 acd02 force-pushed the fix-dialog-popover-hasbutton branch from a56ac6d to 792a1ac Compare June 21, 2024 14:38
…ntial infinite loop (React 19)

use css :has selector + group tailwind modifer instead of relying on JS
Copy link
Contributor

@soykje soykje left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉

@acd02 acd02 merged commit 19d48c5 into main Jun 21, 2024
10 of 11 checks passed
@acd02 acd02 deleted the fix-dialog-popover-hasbutton branch June 21, 2024 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: dialog Related to dialog component Component Related to a component Type: Bug A bug to fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants