Reservant-inc / reservant-frontend

Frontend part of Reservant project
0 stars 1 forks source link

Ms issues #138

Closed s26229 closed 8 hours ago

s26229 commented 3 months ago

Dodałam tooltipy w navarze do do threads notifications I tools. Poprawiłam dark theme tam gdzie było wskazane. Czy zając się od razu poprawą dark theme w całej aplikacji?? Te issue z zmianą koloru tej ramki okazało się wcale nie takie proste żeby zmienić jej kolor lub usunąć chociaż.

KubaBili001 commented 3 months ago

Pisz proszę numerki issue które rozwiązujesz żeby mi było łatwiej się do niech przenieść, np w opisie commita :

123

101

991

Tooltipy są git, przydałoby się jeszcze dodać do tych 2 głównych sekcji, czyli Home i Restaurant Management. Reszte będzie można zrobić jak dam merge swojego brancha.

Do tytułów tooltipów od razu daj tłumaczenie, żeby potem nie zapomnieć.

Tryb ciemny w całej aplikacji gdzie ci się rzuci w oczy, ale tak jak pisałem ja ogarnę ten cały ekran z mapą.

image

Tutaj ten hover jest trochę za jasny, lepiej będzie chyba bg-grey-4

image

Tutaj też proszę zrób ciemny theme. W niektórych miejscach może być błąd jak będziesz próbowała wejść, ale fix jest na branchu jarka który mam nadzieję zaraz zmerguje.

image

To nie było częścią issue żadnego, tylko informacyjnie, ale zrobiłem klasę css scroll, która na komponencie który jest scrollowalny zmienia jego wygląd tak, żeby było jednolicie w całej aplikacji. Czyli np:

Jak byś chciała to możesz w tych tabelach i w innych miejscach jak czasem zauważysz że jest default scroll przy okazji dodać tą klasę to będzie wszędzie ładnie, i nie przejmuj się, że kolor nie zmienia się w zależności od theme, u mnie na branchu już będą 2 różne scrolle.

Ramki na razie zostaw.

Nie zamykam żadnego issue na razie, na tym branchu dokończ tylko Tooltipy i zrób commit to zamknę tego pulla. Resztę ciemnego theme zrobisz na nowym branchu który zrobisz po tym jak skończę mergować wszystko za jakąś godzinę.