Skip to content

fix(web): move context menus setting to appearance - #90

Merged
tarik02 merged 2 commits into
mainfrom
fix/context-menus-appearance
Jul 29, 2026
Merged

tarik02 merged 2 commits into
mainfrom
fix/context-menus-appearance

Conversation

@tarik02

@tarik02 tarik02 commented Jul 29, 2026 •

Copy link
Copy Markdown
Collaborator

Problem

The Context menus preference was grouped under General even though it controls visual presentation.

Fix

Moved the setting to Appearance after Glass opacity. Added a user guide for the new path and each menu style, and updated the fork note.

Verification

  • vp check
  • vp run typecheck
  • targeted Markdown formatting check

Model: GPT-5
Harness: Codex

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:M labels Jul 29, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 455c06c881

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread apps/web/src/components/settings/SettingsPanels.tsx
@tarik02 tarik02 changed the title move context menus setting to appearance fix(web): move context menus setting to appearance Jul 29, 2026
@tarik02
tarik02 merged commit 82bca4a into main Jul 29, 2026
10 checks passed
@tarik02
tarik02 deleted the fix/context-menus-appearance branch July 29, 2026 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant