search

pirati-web / jekyll-theme-pirati

Web desing of Czech Pirate Party
6 stars 17 forks source link

User calendar #36

Open milladekilla opened 5 years ago

milladekilla commented 5 years ago

Pridava pro uzivatele ktery ma vyplneny meta "calendar:@group.calendar.google.com" embed kalendare na profil

xaralis commented 5 years ago

Za PR děkujeme, ale před mergnutím je nutné vyřešit několik připomínek, které jsem přidal do code review.

milladekilla commented 5 years ago

A kde je teda najdu?

xaralis commented 5 years ago

@milladekilla https://github.com/pirati-web/jekyll-theme-pirati/pull/36/files

milladekilla commented 5 years ago

ja teda nevim, ale zadne pripominky tam nevidim ...

xaralis commented 5 years ago

Divný.. je tam tohle.

image

milladekilla commented 5 years ago

Hele, id neni class - bem je pro classy ne ID. Pokud se hleda dom prvek v JS je ID nejryclejsi cesta, proto je pouziti ID na miste - ono to neni zrejme z tohoto kodu ale nakonec tam bude.

On je toto pouze priskok k tomu abychom prenesli funkcionalitu z https://www.pirati.cz/lide/ivan-bartos/ do tematu. Jednak je to z popudu Adama Zabranskeho ale i jinych novych radnich kteri po tomto vyslali poptavku.

V pravem sloupci můžeš mít tak agendu, ale ta je při větším počtu eventů celkem nepřehledná, je tedy záměr, mít na profilech kalendře měsíční.

Můžeš to konzultovat klidně s Jitkou, momentálně se tento PR stává irelevantní - jitka to custom nacpala do prahy, aby to Adam měl. Ale jak říkám brzi budeme resit cely ten profil, tak at vime proc to je.

BTW. ty bys asi nevedel jak pridat do webpack buildu novy JS file, coz?

xaralis commented 5 years ago

@milladekilla BEM je samozřejmě pro classy. Otázka je, k čemu používat ID? Obecně to má spoustu nevýhod z nichž největší je ta, že to neumožňuje ten kalendář mít v nějaké podobě na jedné stránce víckrát. Čili řešení pls formou classy, které to spustí na všech matchnutých elementech tak jako to máme všude jinde.

Pokud potřebujeme kalendář ryze měsíční, tak nejlepší řešení je vyprdnout se na google a naimplenetovat si to opět po svém (podobně jako ten agenda widget).

Důvod je ten, že jedním z hlavních cílů redesignu je docílit vzhledu, který má nějakou úroveň. Ten googlovský iframe je pěkná žumpa a upravit to nějak, aby to vypadalo k světu, moc bohužel nejde.

Do webpack buildu se nový file dostává jedinou cestou, skrze require (resp. import pro ES6). Např. zde: https://github.com/pirati-web/jekyll-theme-pirati/blob/master/assets/js/apps.js#L2