Skip to content

Pe/shortcut bg color #5519

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

Closed
wants to merge 2 commits into from
Closed

Pe/shortcut bg color #5519

wants to merge 2 commits into from

Conversation

Patrick-Erichsen
Copy link
Collaborator

Description

Resolves #5507

Removes a hardcoded color on the shortcuts table with an appropriate theme based value.

@Patrick-Erichsen Patrick-Erichsen requested a review from a team as a code owner May 5, 2025 18:26
@Patrick-Erichsen Patrick-Erichsen removed the request for review from a team May 5, 2025 18:26
@Patrick-Erichsen Patrick-Erichsen requested a review from RomneyDa May 5, 2025 18:26
Copy link

netlify bot commented May 5, 2025

Deploy Preview for continuedev ready!

Name Link
🔨 Latest commit df12f57
🔍 Latest deploy log https://app.netlify.com/sites/continuedev/deploys/681eb37a82e9960008be383e
😎 Deploy Preview https://deploy-preview-5519--continuedev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

sestinj
sestinj previously approved these changes May 10, 2025
Copy link
Contributor

@sestinj sestinj left a comment

Choose a reason for hiding this comment

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

continue-sdk updates probably shouldn't be in here (maybe we can automate them), but otherwise this is very obviously good to go

@Patrick-Erichsen Patrick-Erichsen dismissed sestinj’s stale review May 10, 2025 00:08

The merge-base changed after approval.

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label May 10, 2025
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label May 10, 2025
@RomneyDa
Copy link
Collaborator

@Patrick-Erichsen looks like no file changes?

@github-project-automation github-project-automation bot moved this from Todo to Done in Issues and PRs May 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

idea unreadable black font an black back
3 participants