Conversation
Editor controls - add shared editor affordance buttons and action icon entries - move hover-expand color shifts into the shared theme utility - support icon-name overrides so widgets can swap icons w/o inlining svg Wiki - remove old legacy verbiage
Shared editor controls - add a shared card-action wrapper for on-card overflow buttons - move the clipped corner geometry into the editor component layer - keep default affordance buttons on rounded-square hover surfaces Services - replace the desktop card edit pencil with the shared overflow action - preserve service card classes during status refreshes instead of rebuilding the card class string - add a modal edit action for compact and mobile service flows - reserve the card corner so service text does not run under the action zone Reminders - switch reminder card edit actions to the shared card-action helper
Services cards - remove the desktop info icon from service cards - make the shared overflow action open the service info modal - keep status refreshes updating card state without rebuilding card classes Services modal - replace the bottom edit button with a right-aligned wrench in the modal header row - style the wrench with the shared affordance hover treatment - keep compact and desktop service editing reachable from the info modal Shared icons - add a wrench action icon for modal edit entrypoints
Metrics snapshot - remove the hidden widget-wide configure controls row - add the shared visible overflow affordance to the snapshot feature-header - scope metrics editing to the snapshot subwidget only Metrics styling - align the snapshot header affordance with the shared widget-level action treatment - leave history and events controls unchanged
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
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.
Summary
This updates edit and configuration affordances across editable widgets to use quieter shared controls with clearer hover and focus behavior. This refactor is inspired by the Pinned Pages on Firefox's start page, whose edit modals are accessible by a minimal
...icon on the app tile. This follows an initial style change in 0b705d6 which added better "app icons" to compact service cards.What changed
Shared affordances
...treatmentWiki
Keeps the document edit iconography, but reveal the wiki edit affordance from the content header or notes container using the new shared behavior.
Reminders
Replaces add and per-card edit controls with shared overflow affordances and moves card editing to the top-right corner.
Services
Replaces add and per-card desktop controls with shared overflow affordances and:
Metrics
Moves configure from the old widget-wide controls row into the snapshot feature header, since Metrics widget only has editing on the snapshot tiles anyway. Further UI config on Network widget or othe multi-subwidget widgets should use
...on subwidget feature-headers.Demo and docs
Updates editor demo and service demo text to match the new UI paths.
Validation
Refreshing browser page and using the app.