Repository navigation
Fix/context menu theme - #744
ronakmaheshwari wants to merge 8 commits into
Conversation
🦋 Changeset detectedLatest commit: 0932a15 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
@evenstensberg can u just take a look |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
WalkthroughThe context menu container now uses CSS variables for its background and border. Disabled context menu items now use the secondary text color and have reduced opacity. A changeset records the use of Webpack’s infrastructure logger when available and the deprecation of Priority: ⬇️ Low Merge Risk: ⚪ Minimal · up to
|
Context:
Follow-up to #690. The context menu still had hard-coded colors, and disabled items were low contrast in both themes
What changed:
This applies the same CSS changes that were already discussed and reviewed there based on the work done by @theEquinoxDev, so it should be easy to merge:
Summary by CodeRabbit
infrastructureLoggingconfiguration.logLevelis deprecated in favor of Webpack’sinfrastructureLoggingconfiguration.