Skip to content

fix(editor): normalize documentation and harden editor utilities #182

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
VPDPersonal merged 1 commit into from
Sep 8, 2026
Merged

fix(editor): normalize documentation and harden editor utilities #182

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
178 changes: 77 additions & 101 deletions ...tTools/Packages/tech.aspid.fasttools/Editor/Resources/UI/Aspid-FastTools-Default-Dark.uss
View file
Open in desktop

Large diffs are not rendered by default.

7 changes: 1 addition & 6 deletions ....fasttools/Editor/Resources/UI/Components/Aspid-FastTools-AspidAnimatedDotsBackground.uss
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,6 @@
--aspid-fasttools-colors-dot_blob-color_2: var(--aspid-colors-status-warning-text-dark);
--aspid-fasttools-colors-dot_blob-color_3: var(--aspid-colors-status-error-text-dark);

/* Status washes: dim, desaturated hues a status class paints across all three blobs at once, so the canvas
reads as one calm state wash instead of the default good→bad gradient. Component-scoped — the palette's
status families are tuned for text and borders and read far too saturated behind a whole window. */
--aspid-fasttools-colors-dot_blob-info: rgb(38, 54, 77);
--aspid-fasttools-colors-dot_blob-success: rgb(41, 71, 54);
--aspid-fasttools-colors-dot_blob-warning: rgb(77, 64, 31);
Expand All @@ -22,9 +19,7 @@
--aspid-fasttools-metrics-dot_scale_reference: 420;
}

/* Declared after :root deliberately — a status class and :root carry the same specificity, so source order is what
lets a status win over the default gradient. Dropping the class falls back to :root, which is how StatusStyle.Type.None
restores the three-tone signal look. */
/* Status rules must follow :root to win at equal specificity and restore the default palette when removed. */
.aspid-fasttools-status--info {
--aspid-fasttools-colors-dot_blob-color_1: var(--aspid-fasttools-colors-dot_blob-info);
--aspid-fasttools-colors-dot_blob-color_2: var(--aspid-fasttools-colors-dot_blob-info);
Expand Down
2 changes: 1 addition & 1 deletion ...Packages/tech.aspid.fasttools/Editor/Resources/UI/Components/Aspid-FastTools-AspidBox.uss
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
:root {
flex-grow: 1;
}
}
10 changes: 3 additions & 7 deletions ...ages/tech.aspid.fasttools/Editor/Resources/UI/Components/Aspid-FastTools-AspidHelpBox.uss
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,6 @@
border-color: var(--aspid-colors-status-info-shade-lightness);
}

/* --- ICON --- */
.aspid-fasttools-help-box__icon {
min-width: 34px;
max-width: 34px;
Expand Down Expand Up @@ -57,13 +56,10 @@
--aspid-fasttools-icons-help_box: resource("d_console.erroricon@2x");
}

/* --- TEXT --- */
.aspid-fasttools-help-box__text-container {
flex-grow: 1;
}

/* Small gap below the title (and its dividing line) so the message is not glued to the line above;
only present when a title is set, so title-less help boxes keep the message centered on the icon. */
.aspid-fasttools-help-box__title {
margin-bottom: 4px;
}
Expand All @@ -73,18 +69,18 @@
overflow: hidden;
}

:root.aspid-fasttools-status--success AspidLabel > Label {
:root.aspid-fasttools-status--success AspidLabel > Label {
color: var(--aspid-colors-status-success-text-lightness);
}

:root.aspid-fasttools-status--warning AspidLabel > Label {
:root.aspid-fasttools-status--warning AspidLabel > Label {
color: var(--aspid-colors-status-warning-text-lightness);
}

:root.aspid-fasttools-status--error AspidLabel > Label {
color: var(--aspid-colors-status-error-text-lightness);
}

:root.aspid-fasttools-status--info AspidLabel > Label {
:root.aspid-fasttools-status--info AspidLabel > Label {
color: var(--aspid-colors-status-info-text-lightness);
}
3 changes: 0 additions & 3 deletions ...ckages/tech.aspid.fasttools/Editor/Resources/UI/Components/Aspid-FastTools-AspidLabel.uss
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@ AspidDividingLine {
margin: 5px 0 0 0;
}

/* --- LABEL SIZES --- */
.aspid-fasttools-label-size--h1 {
font-size: 36px;
}
Expand Down Expand Up @@ -41,7 +40,6 @@ AspidDividingLine {
font-size: 12px;
}

/* General Text */
.aspid-fasttools-theme--darkness Label {
color: var(--aspid-colors-text-darkness);
}
Expand All @@ -58,7 +56,6 @@ AspidDividingLine {
color: var(--aspid-colors-text-lightness);
}

/* Status Text */
AspidLabel.aspid-fasttools-status--success.aspid-fasttools-theme--darkness > Label {
color: var(--aspid-colors-status-success-text-darkness);
}
Expand Down
34 changes: 1 addition & 33 deletions ...s/tech.aspid.fasttools/Editor/Resources/UI/Components/Aspid-FastTools-InspectorNotice.uss
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,13 +1,8 @@
/* --- Inspector notice ---
Compact single-row notice shared by the [SerializeReference] and [TypeSelector] drawers (missing type, required,
shared reference): a small warning icon, a terse yellow message and a bold, right-aligned clickable action word that
lightens on hover. The full explanation rides the element's tooltip on hover. The palette variables come from the
Aspid theme sheet the element adds itself. */
.aspid-fasttools-inspector-notice {
flex-direction: row;
align-items: center;
flex-wrap: wrap;
/* No left inset of its own — the foldout's left margin already clears the stripe gutter for the whole content. */

margin: 2px 0 1px 2px;
}

Expand All @@ -22,25 +17,16 @@
background-image: resource("d_console.warnicon");
}

/* Both states use the same yellow text per the design — warning, not info. */
.aspid-fasttools-inspector-notice__message {
color: var(--aspid-colors-status-warning-text-light);
-unity-font-style: normal;
white-space: normal;
}

/* The shared notice's message is itself clickable — it reveals the other members of the group (scroll + pulse) — so
it carries the link cursor. Its hover lighten comes from code, since the rid colour is dynamic; no underline — that
affordance is reserved for the action words, so the message reads as a label that happens to navigate. */
.aspid-fasttools-inspector-notice__message--navigable {
cursor: link;
}

/* The clickable action word (Fix / Make unique): bold, pinned to the right of the row — margin-left:auto eats the free
space so it reads as a deliberate affordance rather than a word trailing the message — underlined (a rich-text <u>
set from code, since USS has no text-decoration) so "this is a button" is unambiguous, and it lightens on hover.
Missing type keeps the warning amber here; the shared notice overrides the colour inline with its per-rid colour
(and lightens on hover from code, since that colour is dynamic). */
.aspid-fasttools-inspector-notice__action {
margin-left: auto;
color: var(--aspid-colors-status-warning-text-light);
Expand All @@ -52,9 +38,6 @@
color: var(--aspid-colors-status-warning-text-lightness);
}

/* Trailing Smart Fix suggestion ("· → Pistol?"): a second clickable word riding just after Fix (a 6px gap, so the two
actions cluster together at the right edge), same warning palette, underlined and lightening on hover like Fix.
Hidden by default; the --visible modifier reveals it when a suggestion is set. */
.aspid-fasttools-inspector-notice__suggestion {
margin-left: 6px;
color: var(--aspid-colors-status-warning-text-light);
Expand All @@ -67,8 +50,6 @@
display: flex;
}

/* The "·" between Fix and the suggestion: decoration, not an action — same palette and weight as its neighbours,
but never underlined, no link cursor, no hover (an underlined separator would read as part of the button). */
.aspid-fasttools-inspector-notice__suggestion-separator {
margin-left: 6px;
color: var(--aspid-colors-status-warning-text-light);
Expand All @@ -84,10 +65,6 @@
color: var(--aspid-colors-status-warning-text-lightness);
}

/* Leading rid swatch: a small colour-coded circle at the head of the shared row. It shares its per-rid colour inline
with the message text and the field's left stripe, so the whole field reads in one colour and aliased fields match at
a glance by their lined-up swatches. Its colour is set inline from code (unique per reference); hidden by default,
the --visible modifier reveals it when a colour is supplied (the shared notice). */
.aspid-fasttools-inspector-notice__dot {
width: 8px;
height: 8px;
Expand All @@ -101,20 +78,11 @@
display: flex;
}

/* --- Shared-reference variant ---
A shared reference is not an error but a "these two fields point at the same object" state, so it drops the warning
amber for a calm treatment carried by the reference's per-rid colour: the leading swatch, the message text, the
Make-unique action and the field's left stripe are all tinted that one colour (set inline from code), with no icon.
The Make-unique action keeps the base action treatment (bold, pinned right, underlined, lightens on hover) — only
its colour differs: the inline rid colour (its hover lighten comes from code, since that colour is dynamic). */
.aspid-fasttools-inspector-notice--shared
.aspid-fasttools-inspector-notice__icon {
display: none;
}

/* Info variant — a non-actionable, dim hint (the multi-object "different types" notice that stands in for the
suppressed per-instance child fields). Swaps the warning icon for the info icon and the yellow text for the
generic dim shade, so it reads as informational rather than something to fix. */
.aspid-fasttools-inspector-notice--info
.aspid-fasttools-inspector-notice__icon {
background-image: resource("d_console.infoicon");
Expand Down
9 changes: 0 additions & 9 deletions ...ages/tech.aspid.fasttools/Editor/Resources/UI/Components/Aspid-FastTools-WindowFooter.uss
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,13 +1,9 @@
/* Shared bottom bar for Aspid FastTools editor windows: a faded dividing line above a version/GitHub row.
Kept transparent so a host window's dotted canvas reads continuously behind it. The row mirrors the old
Welcome-footer inset (10px sides) and the block lifts the text off the window's bottom edge. */
.aspid-fasttools-window-footer {
flex-shrink: 0;
margin-top: 2px;
padding-bottom: 8px;
}

/* The divider is faded so it reads as a hairline separator rather than a hard rule. */
.aspid-fasttools-window-footer AspidDividingLine {
opacity: 0.5;
}
Expand All @@ -20,7 +16,6 @@
padding-right: 10px;
}

/* Version: dim grey at rest, brightening on hover to signal it links to the tagged release. */
.aspid-fasttools-window-footer__version {
color: var(--aspid-colors-text-darkness);
font-size: 14px;
Expand All @@ -30,9 +25,6 @@
color: var(--aspid-colors-text-light);
}

/* The keyboard-ring key, absolutely centred over the whole row (independent of the side links' widths) in the
quietest text tone — present for discoverability, but quieter than the version so it never competes. Click-
transparent from code so the links keep their hits. */
.aspid-fasttools-window-footer__keys {
position: absolute;
left: 0;
Expand All @@ -44,7 +36,6 @@
-unity-text-align: middle-center;
}

/* GitHub link: the signature green, brightening on hover. */
.aspid-fasttools-window-footer__link {
color: var(--aspid-colors-status-success-text-dark);
font-size: 14px;
Expand Down
13 changes: 3 additions & 10 deletions ...ls/Packages/tech.aspid.fasttools/Editor/Resources/UI/Enums/Aspid-FastTools-EnumValues.uss
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,25 +1,20 @@
/* Negative right margin cancels the inspector's default field padding, aligning the card's
right edge with sibling fields. */
/* The negative margin cancels the inspector's field padding. */
:root {
margin: 1px -2px 1px 3px;
}

/* Zeroes Unity's built-in field margin, which otherwise double-counts against this drawer's
own padding and pushes nested fields inward. */
/* Cancel Unity's field margins to avoid double indentation inside the drawer. */
.unity-property-field__inspector-property .unity-base-field__inspector-field {
margin-left: 0;
margin-right: 0;
}

/* Toggle carries an extra 1px inset Unity bakes into its own style — nudge it back to match
the other fields. */
/* Toggle adds a built-in one-pixel inset beyond the other fields. */
.unity-property-field__inspector-property
Toggle.unity-base-field__inspector-field {
margin-left: -1px;
}

/* Header and container are separate elements styled to read as one rounded card — top corners
on the header, bottom on the container, same border color to hide the seam. */
.aspid-fasttools-enum-values__header {
padding: 5px 5px 2px 5px;
border-width: 1px;
Expand All @@ -36,8 +31,6 @@
background-color: var(--aspid-colors-bg-lightness);
}

/* Single-line values sit next to the key in two equal columns (mirrors the IMGUI layout);
values with a foldout keep the default column flow — key line, then a "Value" foldout. */
.aspid-fasttools-enum-value--inline {
flex-direction: row;
}
Expand Down
Loading
Loading