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

Obrázky k tréninkům a přihlašovací tlačítka #593

Merged
merged 3 commits into from
Jun 18, 2024

Conversation

Grejdy
Copy link
Contributor

@Grejdy Grejdy commented Jun 8, 2024

Změny

  • 2 ilustrační fotografie k tréninkům na K
  • přihlašování na tréninky (na K i v troji) jako velké tlačítko místo nenápadného odkazu

Testovací scénář

  1. Otevři si test.bosan.cz a obnov stránku.
    • Windows: CTRL + F5
    • Mac: ⌘ Cmd + ⇧ Shift + R
  2. Zkontroluj, že …
    • to vypadá takhle dobře
    • všechny odkazy fungují (mne tam třeba některé nefungovaly, ale možná to bylo tím lokalhostem)

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

@Grejdy
Copy link
Contributor Author

Grejdy commented Jun 8, 2024

Přijde mi teda, že ty linky nahoře teďka scrollují až moc dolu, ale nevím, jak to napravit. Nevidím mezi těmi svými úpravami, co by to mohlo mít na svědomí.:-)

@DavidTvrdy
Copy link
Member

Přijde mi teda, že ty linky nahoře teďka scrollují až moc dolu, ale nevím, jak to napravit. Nevidím mezi těmi svými úpravami, co by to mohlo mít na svědomí.:-)

Taky mi přijde

@DavidTvrdy
Copy link
Member

Ty lidi na fotce (nemyslím děti a členy) jsou v pohodě s tím, že budou na webu?

@Grejdy
Copy link
Contributor Author

Grejdy commented Jun 8, 2024

Ty lidi na fotce (nemyslím děti a členy) jsou v pohodě s tím, že budou na webu?

Good point. Předpokládám, že s tím budou v pohodě, ale můžu to zkusit v pondělí ověřit (jde o Tramvajákovu maminku a… toho pána budu muset ještě zjistit, ale je to někdo od Žabáka).

@Grejdy
Copy link
Contributor Author

Grejdy commented Jun 8, 2024

Jo a teďka jsem si všiml, že v tom souhlasu s Trójou je ještě pořád uvedený rok 2016. S tím by se asi mělo něco udělat. Je potřeba mít takovýhle dokument na konkrétní rok? Nestačilo by tam napsat třeba „v tomto kalendářním roce“? Můžu teda zkusit vykoumat, jestli se nedá dát do pdf proměný rok, ale nejsem si vůbec jistý, že to jde.

@Grejdy
Copy link
Contributor Author

Grejdy commented Jun 8, 2024

Přijde mi teda, že ty linky nahoře teďka scrollují až moc dolu, ale nevím, jak to napravit. Nevidím mezi těmi svými úpravami, co by to mohlo mít na svědomí.:-)

Taky mi přijde

Teďka koukám, že to pravděpodobně není moje chyba - ono to scrolluje takhle hloupě i ve staré verzi a akorát u tréninků na K to doteď němělo kam scrollovat níž:-)
Asi se mi to povedlo opravit… je to sice dost čuňácké řešení, ale vypadá, že funguje:-) Tipuju, že ta komponenta, co to scrollování zařizuje, nepočítá s tím, že ten horní panel překrývá část obsahu a scrolluje prostě, jako by tam nebyl.

@Grejdy
Copy link
Contributor Author

Grejdy commented Jun 10, 2024

Ty lidi na fotce (nemyslím děti a členy) jsou v pohodě s tím, že budou na webu?

Dneska jsem se obou ptal (Tramvajákovy maminky přímo a Žabákovýho dědy skrze Kaštana) a oba se tvářili jako samozřejmě, že s tím nemaj problém, proč se na to vůbec ptám:-)
Tedy posvěcení od zúčastněných máme.

@DavidTvrdy
Copy link
Member

  1. test.bosan.cz

Jo, předělal bych to, rok 2016 nevypadá dobře. Jinak všechno ostatní vypadá dobře.

@Grejdy
Copy link
Contributor Author

Grejdy commented Jun 11, 2024

  1. test.bosan.cz

Jo, předělal bych to, rok 2016 nevypadá dobře. Jinak všechno ostatní vypadá dobře.

Ok. Bylo by v pohodě tam napsat prostě „pro tento kalendářní rok? (@elenfant ?)
Používá se ten souhlas ještě vůbec, když si za posledních 7 let nikdo nevšiml, že je tam špatný rok?

@Grejdy
Copy link
Contributor Author

Grejdy commented Jun 18, 2024

  1. test.bosan.cz

Jo, předělal bych to, rok 2016 nevypadá dobře. Jinak všechno ostatní vypadá dobře.

Dal jsem tam místo toho prostě „pro tento kalendářní rok“. Tím by to mohlo být vše a může se to pustit ven.

@Grejdy Grejdy enabled auto-merge June 18, 2024 12:43
@Grejdy Grejdy merged commit c9ea213 into master Jun 18, 2024
3 checks passed
@Grejdy Grejdy deleted the feature/K-training-images-and-buttons branch June 18, 2024 20:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants