Skip to content
This repository has been archived by the owner on Sep 30, 2024. It is now read-only.

cody: fix light theme issue #49881

Merged
merged 7 commits into from
Mar 23, 2023
Merged

cody: fix light theme issue #49881

merged 7 commits into from
Mar 23, 2023

Conversation

abeatrix
Copy link
Contributor

@abeatrix abeatrix commented Mar 22, 2023

1. Fix light theme issue

RE issue reported by user 1 issue reported by user 2

Issue:
image

We have been using --vscode-background and --vscode-foreground as the default color, which is different than the default color theme for the sidebar. Replace these values with --vscode-sideBar-background and --vscode-sideBar-foreground fixed the issue.

2. Add button to copy code block text

RE feature requested by user

Added buttons to each code block:

cody_copy

Screenshots
image
image
Screenshot 2023-03-22 at 3 24 43 PM

Test plan

Tested on multiple light themes and dark themes:

Screenshot 2023-03-23 at 7 26 47 AM

Screenshot 2023-03-23 at 7 27 00 AM

Screenshot 2023-03-23 at 7 27 31 AM

Screenshot 2023-03-23 at 7 27 22 AM

App preview:

Check out the client app preview documentation to learn more.

@cla-bot cla-bot bot added the cla-signed label Mar 22, 2023
@github-advanced-security
Copy link

You have successfully added a new SonarCloud configuration ``. As part of the setup process, we have scanned this repository and found no existing alerts. In the future, you will see all code scanning alerts on the repository Security tab.

@abeatrix abeatrix marked this pull request as ready for review March 23, 2023 15:31
@abeatrix abeatrix changed the title Draft cody: fix light theme issue and add buttons to copy text cody: fix light theme issue and add buttons to copy text Mar 23, 2023
@abeatrix abeatrix self-assigned this Mar 23, 2023
@abeatrix abeatrix requested a review from novoselrok March 23, 2023 16:22
@abeatrix abeatrix changed the title cody: fix light theme issue and add buttons to copy text cody: fix light theme issue Mar 23, 2023
@abeatrix abeatrix merged commit ef0ef64 into cody-main Mar 23, 2023
@abeatrix abeatrix deleted the bee/cody-login branch March 23, 2023 16:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants