Skip to content

fix: scope thread options hover state#5606

Merged
timothycarambat merged 2 commits into
Mintplex-Labs:masterfrom
officialasishkumar:4994-thread-options-hover-state
May 12, 2026
Merged

fix: scope thread options hover state#5606
timothycarambat merged 2 commits into
Mintplex-Labs:masterfrom
officialasishkumar:4994-thread-options-hover-state

Conversation

@officialasishkumar
Copy link
Copy Markdown
Contributor

Pull Request Type

  • ✨ feat (New feature)
  • 🐛 fix (Bug fix)
  • ♻️ refactor (Code refactoring without changing behavior)
  • 💄 style (UI style changes)
  • 🔨 chore (Build, CI, maintenance)
  • 📝 docs (Documentation updates)

Relevant Issues

resolves #4994

Description

Scopes the thread options hover state to the individual thread row. The active workspace wrapper also uses Tailwind's generic group class, so the nested thread option button was responding to the workspace hover state and revealing the three-dot menu for every thread in that workspace.

This uses a named Tailwind group for each thread row and keeps the existing mobile-visible, desktop-hover behavior. The option button also appears while focus remains within the row.

Visuals (if applicable)

N/A

Additional Information

Validated that the production CSS build emits the named group/thread hover and focus-within rules.

Developer Validations

  • I ran yarn lint from the root of the repo & committed changes
  • Relevant documentation has been updated (if applicable)
  • I have tested my code functionality
  • Docker build succeeds locally

Validation performed:

  • YARN_CACHE_FOLDER=.yarn-cache yarn install --frozen-lockfile --ignore-scripts in frontend
  • yarn lint:check in frontend
  • yarn build in frontend
  • git diff --check

@shatfield4 shatfield4 self-requested a review May 11, 2026 21:22
Copy link
Copy Markdown
Collaborator

@shatfield4 shatfield4 left a comment

Choose a reason for hiding this comment

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

LGTM, no longer shows the 3 dot menu for all threads, just the currently selected thread.

@shatfield4 shatfield4 requested a review from angelplusultra May 11, 2026 21:23
@timothycarambat timothycarambat added the PR: Ready for QA Indicates that a PR is ready for QA and the GH image may build the image label May 11, 2026
@timothycarambat timothycarambat self-assigned this May 11, 2026
@timothycarambat timothycarambat removed the PR: Ready for QA Indicates that a PR is ready for QA and the GH image may build the image label May 12, 2026
@timothycarambat timothycarambat merged commit b90aa12 into Mintplex-Labs:master May 12, 2026
1 of 2 checks passed
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.

[BUG]: 3 dot menu hover state on thread

4 participants