Skip to content

AI: On mobile, closing the notification in window toggles the admin menu in #111826

Draft
agent-sandbox-automattic[bot] wants to merge 1 commit into
trunkfrom
ai/agent/dotmsd-1317-1781843456
Draft

AI: On mobile, closing the notification in window toggles the admin menu in #111826
agent-sandbox-automattic[bot] wants to merge 1 commit into
trunkfrom
ai/agent/dotmsd-1317-1781843456

Conversation

@agent-sandbox-automattic

Copy link
Copy Markdown
Contributor

Summary

On mobile, closing the notification in window toggles the admin menu in Omnibar.

If you're on a mobile device and you open your notifications panel, then click on the X to close it. It also opens the admin menu behind it, so the event is not being trapped in the notification.

Changes

Files

apps/notifications/src/app/templates/close-button.tsx | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

Linear: DOTMSD-1317

Co-authored-by: StevenDufresne steve.dufresne@automattic.com

This PR was auto-generated by Agent Sandbox. Please review carefully before merging.

Co-authored-by: StevenDufresne <steve.dufresne@automattic.com>
@matticbot

Copy link
Copy Markdown
Contributor

This PR modifies the release build for the following Calypso Apps:

For info about this notification, see here: PCYsg-OT6-p2

  • notifications

To test WordPress.com changes, run install-plugin.sh $pluginSlug ai/agent/dotmsd-1317-1781843456 on your sandbox.

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.

1 participant