googlefonts/glyphsLib

Handle default layer with custom name

Aperta

#619 aperta il 17 ago 2020

Vedi su GitHubGuida contributor
 (0 commenti) (1 reazione) (0 assegnatari)Python (55 fork)github user discovery
enhancementhelp wanted

Metriche repository

Star
 (198 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

https://unifiedfontobject.org/versions/ufo3/layercontents.plist/ says:

public.default

This layer name indicates that the default layer has no user defined name. It must not be used as the layer name for any directory other than the default glyphs directory.

GlyphLib assumes the default layer has no user defined name but that may not be the case. https://github.com/googlefonts/glyphsLib/blob/638cbddd00b2334410bab914d932bf673d6342ca/Lib/glyphsLib/builder/layers.py#L74 Instead it should check that the layer is font.layers.defaultLayer or that its name is font.layers.defaultLayer.name.

Guida contributor

Direzione di ricerca
Esamina la condizione alla riga 74 in layers.py e modificala per verificare se il layer è quello predefinito confrontandolo con font.layers.defaultLayer o il suo attributo name.
Tech stack
python
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Python
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65