Přidání návodu k odebírání kalendáře akcí #574
Conversation
|
Jo a myslím, že tam možná bude problém s velikostí obrázků na menší obrazovce - třeba na mobilu a nepřišel jsem na to, jak to vyřešit. |
|
test.bosan.cz .. 404 page not found |
už to funguje |
Kde byl problém? |
|
Random poznámka - dal bych ten odkaz na návod třeba jen na slova "odběr akcí" |
Good point. Už nevím, co jsem tam napsal, ale bylo to dlouhý:-) |
|
A možná, když na to tak koukám, tak ještě třeba v rámci příští branche trošku přepíšu jeden bod v tom Contributing.md (on asi aktuálně není nikdo, kdo by to četl, ale ať je to aktuální…), zase předělali prostředí VS Code a chvíli mi trvalo, než jsem zjistil, proč mi to hází chybu:-) |
There was a problem hiding this comment.
Okno s návodem se musí scrollovat do strany - nemůže být prostě širší? Je docela úzké. Proč je tam vevnitř to šedé pole s obsahem, nemůže to být rovnou v tom okně (nebo prostě aby tam nebyly dva posouvače do strany)?
Třetí příklad mi přijde zbytečný... Nevím, jestli někdo používá speciální aplikace na kalendář a neumí si do něj kalendáře přidat.
Text podle mě všude do bloku, ne na střed.
Pár drobných poznámek v komentářích u kódu.
Velikost obrázků na mobilu mi nevadí, nepřijdou mi nepřiměřeně velké, naopak pro zvětšení se dá zoomovat.
web/src/app/components/calendar-sync-manual/calendar-sync-manual.component.html
Outdated
Show resolved
Hide resolved
| <div class="col-12 text-center"> | ||
| <h5>Stačí postupovat podle návodu:</h5> | ||
| </div> | ||
| <div class="col-12 text-center"> |
There was a problem hiding this comment.
Zarovnat text do bloku, ne jen na střed
web/src/app/components/calendar-sync-manual/calendar-sync-manual.component.html
Outdated
Show resolved
Hide resolved
| </ol> | ||
| </div> | ||
|
|
||
| <div class="col-12 text-center" style="color: dimgray;"> |
web/src/app/components/calendar-sync-manual/calendar-sync-manual.component.html
Outdated
Show resolved
Hide resolved
ad 1) myslím, že to šedé pole by se z toho vyhodit dalo, ale ve stejném dizajnu je většina webu… a ty posuvníky jsem se z toho snažil dostat a nepovedlo se mi to, ať jsem se to tam snažil zmenšovat kde to šlo… ale jsou tam zbytečné. Když s tím, člověk posouvá, už tam žádný obsah není… a širší už to okno je:-) Ale můžu se ještě zkusit podívat, jestli s tím něco nesvedu:-) ad 2) Tahle konkrétní aplikace je asi 10 nejstahovanější na celém google play a myslím, že velká část lidí, co používá něco jiného než GCalendar se nikdy nesetkala s žádným sdíleným kalendářem, co by měli potřebu tam přidávat (většina bude prostě používat jen lokálně uložený kalendář)… ale možná se pletu.:-) ad 3) Nevím, jestli všude, ale minimálně části by zarovnání do bloku slušelo… mrknu na to. ad 4) Poznámky vyřeším ad 5) To je supr. Ze začátku jsem tam trošku bojoval s tím, aby se tam vůbec vešly i na počítači a nebyl jsem si jistý, jestli je nebude potřeba zmenšit pro menší obrazovku víc, než pro velkou… |
|
Něco jsem vyřešil, nevím, jestli to s tím zarovnání do bloku vypadá líp, nebo jestli to třeba na některé řádky nedávat… |
web/src/app/components/calendar-sync-manual/calendar-sync-manual.component.html
Outdated
Show resolved
Hide resolved
|
Přišel jsem na to, jak odstranin ten vnější posuvník… problémy tam dělal ten křížek… snad to neudělá nějaký nepořádek třeba na mobilu, ale snad by nemělo… |
|
Tak horizontálních posuvníků se mi povedlo zbavit (u toho druhého nevím, proč tam byl, ale povedlo se mi ho odstranit… co přesně dělá class="row"? Z nějakého důvodu to strkalo ten text moc do šířky. |
v noci byl nějaký problém se serverem a ten test nějak nenaběhl, netuším proč |
Myslím, že totéž je i u interní (resp. když jsem to zkoušel, tak se mi střídavě nezobrazil test interní a veřejné sekce) |
Změny
Na kartě „program“ přibyl nahoře (ještě nad programem) řádek s odkazem. Odkaz otevře okýnko s návodem, jak odebírat kalendář akcí včetně tří příkladů.
Testovací scénář
CTRL + F5⌘ Cmd + ⇧ Shift + RPo otestování vždy napiš feedback, buď
Approve review, neboRequest changes. Návod zde: Jak na testování