Skip to content

Add set_oblique setter and finalise oblique projection docs - #94

Merged
bjmorgan merged 1 commit into
mainfrom
docs-oblique-view
Aug 17, 2026
Merged

Add set_oblique setter and finalise oblique projection docs#94
bjmorgan merged 1 commit into
mainfrom
docs-oblique-view

Conversation

@bjmorgan

Copy link
Copy Markdown
Owner

Adds a set_oblique(angle, foreshortening) setter so oblique is selected the same way as the other modes (set_orthographic, set_perspective): it sets an Oblique projection, returns self for chaining, and validates through the type.

Presents the three projection modes together in the rendering guide, with a three-panel perovskite comparison (orthographic and perspective on the shared view, oblique looking down [100] so the receding axis reads clearly). The changelog and examples now use set_oblique.

Also drops an unused import and a dead local in the docs image generator.

Give oblique a set_oblique(angle, foreshortening) setter alongside
set_orthographic and set_perspective, so every mode is selected the
same way; it sets an Oblique projection, returns self for chaining,
and validates through the type.

Present the three projection modes together in the rendering guide,
with a three-panel perovskite comparison (orthographic and perspective
on the shared view, oblique looking down [100] so the receding axis
reads clearly). The changelog and examples now use set_oblique.

Also drop an unused import and a dead local in the docs image
generator.
Copilot AI lite review requested due to automatic review settings August 17, 2026 15:06

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@bjmorgan
bjmorgan merged commit bb8c4c7 into main Aug 17, 2026
5 checks passed
@bjmorgan
bjmorgan deleted the docs-oblique-view branch August 17, 2026 15:11
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.

2 participants