Skip to content

Conversation

@SuperKali
Copy link
Member

Summary

This PR refactors the custom image button visibility logic to create a cleaner UI flow by making the manufacturer selection and custom image paths mutually exclusive. When a manufacturer is selected, the custom image button is now hidden, and reappears when the manufacturer is deselected.

Motivation

The current UI shows both selection paths simultaneously, which can lead to user confusion and conflicting states. This change improves the user experience by:

  • Clarifying the two distinct workflows: Standard path (Manufacturer → Board → OS Image) vs. Custom path (Direct custom image selection)
  • Preventing selection conflicts: Users can't have both a manufacturer selected and use a custom image simultaneously
  • Preparing for future features: This refactor lays the groundwork for the upcoming board pre-setup configuration phase

Changes

  • Modified: src/components/layout/HomePage.tsx
    • Wrapped custom image button section in conditional render (!selectedManufacturer)
    • Button now only shows when no manufacturer is selected

UI Behavior

Before

  • Custom image button always visible, regardless of manufacturer selection state
  • Potential confusion about interaction between the two selection paths

After

  • Initial state: Custom image button visible
  • Manufacturer selected: Custom image button hidden
  • Manufacturer deselected: Custom image button reappears
  • Custom image selected: Button remains visible (since selectedManufacturer is null)

Screenshots

image

… selected

Improve UI flow by making the custom image button mutually exclusive with the
manufacturer selection path. This creates a clearer distinction between the two
image selection workflows:

- Standard path: Manufacturer → Board → OS Image
- Custom path: Direct custom image selection

The custom image button is now hidden when a manufacturer is selected, and
reappears when the manufacturer is deselected. This change prepares the UI for
future integration of board pre-setup configuration phases.

This modification ensures better user experience by preventing conflicting
selection states and provides a cleaner interface for the upcoming board
pre-setup features planned for future releases.
@SuperKali SuperKali added enhancement New feature or request component: ui User interface related refactor Code refactoring without functional changes labels Jan 20, 2026
Copy link
Member

@igorpecovnik igorpecovnik left a comment

Choose a reason for hiding this comment

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

nice one

@github-actions
Copy link

🧪 Test Builds

Version: 0.0.0-pr.85 | PR: #85 | Status: ready for testing

Platform Download
Linux x64 📦 .deb / .AppImage
Windows x64 📦 .exe
macOS ARM64 📦 .dmg
ℹ️ About these builds
  • 🔓 Public downloads via nightly.link (no GitHub login required)
  • ⚠️ Unsigned builds for testing purposes only
  • Expires in 7 days
  • 🔄 Updated on every push to this PR

@github-actions github-actions bot added the status: ready for review Ready to be reviewed label Jan 20, 2026
@SuperKali SuperKali merged commit 7bb07e7 into main Jan 20, 2026
8 checks passed
@SuperKali SuperKali deleted the feature/conditional-custom-image-ui branch January 20, 2026 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: ui User interface related enhancement New feature or request refactor Code refactoring without functional changes status: ready for review Ready to be reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants