facebookarchive/atom-ide-ui

Allow code format providers to return null to 'fall through'

Open

#26 aperta il 13 set 2017

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)JavaScript (110 fork)batch import
code formatenhancementhelp wanted

Metriche repository

Star
 (937 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Code format providers are currently forced to return something: https://github.com/facebook-atom/atom-ide-ui/blob/master/modules/atom-ide-ui/pkg/atom-ide-code-format/lib/types.js

However, this makes it difficult to have multiple formatting providers for the same document, since we only use the one with the highest priority.

Instead, we should use ProviderRegistry.getAllProvidersForEditor (e.g. on https://github.com/facebook-atom/atom-ide-ui/blob/master/modules/atom-ide-ui/pkg/atom-ide-code-format/lib/CodeFormatManager.js#L257) and take the first one with a non-nullable result.

cc @damieng

Guida contributor

Direzione di ricerca
Esamina il file types.js per vedere il tipo di ritorno corrente. Quindi aggiorna CodeFormatManager.js per usare getAllProvidersForEditor e iterare per trovare il primo risultato non nullo.
Tech stack
javascript
Dominio
frontenddesktop
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
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.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70