Add zoom keyboard shortcuts (Cmd+, Cmd-, Cmd+0)

[ssstonebraker]

Implements per-document zoom as requested in #335.

This PR adds standard macOS zoom shortcuts:

• Cmd+ zooms in (10% increments, max 300%)
• Cmd- zooms out (10% decrements, min 50%)
• Cmd+0 resets to actual size

Implementation:

• Uses a transient per-document zoom multiplier (not saved to preferences)
• Zoom applies to both editor and preview when the preference is enabled
• Menu items validate and disable at zoom limits
• Does not modify the base font preference
• Preserves existing Cmd+Shift+0 shortcut for split view

Fixes #335

[schuyler]

Thanks for tackling #335 — the overall design is clean. Transient per-document zoom via a multiplier (not mutating preferences) is the right call. A few things to address before this is ready to merge:

Bug: Preview zoom broken when preference is off

This is the main issue. scaleWebview has an early return when previewZoomRelativeToBaseFontSize is NO:

if (!self.preferences.previewZoomRelativeToBaseFontSize)
    return;

Because the zoomMultiplier is applied inside that method, Cmd+/Cmd- won't affect the preview at all when the preference is off. The zoom shortcuts should work on the preview regardless of that setting.

Duplicated zoom constants

kMaxZoom (3.0) and kMinZoom (0.5) are each defined as static const in three separate places — validateUserInterfaceItem:, zoomIn:, and zoomOut:. These should be file-level constants defined once to avoid the risk of them getting out of sync.

Floating-point comparison in resetZoom: validation

return self.zoomMultiplier != 1.0;

After several +0.1 / -0.1 round trips, IEEE 754 can drift — 1.0 + 0.1 - 0.1 may not exactly equal 1.0. This could leave the "Actual Size" menu item enabled at 100%. Something like fabs(self.zoomMultiplier - 1.0) > 0.001 would be safer.

[ssstonebraker]

Thanks for the review. Pushed a fix for all three.

Moved the zoom constants to file scope with the kMP prefix to match the rest of the codebase. Restructured scaleWebview so the zoom multiplier always applies to the preview regardless of the previewZoomRelativeToBaseFontSize setting. Fixed the floating-point comparison in resetZoom: validation.

Tested locally, everything works as expected.

[schuyler]

Thank you, @ssstonebraker — this PR was the foundation for the entire zoom effort, and it's appreciated. 🙏

The three fixes you pushed in response to review (file-scope zoom constants, preview zoom working regardless of the previewZoomRelativeToBaseFontSize setting, and the floating-point comparison in the reset-zoom validation) are already productized in #395, with your original commits cherry-picked and credited there.

We've now consolidated the design across all the zoom PRs into a single plan in #470, which unifies editor and preview zoom behind one ⌘+/⌘−/⌘0 key pair. Closing this as superseded — but the lineage traces straight back to your work here. Thanks again!

---

Generated by Claude Code