feat: drop the monitor icon from Home's tablet rows - #656
Merged
Conversation
Every row carried the same glyph, so it distinguished nothing -- it just indented the names away from the column's left edge. Without it they line up with "about", "your tablets" and the paragraph above, giving the column one straight edge the whole way down. Entity's icon column collapses on its own when Icon is null, so removing the attribute is the whole change on that side. IconMonitor went with it: this was its only use, and a glyph nothing draws is the same dead weight as an unused key. The link / link-off icons beside each name stay. Those are not decoration -- they are how a row shows whether the tablet is currently detected. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
TheSevenPens
added a commit
that referenced
this pull request
Sep 4, 2026
Both lists put the same glyph in front of every row — a card outline on presets, a toy brick on plugins. Identical on every row, so neither distinguished anything; they just held a fixed column of decoration in front of each name and pushed the names off the left margin the headings and body text share. Home's tablet list lost its monitor icon for the same reason in #656. This brings the other two object lists in line, so an Entity row reads the same wherever it appears. IconCardBulletedOutline and IconToyBrickOutline each had exactly one consumer, so both are removed from Icons.axaml too — as IconMonitor was. Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Removes the monitor glyph from each row of Home's your tablets list.
The icon was the same for every row, so it distinguished nothing — it just put a fixed column of decoration in front of every tablet name and pushed the names off the left margin the rest of the page aligns to. Removing it lets the names start where every other heading and body line on Home starts.
This is one step of the ongoing de-boxing pass: the standing rule is that separation comes from type, spacing and at most a hairline, and that ornament repeating identically across every row is carrying no information.
Trivial diff — the glyph reference in
DashboardView.axaml, plus the now-unused icon path inIcons.axaml.