Skip to content

Conversation

@Argeento
Copy link
Contributor

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)

@Argeento Argeento added the Bug label Feb 16, 2018
@Argeento Argeento self-assigned this Feb 16, 2018
@awaluk
Copy link
Member

awaluk commented Feb 17, 2018

Jest w porządku.
Można by przy okazji poprawić drobiazgi na tym panelu - tzn. przy jasnym motywie rozsypuje się lista kategorii, a z kolei przy dark theme kategorie wyglądają nawet ok, ale kolejne sekcje sidebara oddzielają białe linie (skoro ciemny motyw to raczej nie powinno ich tam być). Ale to jak chcesz, tak przy okazji mówię że taka kwestia jest :P

@Argeento
Copy link
Contributor Author

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ę.

@awaluk
Copy link
Member

awaluk commented Feb 17, 2018

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?

@Argeento
Copy link
Contributor Author

Poczekajmy, tutaj wrzucę.

width: 100% !important;
display: block !important;
}
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@awaluk awaluk marked this pull request as draft June 7, 2020 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants