Skip to content

Conversation

@ryanlua
Copy link
Owner

@ryanlua ryanlua commented Jan 13, 2026

Use the backpack vector icon from the builder icons font for the icon.

image

This pull request updates the appearance and configuration of the topbar inventory icon in src/TopbarIcon.client.luau. The main focus is on improving the icon's label, font, and layout for better clarity and consistency.

UI and Label Improvements:

  • Replaced the previous image-based icon with a text label ("backpack") and updated the label styling, including font, size, and weight for both selected and deselected states.
  • Forced a minimum width for the icon label container to ensure consistent layout.

Configuration Adjustments:

  • Set the icon's order, toggle key, and auto-deselect behavior, and restored the "Inventory" caption.

Signed-off-by: Ryan Luu <ryan.luu@ryanluu.dev>
Signed-off-by: GitHub <noreply@github.com>
Signed-off-by: GitHub <noreply@github.com>
Signed-off-by: GitHub <noreply@github.com>
@ryanlua ryanlua self-assigned this Jan 13, 2026
Copilot AI review requested due to automatic review settings January 13, 2026 08:16
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 pull request modernizes the topbar inventory icon by replacing image-based rendering with a font-based icon from the BuilderIcons library. The change improves consistency and maintainability by leveraging vector icons instead of static images.

Changes:

  • Replaced image-based icon (ICON_SELECTED_IMAGE and ICON_DESELECTED_IMAGE) with font-based "backpack" label using BuilderIcons
  • Configured separate font weights (Bold for selected, Regular for deselected) for visual state differentiation
  • Added minimum width constraint to ensure consistent icon layout

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ryanlua ryanlua merged commit ce6e79b into main Jan 13, 2026
10 checks passed
@ryanlua ryanlua deleted the font-topbar-icon branch January 13, 2026 08:20
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.

1 participant