Repository navigation
Part of #758: .ai import doesn't weigh text objects when the layers can't be used - #869
Merged
Merged
Conversation
bflatastic
force-pushed
the
fix/758-skip-hopeless-prune
branch
from
October 10, 2026 03:14
4e86c54 to
56a45a5
Compare
…xt document Illustrator EPS and .ai type is made whole from the file's text document, shown or hidden: point type, area type (a story in several frames as threaded type that flows when the file opens) and type on a path, with its fonts, size, leading, tracking, scaling, baseline shift, fill and stroke colours (grey, RGB or CMYK), alignment, indents and paragraph spacing. The page's drawing of that type, often in pieces, is left out; where the text document can't say, type that shows still comes from the page. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The effect versions of Offset Path, Zig Zag and Roughen showed Joins and Points as free text fields. They now pick Miter, Round or Bevel and Smooth or Corner, as Object > Path > Offset Path does. A size's Relative / Absolute (Roughen, Zig Zag, Tweak) is a pair of radio buttons, and the effect dialogs' other on/off options are checkboxes that show when they are off too (they were invisible unchecked). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Raw app.quit closed the window and answered null at once, so a client
couldn't tell that a save prompt was waiting. It now runs the menu's
Quit: {"pending": "saveChanges"} while a modified document is asked
about, as file.close and file.closeAll answer and the protocol
documentation says; otherwise null, and the window closes.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Gaussian Blur and Feather drew through a filter layer whose blur works on a coarser grid tied to the canvas: the blurred art moved by up to about a pixel, and its steps showed, as the view moved by parts of a pixel (scrolling, zooming about the pointer). They now blur the content's own pixels on the CPU, as the Photoshop-style effects already do: the same look wherever the view is, cached while only the view moves. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…an't be used Before deciding to use the layers of an Illustrator file, the import drew them again for each text object made from the file's text document, to see whether the page draws it. When the layers differ from the page by more than the check allows even with every such object gone (taking one out only changes what is drawn in its box), the layers aren't used whatever is taken out: that weighing is now skipped. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
bflatastic
force-pushed
the
fix/758-skip-hopeless-prune
branch
from
October 10, 2026 03:29
56a45a5 to
495694e
Compare
This was referenced Oct 10, 2026
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.
Part of #758.
Before deciding whether to use the layers of an Illustrator file, the import drew them again for each text object made from the file's text document, to see whether the page draws it. When the layers differ from the page by more than the check allows even with every such object taken out, they aren't used whatever is taken out. Taking one out only changes what is drawn in its box, so that case is now found first (the boxes' share of the page against the margin) and the weighing is skipped.
This only skips work whose answer is already known: what a file opens as doesn't change.
Tested:
cargo xtask ci(all 7 steps) on the stack of this batch's PRs, rebased onmain..ai/EPS import tests (tests_native,tests_aiimport) andimport_fuzzpass unchanged.🤖 Generated with Claude Code