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

Přidej se k FIT++ #78

Open
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

Přidej se k FIT++ #78

wants to merge 12 commits into from

Conversation

Bertik23
Copy link
Contributor

Stránka hotová s textem, v navbaru jsem jí nijak nezvíraznil, protože netuším jak 🙈

image

@stepech
Copy link
Member

stepech commented Dec 3, 2024

Tady se vyjadřujeme hlavně k tom layout, stránce navíc apod., nebo i textaci samotné? 😃

@Bertik23
Copy link
Contributor Author

Bertik23 commented Dec 3, 2024

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

Copy link
Member

@stepech stepech left a 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.

_data/navbar.yml Outdated Show resolved Hide resolved
assets/css/input.css Outdated Show resolved Hide resolved

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?**
Copy link
Member

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.

@Standardik
Copy link

...kterou najdeš tady v kalendáři - pod to slovo odkaz na stránku s kalendářem?
...někomu z představenstva - používejme všude velké P, Představenstva

@Bertik23
Copy link
Contributor Author

Bertik23 commented Dec 5, 2024

...kterou najdeš tady v kalendáři - pod to slovo odkaz na stránku s kalendářem?
je to teď link na kalendář
...někomu z představenstva - používejme všude velké P, Představenstva
fixed snad

Copy link

@rakmaty1 rakmaty1 left a 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;
Copy link

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;
Copy link

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;
Copy link

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.
Copy link

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ů
Copy link

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?**
Copy link

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
Copy link

Choose a reason for hiding this comment

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

tady taky

@stepech
Copy link
Member

stepech commented Dec 6, 2024

Ještě dotaz k tomu navbaru - teď je široký stejně jako text na stránce, což je za mě osobně preferovné, viz:

image

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

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.

4 participants