[M] Zmiana sposobu zaznaczania linii zawierającej komentarze

mobilecodereviewer / MobileCodeReviewer

0 stars 0 forks source link

[M] Zmiana sposobu zaznaczania linii zawierającej komentarze #29

Closed fracz closed 9 years ago

fracz commented 10 years ago

Na telefonie nadal zajmuje to 1/10 ekranu. Tym bardziej, że na początku nie ma żadnych komentarzy i jest zabrane miejsce w sumie nie wiadomo na co. Nadal jestem za podświetlaniem linii na wyraźny kolor - może żółty? Jako że zielone, czerwone i niebieskie mamy już zajęte.

liscju commented 10 years ago

Oczywiście tak możemy to zrobić - w jaki sposób proponowałbyś aby użytkownik mógł zobaczyć komentarze do linii np przycisk na action bar?

fracz commented 10 years ago

Jak przeglądam plik w celu odebrania review, mogę założyć że główną czynnością którą chcę wykonać jest przeczytanie komentarzy. Dlatego można je wyświetlić od razu po tapnięciu (zaznaczeniu) linii.

liscju commented 10 years ago

Zmienione od 0.3.24, linie są kolorowane na żółto ponadto jest robiony underline tych linii ,aby w diffie dla linii dodanych/odjętych czyli już pokolorowanych było widoczne ,że jest w nich komentarz , gdyż od tej wersji można oglądać komentarze tapnięciem zarówno z poziomu zwykłego kodu źródłowego jak i diffa.

fracz commented 10 years ago

Jest jeszcze niepotrzeny margines.

liscju commented 9 years ago

Margines usunięty od wersji 3.34