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

feat(svelte): Add cody button to repo home page#63684

Merged
fkling merged 1 commit into
mainfrom
fkling-srch-691-add-cody-button-to-repo-root-page
Jul 9, 2024
Merged

feat(svelte): Add cody button to repo home page#63684
fkling merged 1 commit into
mainfrom
fkling-srch-691-add-cody-button-to-repo-root-page

Conversation

@fkling

@fkling fkling commented Jul 8, 2024

Copy link
Copy Markdown
Contributor

Based on #63677

Closes srch-691
Closes srch-695

This adds the cody button to the repo "home page" so that cody can be opened from there.
This commit also makes it so that the cody button is hidden when the sidebar opens (like in the React app) and it adds a fixed height to the various headers so that they align well with the sidebar header and don't change size when the cody button is hidden when the sidebar is open.

Test plan

Manual testing.

2024-07-08_11-41_1
2024-07-08_11-41

@fkling fkling requested a review from a team July 8, 2024 09:48
@cla-bot cla-bot Bot added the cla-signed label Jul 8, 2024
@fkling fkling force-pushed the fkling-srch-692-respect-cody-ignore-feature branch from 3756701 to 14d33ca Compare July 8, 2024 18:56
Base automatically changed from fkling-srch-692-respect-cody-ignore-feature to main July 9, 2024 07:40
This adds the cody button to the repe "home page" so that cody can be
opened from there.
This commit also adds a fixed height to the various headers so that they
align well with cody (and don't change size when the cody button is
hidden when the sidebar is open).
@fkling fkling force-pushed the fkling-srch-691-add-cody-button-to-repo-root-page branch from 241f8a3 to d00e959 Compare July 9, 2024 07:46
@fkling fkling enabled auto-merge (squash) July 9, 2024 07:46
@fkling fkling merged commit f007a73 into main Jul 9, 2024
@fkling fkling deleted the fkling-srch-691-add-cody-button-to-repo-root-page branch July 9, 2024 09:33
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