Skip to content

Conversation

@angiejones
Copy link
Collaborator

Summary

Moved goose branding from the homepage to the sidebar so that it's always visible.

Type of Change

  • Feature
  • Bug fix
  • Refactor / Code quality
  • Performance improvement
  • Documentation
  • Tests
  • Security fix
  • Build / Release
  • Design

AI Assistance

  • This PR was created or reviewed with AI assistance

Testing

  • ✅ TypeScript - passes
  • ✅ ESLint - passes
  • ✅ Tests - 289 passed, 4 failed (pre-existing failures unrelated to my changes)

Screenshots/Demos (for UX changes)

Before:

image

After:

image image

Copilot AI review requested due to automatic review settings January 18, 2026 18:57
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR moves the goose branding from the homepage to the sidebar footer, making it persistently visible across all views.

Changes:

  • Removed goose icon from SessionsInsights homepage header
  • Added goose branding (icon + text) to the sidebar footer alongside the EnvironmentBadge

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
ui/desktop/src/components/sessions/SessionsInsights.tsx Removed Goose icon and associated container from the header section in both the skeleton loader and main render
ui/desktop/src/components/GooseSidebar/AppSidebar.tsx Added Goose icon and "goose" text label to the sidebar footer, positioned alongside EnvironmentBadge

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings January 18, 2026 19:03
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

Copy link
Collaborator

@The-Best-Codes The-Best-Codes left a comment

Choose a reason for hiding this comment

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

There's a small weird flicker when I open the sidebar via the sidebar button (when the sidebar is a sheet, e.g., when the window isn't very wide), but it's barely noticeable.

@angiejones
Copy link
Collaborator Author

There's a small weird flicker when I open the sidebar via the sidebar button (when the sidebar is a sheet, e.g., when the window isn't very wide), but it's barely noticeable.

ugh i dont want a flicker lol. it's not the fly in animation? something else? feel like making a lil video?

@The-Best-Codes
Copy link
Collaborator

The-Best-Codes commented Jan 18, 2026

I just made a screen recording. I think it's a scrollbar. It probably was an issue before this PR anyway... but still makes a little white flicker above the goose for a second when opening the sidebar.

Screencast_20260118_133331.webm

Freeze-frame from the video:
image

It's a VERY minor nitpick lol
Since it's likely not caused by this PR it might be a good fix for a different PR?

@angiejones
Copy link
Collaborator Author

I just made a screen recording. I think it's a scrollbar. It probably was an issue before this PR anyway... but still makes a little white flicker above the goose for a second when opening the sidebar.

Screencast_20260118_133331.webm
Freeze-frame from the video: image

It's a VERY minor nitpick lol

ah yes thats goose flying in! it does that on the home screen today. lol shall i take it out?

@The-Best-Codes
Copy link
Collaborator

No I like the fly in animation! I'm talking about this:
image

Which is a scrollbar appearing for some reason while the sidebar opens, and it causes a white flash above the goose. Unless that's supposed to happen?

@The-Best-Codes
Copy link
Collaborator

The-Best-Codes commented Jan 18, 2026

Here's a demo of it happening on the main branch (not this PR):

Screencast_20260118_134041.webm

Kind of hard to tell in the recording, when I test locally it's more obvious (especially now that it's right above the goose). But yeah very minor 😆

@angiejones
Copy link
Collaborator Author

Here's a demo of it happening on the main branch (not this PR):

Screencast_20260118_134041.webm

whew ok so not me. thanks so much for checking! ❤️

@angiejones angiejones merged commit eeefe3e into main Jan 18, 2026
26 checks passed
@angiejones angiejones deleted the goose-logo branch January 18, 2026 19:42
fbalicchia pushed a commit to fbalicchia/goose that referenced this pull request Jan 23, 2026
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: fbalicchia <fbalicchia@cuebiq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants