Closed gmukul01 closed 1 year ago
Please retry analysis of this Pull-Request directly on SonarCloud.
CI is running/has finished running commands for commit 26517339967f3042219cf9e7c596fa7623d37126. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.
📂 See all runs for this branch
Sent with 💌 from NxCloud.
affects: @medly-components/core
PR Checklist
Description
Set overflow visible to
Accordion.Content
component when accordion is visible so that if that renders any popup that can be visibleType of change
How has this been tested?
(Replace This Text: Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration.)
[ ] Test A
[ ] Test B
Fixes #
What is the current behaviour?
Accordion.Content
is by default overflow hidden and because of that any pop up like datepicker is not visibleWhat is the new behaviour?
Overflow is set to visible for
Accordion.Content
component when accordion is visible so that any pop up inside the content can be visibleDoes this PR introduce a breaking change?
Note: (Replace This Text: If this PR contains a breaking change please describe the impact and migration path for existing application.)
Additional context
(Replace This Text: Please describe any other related information or add screenshots of the PR.)
Checklist
[ ] My code follows the style guidelines of this project
[ ] 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
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules