Skip to content

style: simplify WebUI sidebar#9313

Merged
Soulter merged 1 commit into
masterfrom
codex/remove-sidebar-branding
Jul 18, 2026
Merged

style: simplify WebUI sidebar#9313
Soulter merged 1 commit into
masterfrom
codex/remove-sidebar-branding

Conversation

@Soulter

@Soulter Soulter commented Jul 18, 2026

Copy link
Copy Markdown
Member

Summary

  • Remove the AstrBot WebUI branding block from the expanded sidebar.
  • Remove the divider above the Settings footer action.
  • Delete the unused branding styles.

Why

The branding block and footer divider add unnecessary visual chrome and consume sidebar space. Removing them keeps the navigation focused on its actions.

Impact

The expanded sidebar now starts directly with the navigation items, and the Settings footer no longer has a top divider. Rail mode behavior is unchanged.

Validation

  • pnpm typecheck
  • uv run ruff format --check .
  • uv run ruff check .
  • git diff --check

Summary by Sourcery

Simplify the vertical sidebar layout by removing unused branding elements and visual separators to focus on navigation content.

Enhancements:

  • Remove the AstrBot WebUI branding block from the expanded sidebar layout.
  • Eliminate the top border divider from the non-rail sidebar footer to reduce visual chrome.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request removes the brand logo and title header from the vertical sidebar, along with its associated CSS styles, and removes the top border from the sidebar footer. The reviewer suggested adjusting the footer's top padding to 16px to ensure symmetric and consistent spacing now that the border has been removed.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment on lines 235 to 238
.leftSidebar:not(.v-navigation-drawer--rail) .sidebar-footer {
align-items: stretch;
padding: 10px 16px 16px !important;
border-top: 1px solid rgba(var(--v-theme-borderLight), 0.35);
}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

Since the top border (border-top) of the footer has been removed, the top padding of 10px (which was likely reduced to account for the border's visual weight) can now be made symmetric with the other sides (16px). This will provide a more balanced and consistent spacing between the list items and the footer button.

.leftSidebar:not(.v-navigation-drawer--rail) .sidebar-footer {
  align-items: stretch;
  padding: 16px !important;
}

@Soulter
Soulter marked this pull request as ready for review July 18, 2026 10:31
@dosubot dosubot Bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Jul 18, 2026
@Soulter
Soulter merged commit 390c538 into master Jul 18, 2026
20 checks passed
@dosubot dosubot Bot added the area:webui The bug / feature is about webui(dashboard) of astrbot. label Jul 18, 2026

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:webui The bug / feature is about webui(dashboard) of astrbot. size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant