After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 694335 - use combining diacritics and anchors
use combining diacritics and anchors
Status: RESOLVED FIXED
Product: cantarell-fonts
Classification: Core
Component: general
unspecified
Other Mac OS
: Normal normal
: ---
Assigned To: Cantarell fonts maintainer(s)
Cantarell fonts maintainer(s)
: 700614 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2013-02-21 04:16 UTC by Denis Jacquerye
Modified: 2015-12-17 13:16 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch fixing combining circumflex, building acircumflex with it (2.82 KB, patch)
2013-02-21 04:16 UTC, Denis Jacquerye
none Details | Review

Description Denis Jacquerye 2013-02-21 04:16:49 UTC
Created attachment 237012 [details] [review]
patch fixing combining circumflex, building acircumflex with it

It would be good to use combining diacritics as reference in accented characters instead of ASCII or modifier letter diacritics. See how DejaVu fonts or Source Sans do it.

See attached patch fixing combining circumflex U+0302 and building acircumflex U+00E2.
- U+0302 had the wrong outline (ASCII circumflex instead of the circumflex used in accented letters)
- 'mark' feature anchor is added to base letter (a) and mark (U+0302)
- in Fontforge Prefs: Accents > PreferSpacingAccents = Off
- build accent: Element > Build > Build Accented Glyph

Once all diacritics and base letters have anchors, all accented characters can be built and others can be composed when not in Unicode as precomposed.
Comment 1 Jakub Steiner 2013-02-27 11:56:50 UTC
The provided patch doesn't seem to apply cleanly. Can you provide one against the current git master, preferably using format-patch?
Comment 2 Denis Jacquerye 2013-03-01 08:01:05 UTC
Applying the patch with
$ patch src/Cantarell-Regular.sfd cantarell.patch
works here on a freshly git cloned source.

For some reason git 'format-patch' doesn't give me anything.
Comment 3 Denis Jacquerye 2013-03-02 19:30:05 UTC
I started a tree with anchors: https://github.com/moyogo/cantarell
I also fixed non-rounded points and started fixing some outlines.
Comment 4 Jakub Steiner 2013-04-23 16:53:56 UTC
After resolving some conflicts I have successfully pulled your changes in. Let me know when you're done with the diacritics. Thanks for your work!
Comment 5 Jakub Steiner 2013-05-20 11:58:37 UTC
*** Bug 700614 has been marked as a duplicate of this bug. ***
Comment 6 Nikolaus Waxweiler 2015-12-17 13:16:02 UTC
Should be done in master.