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

fix(svelte): Update cody web#63710

Merged
fkling merged 1 commit into
mainfrom
fkling/sk/update-cody-web
Jul 9, 2024
Merged

fix(svelte): Update cody web#63710
fkling merged 1 commit into
mainfrom
fkling/sk/update-cody-web

Conversation

@fkling

@fkling fkling commented Jul 9, 2024

Copy link
Copy Markdown
Contributor

This commit updates cody web, making similar changes as ed493c9 and a81d5e7. I wasn't quite sure about some styles, but I left comments on the respective PRs and tried to at match the styling.

Most importantly this also fixes cody web in the Svelte app which currently appears to be broken. I'd like to get this in as quickly as possible (hence stampers), styles can always be tweaked later.

Additional changes:

  • Fixed layout (chat container was overflowing).
  • Set font family and size to the same we use in the Svelte app for consistency.

Test plan

Manual testing.

This commit updates cody web, making similar changes as ed493c9 and
a81d5e7. I wasn't quite sure about some styles, but I left comments on
the respective PRs and tried to at match the styling.

Most importantly this also fixes cody web in the Svelte app which
currently appears to be broken.

Additional changes:

- Fixed layout (chat container was overflowing).
- Set font family and size to the same we use in the Svelte app for
  consistency.
@fkling fkling requested review from a team July 9, 2024 09:09
@fkling fkling self-assigned this Jul 9, 2024
@cla-bot cla-bot Bot added the cla-signed label Jul 9, 2024
@fkling fkling enabled auto-merge (squash) July 9, 2024 09:11
@fkling fkling merged commit ec77521 into main Jul 9, 2024
@fkling fkling deleted the fkling/sk/update-cody-web branch July 9, 2024 09:17
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