search

ZdruzenieSTROM / webstrom-frontend

4 stars 0 forks source link

Refactor SemesterPicker Dropdown #384

Open Matushl opened 1 month ago

Matushl commented 1 month ago

Bolo fajn prerobit semester picker dopdown do mui dropdown componenty, ktora moze mat poriesene edgecases, ktore mi mozno nemame.

rtrembecky commented 1 month ago

pozeral som na to teda, neni to len tak, lebo Select je mysleny ako form element so select state-om, no my to mame ako zoznam linkov... pozeral som, ze vhodnejsie su tym padom Menu elementy, no tie ti by default blokuju scroll stranky a tak... lebo je to interne nakodene cez Popover. alternativa, ktoru odporucaju, je pouzit nejake inner komponenty a Popper - ten neblokuje scroll, ale zas neriesi sam od seba close na click outside... kazdopadne, mam rozrobeny prepis aspon toho absolutne positionovaneho listu linkov do Popperu, to funguje celkom pekne 🙂

Matushl commented 1 month ago

plus som si uvedomil, ze na "emulacii" mobilu v prehliadaci (zmensenie obrazovky) sa to sprava hodne divne, takze je mozne ze to bude trochu urgentnejsie issue ako 3.0, takze presuvam to na 1.1