8000 [Backport 5.6.x] Fix Cody Web CSS by sourcegraph-release-bot · Pull Request #64373 · sourcegraph/sourcegraph-public-snapshot · GitHub
[go: up one dir, main page]

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

[Backport 5.6.x] Fix Cody Web CSS#64373

Merged
camdencheek merged 1 commit into5.6.xfrom
backport-64370-to-5.6.x
Aug 8, 2024
Merged

[Backport 5.6.x] Fix Cody Web CSS#64373
camdencheek merged 1 commit into5.6.xfrom
backport-64370-to-5.6.x

Conversation

@sourcegraph-release-bot
Copy link
Collaborator

closes: https://linear.app/sourcegraph/issue/SRCH-866/improve-cody-web-theming-for-consistency-with-rest-of-product

This PR updates the Cody Web CSS based on the changes requested by @rrhyne.

Few buttons implemented in Cody doesn't satisfy the design requirements and do not use the css variable so we couldn't update them easily but rather had to override the styles.

Test plan

Before:
CleanShot 2024-08-08 at 22 53 48@2x

After:
CleanShot 2024-08-08 at 22 54 12@2x

Changelog

closes:
https://linear.app/sourcegraph/issue/SRCH-866/improve-cody-web-theming-for-consistency-with-rest-of-product

This PR updates the Cody Web CSS based on the changes requested by
@rrhyne.

Few buttons implemented in Cody doesn't satisfy the design requirements
and do not use the css variable so we couldn't update them easily but
rather had to override the styles.

## Test plan

Before:
![CleanShot 2024-08-08 at 22 53
48@2x](https://github.com/user-attachments/assets/95e10b42-b740-4663-a761-69449ec62296)

After:
![CleanShot 2024-08-08 at 22 54
12@2x](https://github.com/user-attachments/assets/bffbd10a-c87b-4eca-9582-b23eebccb60e)

## Changelog

- Make Cody Web styles more accessible.

(cherry picked from commit 2dd38b3)
@camdencheek camdencheek enabled auto-merge (squash) August 8, 2024 19:52
@camdencheek camdencheek merged commit 02a0898 into 5.6.x Aug 8, 2024
@camdencheek camdencheek deleted the backport-64370-to-5.6.x branch August 8, 2024 20:02
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.

3 participants

0