Microsoft/calculator

Centralize BiDirectional string code in the CalcViewModel

Open

#65 aperta il 27 feb 2019

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)C++ (2422 fork)batch import
Pri: 2approvedcodebase qualityhelp wanted

Metriche repository

Star
 (16.196 star)
Metriche merge PR
 (Merge medio 15h 43m) (7 PR mergiate in 30 g)

Descrizione

BiDi code is spread out - Memory panel sets BiDi in the view model, while History sets it in the CalcManager. The UnitTests also do their own thing. We should centralize the code into utility functions in the VM and then have unit tests for their functionality.

Guida contributor

Direzione di ricerca
Esamina il codice esistente per la logica BiDi nel modello di visualizzazione del pannello Memory e in CalcManager del pannello History, quindi crea funzioni di utilità centralizzate in CalcViewModel e scrivi test unitari.
Tech stack
cpp
Dominio
desktop
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
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
GitC++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60