[M] Dodać kolorowanie składni

[fracz]

jw.

[fracz]

Czcionka pasowałaby monospace :-)

[liscju]

Poprawione od wersji 0.3.26

[fracz]

Jest ok. Zmieniłbym tylko kolorek komentarzy bo za bardzo zlewa się z tłem.

[fracz]

Strasznie wolno to działa. Mogę zaakceptować, że chwilę zaczekam na pierwsze pokazanie pliku (pod warunkiem #7) ale już niefajnie jest, jak dodaję komentarz i po wybraniu "OK" znowu czekam aż on się doda. Wydaje mi się, że na dodanie komentarza kolorujecie wszystko od początku? Czy tyle trwa komunikacja z API?

Jeżeli po #7 przy dodawaniu komentarza również będzie animacja a czekanie nie wynika z kolorowania kodu wiele razy, nie trzeba nic poprawiać.

Podobnie wolno działa pokazywanie komentarza po tapnięciu w linię.

[d00d171]

Poprawiłem dodawanie komentarzy które było przyczyną wielokrotnego kolorowania. Teraz powinno działać szybciej, bo po dodaniu komentarza (przez api) updatuje też model po naszej stronie o nowy komentarze, zamiast pobierać całego pliku na nowo, jak robiłem do tej pory ;)

[fracz]

Jest lepiej. Natomiast zastanawiam się, czy na otwarcie listy plików nie dałoby się od razu pobrać ich kodu źródłowego? Nadal długo trwa przełączanie się między plikami a zakładam, że podczas review robi się to dość często. Akceptowalne byłoby więc, żeby lista plików otwierała się trochę dłużej a za to potem przełączanie się między plikami powinno być bez żadnych opóźnień.

Ewentualnie, żeby raz pobrany plik już się zapamiętywał dopóki robię review jednej zmiany i przełączanie się w przód i w tył już nie pytało Gerrita o kod.

I jeszcze jak przeglądam kod to intuicyjnie tapając w strzałkę w lewo w lewym górnym rogu powinienem wrócić się do listy plików, a nie do samego początku (listy zmian).

[d00d171]

Teraz nie trzymamy raz pobranego pliku, tylko przy każdym przełączeniu (nawet powrocie do tego samego) jest on pobierany na nowo. Moglibyśmy trzymać raz wyświetlony plik, żeby później było szybciej. Pobieranie wszystkich plików może trochę zająć i np. gdyby ktoś chciał na szybkości rzucić okiem na jeden (albo tylko sprawdzić jakie pliki zostały zmodyfikowane) to pewnie nie byłby zachwycony że musi czekać aż tyle. Co ty na to?

[fracz]

Tak, dopisałem nawet tą propozycję w komentarzu :)

[d00d171]

Dodałem cachowanie raz wyświetlonego pliku. Cache czyszczony po powrocie do ekranu change explorera albo dodaniu review. Nawigacja up teraz powinna działać. Problem wynikał z rozbieżności w definicji PARENT_ACTIVITY dla source explorera dla Androida <= 4.0