search

mobilecodereviewer / MobileCodeReviewer

0 stars 0 forks source link

[M] Dziwne wyświetlanie flag przy komentowaniu zmiany #17

Closed fracz closed 9 years ago

fracz commented 9 years ago

Nie zawsze. Ale np zawsze przy wyświetlaniu zmiany "Fix error when printing board" z naszego Gerrita.

Dodałem tam customową flagę "MasterApproved" i myślałem że dlatego, ale druga zmiana z tego projektu działa ok.

Nie da się tego okienka zamknąć przez przycisk "Wstecz" w telefonie - trzeba klikać cancel.

d00d171 commented 9 years ago

W tej chwili wyświetlanie jest poprawne jeżeli użytkownik jest dodany jako "Reviewer" do zmiany - do poprawy.

A zachowanie z zamykaniem tylko po naciśnięciu Cancel akurat bym zostawił, ale to już kwestia usability. Wolałbyś żeby back też zamykał okienko?

fracz commented 9 years ago

Wszędzie da się wsteczem z wszystkiego wyjść więc pomimo że jest przycisk cancel - wstecza bym nie blokował.

d00d171 commented 9 years ago

Wstecz zamyka te popupy od wersji 0.3.17

d00d171 commented 9 years ago

Poprawione od wersji v0.3.18