-
Notifications
You must be signed in to change notification settings - Fork 10
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
Přidej se k FIT++ #78
base: main
Are you sure you want to change the base?
Conversation
Tady se vyjadřujeme hlavně k tom layout, stránce navíc apod., nebo i textaci samotné? 😃 |
No mělo by to splňovat #66 Text jsem takhle dostal, takže spíš ten layout a jestli nejsou v kódu warcrimes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Díky za návrh, all in all to vypadá hezky. Ty commenty jsou spíš opinionated než cokoliv jiného.
|
||
Jsme klub FIT++, který působí na Fakultě informačních technologií ČVUT. Pořádáme akce jako [Hello FIT](https://klubfitpp.cz/hellofit), FIT párty, deskovky na fakultě, FITripy, LAN párty a spoustu další edukačních nebo zábavných eventů, na kterých se můžeš nejenom dozvědět knowlage nad rámec předmětů ve škole, ale hlavně poznat spoustu aktivních lidí z fakulty. | ||
|
||
**Proč bych se měl\* přidat?** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Teď to asi není důležité, ale kapitoly bych pak zviditelnil přes markdown nadpisy (úroveň 2 nebo 3, idk.. cokoliv), z toho screenu cos posílal mi stránka přišla hodně... plochá? hodně textu, málo výrazného.. čehokoliv.
...kterou najdeš tady v kalendáři - pod to slovo odkaz na stránku s kalendářem? |
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Kouknul jsem na to a přidal jsem pár komentářů, jinak je to moc pěkné :)
|
||
.page a { | ||
text-decoration: underline; | ||
outline-color: transparent; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Vím že žádný code-style guide asi nevedeme a třeba v app.css jsou všude dvě mezery. Ale v input.css jsou všude jinde čtyři mezery, tak by to chtělo i tady.
} | ||
|
||
.page a:link { | ||
color: #aa83e0; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a tady
} | ||
|
||
.page a:visited { | ||
color: #120682; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a tady
|
||
Tak jsi tady správně! | ||
|
||
Jsme klub FIT++, který působí na Fakultě informačních technologií ČVUT. Pořádáme akce jako [Hello FIT](https://klubfitpp.cz/hellofit), FIT párty, deskovky na fakultě, FITripy, LAN párty a spoustu další edukačních nebo zábavných eventů, na kterých se můžeš nejenom dozvědět knowlage nad rámec předmětů ve škole, ale hlavně poznat spoustu aktivních lidí z fakulty. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
knowlage -> knowledge
|
||
**Proč bych se měl\* přidat?** | ||
|
||
- Rozšíříš si kontakty, poznáš nové kamarády nejenom z FIT ale i z dalších fakult nebo studentských spolků |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FIT ale -> FIT, ale
- Přijď na zasedání Představenstva a společně něco vymyslíme | ||
- Kandiduj do Představenstva a buď nositelem myšlenky | ||
|
||
**Co tam v tom klubu vlastně děláte?** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
V ostatních "kapitolách" máme body s velkými písmeny na začátku a tady jsou najednou malá, asi by se hodilo mít to všude stejně.
|
||
**Jak se můžu oficiálně stát členem/členkou?** | ||
|
||
- připoj se na FIT++ discord a nastav si jako přezdívku na serveru svoje jméno |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tady taky
Ještě dotaz k tomu navbaru - teď je široký stejně jako text na stránce, což je za mě osobně preferovné, viz: Na screenshotu od tebe (na začátku issue) se rozjíždí do plné šíře stránky. Je to omylem, nebo naschvál s přidáním loga? Nešlo by nechat tu šířku jak je a logo jen umístit jako první button vlevo? To roztažení se mi moc nelíbí. |
Stránka hotová s textem, v navbaru jsem jí nijak nezvíraznil, protože netuším jak 🙈