iiuni / projektzapisy

System Zapisów na zajęcia w Instytucie Informatyki Uniwersytetu Wrocławskiego
https://zapisy.ii.uni.wroc.pl
30 stars 10 forks source link

Usunięcie powiadomienia (jednego bądź wszystkich) chowa cały łidżet #1642

Closed lgpawel closed 5 months ago

lgpawel commented 6 months ago

Zdaje się, że mimo Drobiazgowego Testowania™ przegapiliśmy zmianę zachowania (drobną, ale na gorsze) przemyconą przy upgrade Bootstrapa z 4 do 5. W łidżecie powiadomień (który pojawia się po kliknięciu ikonki z dzwonkiem na górze dowolnej strony) jest przycisk "Usuń wszystkie powiadomienia", a każde powiadomienie ma przycisk z krzyżykiem, który usuwa tylko je. W starym Bootstrapie (czyli np. nadal na produkcji) po kliknięciu któregoś z nich łidżet pozostawał rozwinięty (choć z odpowiednio zmienionym stanem); w nowym (czyli np. na master-dev) stan powiadomień dalej zmienia się jak trzeba, ale dodatkowo łidżet chowa się. Pożądane jest oczywiście przywrócenie starego zachowania.

Zdaje się, że podobny (przynajmniej do pewnego stopnia) problem był z panelem rozwijającym się po kliknięciu "Zaloguj się" i tam został on w porę dostrzeżony (bo panel i ważniejszy, i zawsze ma nietrywialną zawartość) i rozwiązany, więc być może wystarczy wygrzebać odpowiednią zmianę i nanieść ją również w łidżecie powiadomień.