Skip to content

fix: intercom bubble positioning properly#6111

Merged
IMB11 merged 6 commits into
mainfrom
cal/dev-1075-icom-bubbles-moves-when-action-bar-present
May 20, 2026
Merged

fix: intercom bubble positioning properly#6111
IMB11 merged 6 commits into
mainfrom
cal/dev-1075-icom-bubbles-moves-when-action-bar-present

Conversation

@IMB11
Copy link
Copy Markdown
Member

@IMB11 IMB11 commented May 16, 2026

  • Last version of doing this was not the right way - intercom bubble management has been moved to root layout for both app and website rather than on root.vue server panel page & the app's App.vue
  • Moved to a single composable which manages the entire intercom state
  • Removed dependency on intercom's sdk
  • Properly fixed collision issues with modals, action bars, sidebar etc.

@IMB11 IMB11 requested a review from a team May 16, 2026 15:24
@modrinth-bot
Copy link
Copy Markdown
Member

modrinth-bot commented May 16, 2026

Note

This changelog has been baked. Any further edits will not be reflected.

Pull request changelog

App

Added

Changed

Deprecated

Removed

Fixed

Security

Website

Added

Changed

Deprecated

Removed

Fixed

Security

Hosting

Added

Changed

Deprecated

Removed

Fixed

  • Fixed various positioning problems with the support portal bubble which appears in the bottom right of the panel.

Security

@modrinth-bot
Copy link
Copy Markdown
Member

modrinth-bot commented May 16, 2026

@IMB11 IMB11 enabled auto-merge May 18, 2026 15:24
Comment thread apps/frontend/src/layouts/default.vue Outdated
Copy link
Copy Markdown
Contributor

@tdgao tdgao left a comment

Choose a reason for hiding this comment

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

LGTM, just one small thing

@IMB11 IMB11 added this pull request to the merge queue May 20, 2026
@tdgao tdgao removed this pull request from the merge queue due to a manual request May 20, 2026
@IMB11 IMB11 enabled auto-merge May 20, 2026 17:09
@IMB11 IMB11 added this pull request to the merge queue May 20, 2026
Merged via the queue into main with commit 3eeb549 May 20, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants