Glyphs-Scripts icon indicating copy to clipboard operation
Glyphs-Scripts copied to clipboard

Kerncrash current glyph error on G3.

Open synthview opened this issue 4 years ago • 2 comments

Error and no message in Macro window.

Screenshot 2021-02-19 at 17 35 29

synthview avatar Feb 19 '21 16:02 synthview

@RafalBuchner:

  • [x] abstract more common functions between Kern Crasher and Kern Crash Current Glyph, put them in a separate kernanalysis.py
  • [x] make sure kernanalysis.py is imported and used in both
  • [x] with enough abstraction, consider adding the (Glyphs 3) RTL option that is mentioned as todo in the code. I guess it should query the current state of the edit view (LTR vs RTL), and act accordingly.

I believe currently the functions are a mess. Some of them are local and should be in kernanalysis.py so they can be shared and maintained between both scripts, and some are in both places. The Kern Crasher script is the way it should work, so it should be the reference for both.

mekkablue avatar Feb 23 '21 09:02 mekkablue

@synthview: This may take a while to sort out. Workaround is to use the main Kern Crasher script with appropriate settings.

mekkablue avatar Feb 23 '21 09:02 mekkablue