Skip to content

fix(core/modal): Modal showing outside viewport when trigger button at bottom of the page#2168

Merged
danielleroux merged 2 commits into
siemens:mainfrom
varun-srinivasa:ix/dropdown-button
Oct 6, 2025
Merged

fix(core/modal): Modal showing outside viewport when trigger button at bottom of the page#2168
danielleroux merged 2 commits into
siemens:mainfrom
varun-srinivasa:ix/dropdown-button

Conversation

@varun-srinivasa

Copy link
Copy Markdown
Collaborator

💡 What is the current behavior?

If the user has scrolled down the web page to trigger the modal, the modal dialog may be rendered outside the visible area of the viewport, making it inaccessible without manual scrolling.

GitHub Issue Number: #2083

🆕 What is the new behavior?

If the user has scrolled down the web page to trigger the modal, the modal dialog will be rendered in visible area of the viewport.

🏁 Checklist

A pull request can only be merged if all of these conditions are met (where applicable):

  • 🦮 Accessibility (a11y) features were implemented
  • 🗺️ Internationalization (i18n) - no hard coded strings
  • 📲 Responsiveness - components handle viewport changes and content overflow gracefully
  • 📕 Add or update a Storybook story
  • 📄 Documentation was reviewed/updated siemens/ix-docs
  • 🧪 Unit tests were added/updated and pass (pnpm test)
  • 📸 Visual regression tests were added/updated and pass (Guide)
  • 🧐 Static code analysis passes (pnpm lint)
  • 🏗️ Successful compilation (pnpm build, changes pushed)

👨‍💻 Help & support

@changeset-bot

changeset-bot Bot commented Sep 25, 2025

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: d3208bb

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages
Name Type
@siemens/ix Patch
@siemens/ix-angular Patch
@siemens/ix-react Patch
@siemens/ix-vue Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@lzeiml lzeiml self-requested a review September 29, 2025 08:11
@varun-srinivasa varun-srinivasa marked this pull request as ready for review September 30, 2025 02:30
@lzeiml lzeiml changed the title Modal showing outside viewport when trigger button at bottom of the page fix(core/modal): Modal showing outside viewport when trigger button at bottom of the page Sep 30, 2025
@sonarqubecloud

Copy link
Copy Markdown

@danielleroux danielleroux merged commit 39425ca into siemens:main Oct 6, 2025
14 checks passed
@github-actions github-actions Bot mentioned this pull request Oct 6, 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.

3 participants