Rework foss filtering#4696
Conversation
Remove the separate "Highlight FOSS" toggle and its theming logic, and instead append an inline "- FOSS" label to FOSS app entries. Changes: remove WPFToggleFOSSHighlight from app navigation and related event handlers, delete FOSSColor resource updates in theme code and button handler, simplify checkbox setup in Invoke-WPFUIElements by removing the special-case toggle logic, and update Initialize-InstallAppEntry to build a horizontal panel containing the app name plus a small green "- FOSS" TextBlock for FOSS items. This consolidates FOSS indication into the item UI and cleans up toggle/theme handling.
Replace the separate StackPanel/TextBlock used for the FOSS indicator with an inline System.Windows.Documents.Run appended to the app name. The run is styled (green RGB(76,175,80) and FontSize=10) and the checkbox content assignment was moved outside the conditional so the appName (with optional FOSS run) is always used. Cleaned up related comments to reflect the FOSS label change.
|
looks kinda ugly |
|
i think its better but if you have any ideas... |
my idea would be to not change it and add a thing to explain what foss is |
I tried icons theyre just hard to understand |
no like under the foss toggle it should explain what foss is |
I think if you understand what open source is then you know what foss is as well |
some people don't know what foss is |
|
the main point is i dont think it needs a toggle ; ) |
but this looks alot worse than it is now |
github-actions
Bot
added
bug
most foss software already mentions that it is open source in its description |
it still looks ugly compare to the current version |
i dont this so #4414 (comment) |
the green color doesnt indicate that its foss |
just add text under foss toggle explaining what it is |
|
maybe just a small green dot like an indicator and a small few words explanation in the bottom of side panel , give this a try ig |
the people who care about it will notice it and people who dont can ignore it better |
AI GENERATED FOR AN EASY PREVIEW |
3 participants
Type of Change
Description
Removed the separate FOSS toggle and replaced it with an inline FOSS label.