trojsten / liahen

1 stars 1 forks source link

Homepage remake + minor changes #28

Open Benjji opened 7 years ago

Benjji commented 7 years ago
Benjji commented 7 years ago

Viem ze este chyba zopar detailov voči #6 (pribudnu coskoro) Mohli by sme este prediskutovat ci tam tie stlpceky naozaj potrebujeme/budu sa nam tam pacit

Myslim ze by nebolo zle ked sa zmesti vsetok obsah na jednu obrazovku, co az na par detailov tak je (napr. začni ešte dnes tam zatial nie je) - čo vieme kludne niekam skomprimovať (pod nadpis?)... kedže sme sa bavili že nechceme veľmi scrollovať.

maaario commented 7 years ago

Mne ten slider nefunguje (chrome, ubuntu).

Možno s tým bude mať niečo spoločné toto console bug

V tých okienkach je podľa mňa príliš veľa textu ... obsah sa ale asi doklepne až nakoniec.

Prečo sú všetky tie stánky o nás, kontakt, ... naľavo? Viem, že keď si pozeráš zadanie, tak máš napravo formulár na odovzdávanie, ale keď tam formulár nie je, nemusí byť celý obsah tak naľavo. Tento problém je aj keď si pozeráš učebné texty. Možno by to chcelo text do stredu na väčšiu šírku, keď tu nie je ten submitform.

Benjji commented 7 years ago

Ohľadom slidera: Používam tiež ubuntu a v chrome, chromiume aj firefoxe to beží v pohode. Takže asi to bude niečo že máme rôzne configy. Ja som stiahol mastra a jedina konfiguračna zmena čo som urobil, že zmenil DATABASES v page/settings.py. Skús pozrieť čo je u teba.

Ohľadom textu: Mne sa lepšie číta text, keď nie je na celú šírku obrazovky (kedže mám široký monitor a nemusím očami stále behať zľava doprava). Potom každý text aj tak trochu nekompaktne vyzerá, že odstavec zaberá 1-2 riadky. Vycentrujem to.

maaario commented 7 years ago

v gite nie je priecinok dist/ https://github.com/trojsten/liahen/tree/introremake/static/unslider

black3r commented 7 years ago

Ten unslider je pridany dost zlym sposobom.. namiesto git clonovania a includovania zdrojakov staci includnut prave skompilovane js a css subory.., pokial si teda nejakym sposobom ten unslider neupravoval..

Benjji commented 7 years ago

@black3r Práveže zdrojáky som upravoval. Unslider totiž obsahoval bug, ktory zobrazuje buttony Prev a Next aj ked ich vyslovene v konfiguracii zakazes.

black3r commented 7 years ago

hm.. nie je jednoduchsie tie buttony css-kom schovat?

Benjji commented 7 years ago

Nefungovalo, unslider ich vytvara dynamicky v javascripte a meni hodnotu display.

Benjji commented 7 years ago

@maaario dakujem :) neviem prečo ale som sa prave z gitu dozvedel ze mal ten priecinok dist v .gitignore :(

black3r commented 7 years ago

btw. to s tym scrollovanim bolo asi myslene tak, ze aby clovek mal vsetky podstatne informacie bez scrollovania (bo vela dnesnych produktov ma take landing page ze co to vlastne robi sa dozvies az na druhej alebo tretej obrazovke (vratane firmy, v ktorej robim :D)), neznamena, ze za kazdu cenu sa musime mestit na jeden page-height..., to sa bez nastavovania vysok niektorych elementov v jednotkach vh moc ani neda, a takto to dost divno vyzera na velkom monitore (napr. v robote mam monitor s rozlisenim 2560x1440, na nom polovica sucasnej landing page je biely priestor)

Benjji commented 7 years ago

OK, pridám teda tie 4 stĺpce.

black3r commented 7 years ago

Nejake napady na vylepsenie / skraslenie dizajnu:

black3r commented 7 years ago

Este napady ku contentu carouselu:

Benjji commented 7 years ago

Uz by malo byt vsetko okrem ksp-login related poznamok.

Benjji commented 7 years ago

@black3r Nenechame cokolvek co sa tyka ksp_loginu na samostatne issues/PR? mam k tomu niekolko napadov co by sme mohli na stretnuti prebrat, vratane redesignu

maaario commented 7 years ago

Podla mna sa login perks mozu spravi vo vlastnom PR ( @Benjji mozes spravit issue a prepisat tam login related vei z konverzacie o PR + nadpisy v nastaveniach, zmene hesla...).

Este ku dizajnu: