Skip to content

[AI Chat]: Fix Suggested Questions Overlapping Menu #30538

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Douglashdaniel
Copy link
Contributor

Description

Fixes a style bug where the Suggested Question buttons would sometimes overlap the Regenerate Answer menu.

Resolves brave/brave-browser#48256

Test Plan:

  1. Start a conversations with Leo ai with a Tab attached to the question
  2. After you get an answer, open the Regenerate Answer menu
  3. The Suggested Questions should not overlap the Menu

Before:

Screenshot 8

After:

Screenshot 7

@Douglashdaniel Douglashdaniel force-pushed the ai-chat-fix-suggested-questions-overlapping-menu branch from 5c3aaef to 149a9db Compare August 7, 2025 19:48
@github-actions github-actions bot added the CI/storybook-url Deploy storybook and provide a unique URL for each build label Aug 7, 2025
Comment on lines +131 to +132
'--iframe-position-for-menus',
isOpen ? 'relative' : 'unset',
Copy link
Contributor

Choose a reason for hiding this comment

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

hey, could we not just set this to relative always and get rid of the mojom interface?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Unfortunately no otherwise the iframe wrapper then overlaps the Suggested Question buttons and they become un-clickable.

Copy link
Contributor

@fallaciousreasoning fallaciousreasoning Aug 8, 2025

Choose a reason for hiding this comment

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

Aaah, so basically we need to pull the iframe to the top when its showing a menu. Okay, I guess that kind of makes sense.

Maaaaan iframes make everything hard 😆

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI/storybook-url Deploy storybook and provide a unique URL for each build
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Leo] Suggested Questions overlaps Regenerate Answer Menu
2 participants