docs: recapture README screenshots against the Macs-era UI - #96
Merged
Conversation
Every annotated screenshot predated the July/August UI work: the Device tab rename to Macs (with Sync and Add by Address), battery levels in the menu and peripheral rows, the uniform action pills, the keyboard-shortcut recorders, and the display-connect list. Recapture all of them, add the previously unillustrated Add a Mac by Address sheet, and update the annotation script's coordinates and legends to match (device-tab becomes macs-tab; sections without a raw capture are now skipped instead of aborting). README captions follow: battery levels in the hero and Peripheral tab, the new sheet figure in Setup, and an Other-tab caption that matches the scrolled capture. The four Pairing screenshots are still the old captures.
The Other tab outgrew the window, so one scrolled capture hid either the top or the bottom. Stitch three scrolled captures into a single full-height raw — slices cut on flat background between sections, aligned by crop offset, overlay scrollbar painted out — and re-badge all six affordances. Revert peripheral-type-picker.png to the previous capture: menu-item icons only render on macOS 26, and the recapture came from a Sequoia Mac, so its picker menu was text-only.
…w itself - menu.png previously showed every row greyed out (peer unreachable) — the state Troubleshooting teaches as "can't switch" — under a caption inviting clicks. Recaptured with the other Mac online; legend background re-matched to the new capture's material. - The Other tab's badge 5 pointed at the marked display's trash button while its legend line describes the row; aim it at the row title. - docs/assets/raw/ is local input to annotate-screenshots.sh — gitignore it so stray git add -A can't commit raw captures.
…ge (#95) (#97) * docs: pair the Macs before connecting them — reorder Setup steps 3 and 4 (#95) An unpaired Mac refuses incoming connections outright, so Add by Address can only ever time out until both sides hold the pairing key — yet Setup walked through the Macs tab (including Add by Address) before the Pairing tab. Pairing needs no connectivity, so pairing-first is safe in every flow. Say so explicitly in both sections, and document the one endpoint staleness case the INTRODUCE exchange can't heal: both Macs changing IP at once on a Bonjour-blocked network. * fix: name the likely causes when Add by Address can't reach the peer (#95) An unpaired peer cancels the connection before the handshake, which the dialing side can't tell apart from a firewall drop or a dead host — the sheet just said the other Mac "didn't respond", steering users at the network when the actual fix is usually pairing first. Give the four unreachable-shaped failures one message naming all three causes; auth failures keep their precise wording. * fix: cover every unreachable-shaped Add by Address failure (#95) - A peer reset can land while the handshake nonce send is in flight (handshakeFailed(.sendFailed)), and a non-Magic-Switch service answering on the port yields framingFailed/frameTooLarge — the same unreachable shape, previously falling through to the generic secure-connection message. Fold all three into the lumped case. - Point at Settings → Pairing in that message, matching .notPaired. - Dialing before this Mac's own listener is bound reported the local condition as a peer-shaped connection failure; fail early with a dedicated .listenerNotReady message instead. - README: an unpaired Mac refuses the connection immediately, so say the add fails rather than times out.
|
🎉 This PR is included in version 2.25.3 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
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.
Every annotated README screenshot predated the July/August UI work (last captured June 8 in #52). Recaptured against the current app:
macs-tab.png(replacesdevice-tab.png) — the renamed Macs tab: Ping, Sync (was "Share"), Remove, the new + Add by Address (feat: Bonjour-less peer discovery — INTRODUCE exchange, stable port, Add by Address, advertising diagnostics #92), Refresh.add-by-address.png(new) — the Add by Address sheet, described in Setup but never pictured. Unannotated, like the pairing dialogs.other-tab.png— the tab outgrew the window, so the raw is stitched from three scrolled captures into one full-height image, all six affordances badged.peripheral-tab.png,menu.png— battery levels (feat: battery levels and VoiceOver support in the peripheral rows #71/fix: publish peripheral battery levels so the Peripherals tab updates live #77) and the uniform action pills (style: uniform fixed-width action pills in the Peripherals tab #75/fix: size the peripheral action pill to its resting titles #80/style: two points of headroom on the peripheral action pill floor #86); the menu legend gains "% = its battery".peripheral-type-picker.png— deliberately kept the older capture: menu-item icons only render on macOS 26 Tahoe, and a Sequoia recapture loses them. It still shows the pre-rename "Device" tab strip.annotate-screenshots.sh: coordinates and legends reworked,device-tab→macs-tab, missing raws skip instead of aborting, the Other-tab stitch is scripted, and the menu legend's%is escaped from ImageMagick. README captions updated to match.Still stale: the four Pairing screenshots (old tab strip; the enter dialog predates the ghost-dash preview from #78/#85). Drop recaptures into
docs/assets/raw/(dialogs straight intodocs/assets/) and re-run the script.