Update from task cc676cbf-bb93-4d98-840c-2c7b8a242427#3
Open
MekaClaude wants to merge 84 commits into
Open
Conversation
- Modified src-js/views-editor/src/visualization-layer-definitions.js to remove the "fontra.guidelines" visualization layer definition while preserving the "fontra.selected.guidelines" layer for selected guideline functionality - Updated .gitignore to include the visualization layer definitions file path - The removal maintains existing guideline selection and interaction capabilities while eliminating the static display of all guidelines in the editor view
MekaClaude
commented
Apr 24, 2026
Owner
Author
MekaClaude
left a comment
MekaClaude
left a comment
There was a problem hiding this comment.
I will review it later and get back to you 👍
Key features implemented: - Remove unused RF guideline lock lib prefix constant from designspace backend - Remove unused guidelines field from UFOGlyph and UFOFontInfo dataclasses - Remove unused unpackGuidelines and packGuidelines functions from designspace backend - Remove guidelines field from FontSource and StaticGlyph classes in core classes - Update clipboard parsing to remove guideline handling This cleanup removes dead code related to RoboFont-style guideline locking that was no longer used, streamlining the codebase and removing unnecessary data handling.
Update from task clean remove Guidelines
Update git ignore env files
Only raise NotImplementedError if there are rules
…e* source to user space, not *unapply* the mapping
Fix avar (un)mapping for conditional substitutions when preparing shaper font
Move it after direction, script, or language have been explicitly set, as it will only try to guess unset properties.
Migrate to harfbuzzjs v1.x API
Bumps [postcss](https://github.com/postcss/postcss) from 8.5.1 to 8.5.12. - [Release notes](https://github.com/postcss/postcss/releases) - [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md) - [Commits](postcss/postcss@8.5.1...8.5.12) --- updated-dependencies: - dependency-name: postcss dependency-version: 8.5.12 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
…stcss-8.5.12 Bump postcss from 8.5.1 to 8.5.12
Bump build shaper font
The `scrollIntoView: true` is equivalent EditorView.scrollIntoView() with default options which is `y: "nearest"`, but this would often scroll to the selection just enough to make the line appear at the end of the view, which is not very helpful. Using EditorView.scrollIntoView() so we can set `y: "center"`, which is nicer as the start of the span becomes in the middle of the page.
Center selection when jumping to feature code link
It looks like: feature ccmp lookup 0 (ccmp_1) The features are reordered to be first so they align better, since lookup names vary in length. Lookup name is also removed from the source location.
[shaping] Make dumb shaper dumber
canvas: make min zoom limit relative to em size (UPM)
To match prominent links on the homepage
Add read-only query parameter for opening a font in read-only mode
Add blog and Discord links to fontra-menus
[Axes panel] Add 'Hidden' checkbox to Axis box and New axis dialog
Bump harfbuzzjs to latest beta
…k-42427 # Conflicts: # src-js/views-applicationsettings/src/panel-server-info.js
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
6 participants
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.
This PR was created by qwen-chat coder for task cc676cbf-bb93-4d98-840c-2c7b8a242427.