Fix side panel items visibility on mobile devices#119
Conversation
|
Jest w porządku. |
|
Uh, @Patrycjerz też zgłaszał uwagi - wygląda na to, że jeszcze jest tam kilka rzeczy do poprawienia. Zajmę się tym, gdy będę miał chwilę. |
|
Pewnie mówił o tym samym, bo wczoraj też o tym rozmawialiśmy między sobą :) Ten PR uznajemy jako gotowy i ewentualnie zrobisz w innym czy czekać z tym? |
|
Poczekajmy, tutaj wrzucę. |
| width: 100% !important; | ||
| display: block !important; | ||
| } | ||
| } |
There was a problem hiding this comment.
Tak jak mówiłem wcześniej @Argeento, w wersji komputerowej pojawia się brzydkie rozciągnięte menu na dole strony, gdy jest ona zwężona. Oczywiście to nie jest bardzo znacząca wada, prawie nikt na to i tak nie spojrzy, no ale wg mnie lepiej nie eksponować takich niedoróbek. Na pewien czas (zanim się nie dorobimy prawdziwego RWD) można dodać warunek z funkcją qa_is_mobile_probably() i wg niego dodawać to display: none. Za niedługo spróbuję wrzucić commita to poprawiającego.
Dawno temu przez przypadek ukryłem cały side panel na mobile (#90 linijka). Okazuje się jednak, że mobile ma własną wersje tego panelu, niezależną od rozdzielczości ekranu - wszystko było okej. (Przy okazji poprawiłem wcięcia)