User calendar #36
User calendar #36
Conversation
…alendar.google.com" embed kalendare na profil
hotfix
|
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. |
|
A kde je teda najdu? |
|
ja teda nevim, ale zadne pripominky tam nevidim ... |
|
Divný.. je tam tohle.
|
|
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? |
|
@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 |
Pridava pro uzivatele ktery ma vyplneny meta "calendar:@group.calendar.google.com" embed kalendare na profil