Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Odkazy na půjčovnu vodáckého materiálu #58

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Grejdy
Copy link

@Grejdy Grejdy commented Nov 2, 2023

Změny

V postranním menu přibyla rozbalovací nabídka „půjčovna“ obsahující odkazy na půjčovnu vodáckého vybavení.

Testovací scénář

  1. Otevři si test.interni.bosan.cz a obnov stránku.
    • Windows: CTRL + F5
    • Mac: ⌘ Cmd + ⇧ Shift + R
  2. Zkontroluj, že to tak je, a že odkazy fungují a odkazují na rozumné místo. Můžeš mi i dát názor, jestli jsou popisy odkazů takhle dobré a jestli by nebylo lepší sjednotit dizajn odkazů s nadřazenými prvky, nebo jinak upravit.

Po otestování vždy napiš feedback, buď Approve review, nebo Request changes. Návod zde: Jak na testování

Copy link
Member

@DavidTvrdy DavidTvrdy left a comment

Choose a reason for hiding this comment

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

Názor na grafiku odkazů nemám, ani na to, jestli nechat pouze "Půjčovna" s odkazem na ten formulář nebo rozdělit na dva odkazy (jak je teď).
Všechny odkazy bych ale každopádně otevíral na nové kartě.

@Grejdy Grejdy linked an issue Nov 2, 2023 that may be closed by this pull request
@Grejdy
Copy link
Author

Grejdy commented Nov 2, 2023

Ještě jsem to změnil z jednoduchého odkazu na něco podobnějšího zbytku toho panelu.

@Grejdy
Copy link
Author

Grejdy commented Nov 2, 2023

Když na to teď koukám, možná by dizajnově dávalo větší smysl, dát to až nad ten odkaz na veřejnou… nebo ne?

Copy link
Member

@DavidTvrdy DavidTvrdy left a comment

Choose a reason for hiding this comment

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

Odkazy jsou takhle hezčí.
Ale všimnul jsem si, že na mobilu po kliknutí někam v menu se menu zavře. Pro rozklikávací položku je to nežádoucí - menu by se nemělo zavírat, protože ho pak musím znovu otevřít, než mžu kliknout na jeden z odkazů.

@Grejdy
Copy link
Author

Grejdy commented Nov 3, 2023

Ale všimnul jsem si, že na mobilu po kliknutí někam v menu se menu zavře. Pro rozklikávací položku je to nežádoucí - menu by se nemělo zavírat, protože ho pak musím znovu otevřít, než mžu kliknout na jeden z odkazů.

Jo, to je hloupý. Nějak se mi to ale nedaří napravit. Zkoušel jsem různé internetové návody, jak tohle řešit u ionicových menu, ale nic nefunguje.
Je to menu vůbec z ionicu, nebo je to něco jinýho? Moc se to nepodobá ničemu, co jsem dokázal najít. A kde je to do toho webu implementovaný (resp. kde se to bude dát konfigurovat) @SmallhillCZ ?

@Grejdy
Copy link
Author

Grejdy commented Nov 14, 2023

Tak, s půjčovnou to aktuálně vypadá tak, že se pravděpodobně bude nějakým způsobem zjednodušovat, takže tuhle záležitost bych uspal, než to ve vodáckém výboru vymyslíme a pak možná přestane dávat smysl mít tam dva odkazy, takže se tím vyřeší i problém se zavíráním menu při rozbalení nabídky.

@SmallhillCZ SmallhillCZ marked this pull request as draft January 28, 2024 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Odkaz na Půjčovnu vybavení
2 participants