Skip to content

Conversation

@cx-rahul-pidde
Copy link
Contributor

@cx-rahul-pidde cx-rahul-pidde commented Nov 3, 2025

By submitting a PR to this repository, you agree to the terms within the Checkmarx Code of Conduct. Please see the contributing guidelines for how to create and submit a high-quality PR for this repo.

Description

This PR addresses visual inconsistencies between light and dark themes across multiple sections of the VS Code extension UI. The update ensures that all interactive elements, tooltips, icons, and info texts adapt to the active theme (data-vscode-theme-kind).

References

Include supporting link to GitHub Issue/PR number

Testing

Test the VS Code plugin UI across the following themes:

  • Light
  • Dark
  • High Contrast
  • High Contrast Light

Checklist

  • I have added documentation for new/changed functionality in this PR (if applicable).
  • All active GitHub checks for tests, formatting, and security are passing
  • The correct base branch is being used

@github-actions
Copy link
Contributor

github-actions bot commented Nov 3, 2025

Logo
Checkmarx One – Scan Summary & Details0cba0fdd-18da-4fd1-a7c4-ea8d8b8ec44a

Great job! No new security vulnerabilities introduced in this pull request

@cx-rahul-pidde cx-rahul-pidde changed the title Other/ast 115631 Support light theme and minor UI Fixes (AST-115631) Nov 3, 2025
cx-anurag-dalke
cx-anurag-dalke previously approved these changes Nov 6, 2025
Copy link
Collaborator

@cx-anurag-dalke cx-anurag-dalke left a comment

Choose a reason for hiding this comment

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

ok

Copy link
Collaborator

@cx-anurag-dalke cx-anurag-dalke left a comment

Choose a reason for hiding this comment

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

ok

@cx-anurag-dalke cx-anurag-dalke enabled auto-merge (squash) November 6, 2025 10:14
@cx-anurag-dalke cx-anurag-dalke merged commit 1cebf9e into main Nov 6, 2025
7 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.

2 participants