Skip to content

Replace the FOSS dot with a corner badge on app entries - #4924

Merged
ChrisTitusTech merged 2 commits into
ChrisTitusTech:mainfrom
MyDrift-user:feat/foss-corner-badge
Aug 6, 2026
Merged

Replace the FOSS dot with a corner badge on app entries#4924
ChrisTitusTech merged 2 commits into
ChrisTitusTech:mainfrom
MyDrift-user:feat/foss-corner-badge

Conversation

@MyDrift-user

@MyDrift-user MyDrift-user commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Type of Change

  • New feature
  • Bug fix
  • Documentation update
  • Refactor
  • UI/UX improvement

Description

This changes how FOSS applications are marked on the Install tab. Previously a small green dot was appended to the application name, which was not very clear to be such kind of indication on first sight

FOSS applications now carry a green triangle in the top right corner of their entry, with a keyhole on it. The badge is drawn as vector paths in a Viewbox, meaning scalable.

Since there is not really a standardized logo for "FOSS" and to avoid missunderstandings i refrained from using official logos like the one of the Open Source Initiative ® and tried to do something similar for recognitial purposes.

The legend in the left panel uses the same badge as a circle. The legend also moved below the buttons. It was rendered above them because entries are sorted by type rank and Note shared rank 0 with checkboxes; notes now sort last. The Order field in appnavigation.json is not read by the renderer, so no config change was needed.

Every app entry now wraps its checkbox in a layout Grid rather than only the FOSS ones, so the checkbox handlers reach the border the same way for every entry. This does not change applications.json or the foss flag it carries.

Comparison

Before After
winutil-foss-before-detail winutil-foss-after-detail
Green dot after the application name, legend above the buttons Corner badge on the entry, circle badge legend below the buttons

Verification

Verified with:

  • local compilation

Issue related to PR

N/A

- Add New-WinUtilFossBadge, the open source keyhole on a green backdrop
- Mark FOSS apps with a corner triangle instead of a dot after the name
- Give the FOSS legend a circle badge and move it below the buttons
- Sort Note entries last, they shared a rank with checkboxes before

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@github-actions github-actions Bot added the ui update UI/UX improvements label Aug 5, 2026
@coderabbitai

coderabbitai Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Summary by CodeRabbit

  • New Features
    • Added scalable FOSS badges for clearer identification of free and open-source applications.
    • Added accessible tooltips and flexible badge styles.
  • Bug Fixes
    • Improved checkbox interaction and app selection reliability.
    • Ensured interface notes consistently appear after buttons and selection controls.
  • Style
    • Standardized checkbox layouts and replaced inline FOSS markers with polished badge artwork.

Walkthrough

The PR adds a reusable WPF FOSS badge, replaces inline FOSS markers in notes and application entries, updates note sorting, and changes install-entry checkbox handling for the nested Grid layout.

Changes

FOSS badge and application entry UI

Layer / File(s) Summary
FOSS badge rendering
functions/private/New-WinUtilFossBadge.ps1, functions/public/Invoke-WPFUIElements.ps1
Adds scalable badge artwork. Notes use the badge instead of a Unicode bullet, and Note items receive explicit sort priority.
Install entry integration
functions/private/Initialize-InstallAppEntry.ps1
Wraps checkbox content in a Grid, adds optional corner badges, and resolves checkbox state through stored app keys and updated parent traversal.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Sequence Diagram(s)

sequenceDiagram
  participant InvokeWPFUIElements
  participant NewWinUtilFossBadge
  participant WPFNote
  InvokeWPFUIElements->>NewWinUtilFossBadge: Create rounded FOSS badge
  NewWinUtilFossBadge->>WPFNote: Return badge element
  InvokeWPFUIElements->>WPFNote: Render badge before note text
Loading

Possibly related PRs

Suggested labels: ui update

Suggested reviewers: christitustech

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly and concisely describes the primary UI change from a FOSS dot to a corner badge on app entries.
Description check ✅ Passed The description directly explains the FOSS badge redesign, legend placement, checkbox layout changes, and verification performed.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot added the new feature New feature or request label Aug 5, 2026

@coderabbitai coderabbitai 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.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@functions/private/Initialize-InstallAppEntry.ps1`:
- Around line 25-26: Use consistent variable casing in the checkbox toggle
logic: update the assignment target in Initialize-InstallAppEntry to use the
existing $childCheckbox variable defined immediately above, replacing the
mismatched $childCheckBox reference.
🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: e05b53fb-5ff4-4502-839c-f9605e0ca185

📥 Commits

Reviewing files that changed from the base of the PR and between f7c0723 and 155ccd5.

📒 Files selected for processing (3)
  • functions/private/Initialize-InstallAppEntry.ps1
  • functions/private/New-WinUtilFossBadge.ps1
  • functions/public/Invoke-WPFUIElements.ps1

Comment thread functions/private/Initialize-InstallAppEntry.ps1 Outdated
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
@coderabbitai coderabbitai Bot removed the new feature New feature or request label Aug 5, 2026
@vyas-devgna

Copy link
Copy Markdown
Contributor

This is really cool

@mewclouds mewclouds 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.

Oh yeah, this looks great. Loving the changes to the UI 😍

@ChrisTitusTech ChrisTitusTech left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Amazing Thanks Drift!

@ChrisTitusTech
ChrisTitusTech merged commit 8d3adb5 into ChrisTitusTech:main Aug 6, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ui update UI/UX improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants