Skip to content

Conversation

@ragafus
Copy link
Contributor

@ragafus ragafus commented Sep 17, 2025

Summary

MC apps hide notifications if the user has scrolled down the content. This happens because we are scrolling main, which hosts the notifications.

Priceless ticket https://commercetools.atlassian.net/browse/PRC-4262

Description

Before

before.mov

After

after.mov

@ragafus ragafus self-assigned this Sep 17, 2025
@changeset-bot
Copy link

changeset-bot bot commented Sep 17, 2025

🦋 Changeset detected

Latest commit: c3ca39e

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

This PR includes changesets to release 36 packages
Name Type
@commercetools-frontend/application-shell Patch
@commercetools-frontend/cypress Patch
@commercetools-applications/merchant-center-template-starter-typescript Patch
@commercetools-applications/merchant-center-template-starter Patch
@commercetools-applications/merchant-center-custom-view-template-starter-typescript Patch
@commercetools-applications/merchant-center-custom-view-template-starter Patch
@commercetools-local/playground Patch
@commercetools-local/visual-testing-app Patch
@commercetools-backend/eslint-config-node Patch
@commercetools-backend/express Patch
@commercetools-backend/loggers Patch
@commercetools-frontend/actions-global Patch
@commercetools-frontend/application-components Patch
@commercetools-frontend/application-config Patch
@commercetools-frontend/application-shell-connectors Patch
@commercetools-frontend/assets Patch
@commercetools-frontend/babel-preset-mc-app Patch
@commercetools-frontend/browser-history Patch
@commercetools-frontend/codemod Patch
@commercetools-frontend/constants Patch
@commercetools-frontend/create-mc-app Patch
@commercetools-frontend/eslint-config-mc-app Patch
@commercetools-frontend/i18n Patch
@commercetools-frontend/jest-preset-mc-app Patch
@commercetools-frontend/jest-stylelint-runner Patch
@commercetools-frontend/l10n Patch
@commercetools-frontend/mc-dev-authentication Patch
@commercetools-frontend/mc-html-template Patch
@commercetools-frontend/mc-scripts Patch
@commercetools-frontend/notifications Patch
@commercetools-frontend/permissions Patch
@commercetools-frontend/react-notifications Patch
@commercetools-frontend/sdk Patch
@commercetools-frontend/sentry Patch
@commercetools-frontend/url-utils Patch
@commercetools-website/components-playground 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

@vercel
Copy link

vercel bot commented Sep 17, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
mc-app-kit-playground Ready Ready Preview Comment Sep 17, 2025 2:23pm
merchant-center-application-kit-components-playground Ready Ready Preview Comment Sep 17, 2025 2:23pm

@ddouglasz
Copy link
Contributor

ddouglasz commented Sep 18, 2025

@commercetools/craft-team-fe please take a look 👀
⚠️ Since this is sort a "root" change that affects a lot of places in our applications, although approved, we should not merge yet until we can test that it is visually consistent in all possible use cases.

Copy link
Contributor

@misama-ct misama-ct left a comment

Choose a reason for hiding this comment

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

The change touches layers that should not be responsible for the intended change. The fact that the navigation appears sticky now is merely a side-effect.

Having that said: I can't tell you how to to achieve sticky notifications that reliably work the same across all applications without spending a significant amount of time on investigating the DOM structure and the surrounding css-rules.

@ragafus ragafus closed this Jan 13, 2026
@ragafus ragafus deleted the PRC-4262-hidden-notifications branch January 13, 2026 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants