Skip to content

Use active view for fallback zoom actions#4

Open
rishabhk571 wants to merge 1 commit into
QuisPic:masterfrom
rishabhk571:fix/active-view-fallback-zoom
Open

Use active view for fallback zoom actions#4
rishabhk571 wants to merge 1 commit into
QuisPic:masterfrom
rishabhk571:fix/active-view-fallback-zoom

Conversation

@rishabhk571

Copy link
Copy Markdown

Description

What this does:
Updates the fallback ExtendScript zoom payloads so they target the active viewer view instead of always using View 1.

Why it matters:
This fixes multi-view layouts where fallback zoom actions can affect views[0] even when another view is selected.

Changes

  • Replaces hardcoded app.activeViewer.views[0] usage in both fallback zoom payloads.
  • Reads activeViewer.activeViewIndex and validates it before selecting a view.
  • Falls back to index 0 only when the active view index is missing or invalid.
  • Returns early when there is no active viewer or no view list.
  • Bumps the plug-in source version to 0.3.2.

Validation

  • Confirmed the updated source uses activeViewer.activeViewIndex in both fallback payloads.
  • Confirmed the updated source no longer contains hardcoded fallback views[0] usage.
  • Compared the supplied old and new Windows plug-ins: the old binary contains two app.activeViewer.views[0] payloads, while the new binary contains two activeViewer.activeViewIndex payloads.
  • Release build was not rerun in this checkout because cmake and ninja are not available on PATH.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant