moravianlibrary / kramerius-for-ios

Kramerius client for iOS
Other
6 stars 0 forks source link

Listování ve svazku #13

Closed jankanka closed 8 years ago

jankanka commented 8 years ago

• U vetsiny dokumentu se po kliku na titul nezobrazi zadna prvni strana nebo obalka • Nekdy zafunguje ikona knihy vlevo dole • Listovani tahanim soupatka je moc hrube, clovek se netrefi tam, kam chce (a pro posun na dalsi stranku je to uplne nepouzitelne) • default velikost stranky je mala (mozna optimalizovana pro telefon?) • Prechod na dalsi stranku: pohyb prstem od boku do stredu displeje (tj. nativne dle papirove knihy) neprejde na dalsi stranku, ale posune stavajici po displeji • Tap na okraj stranky nekdy funguje, ale vetsinou ne (spis zafunguje jako schovani/zobrazeni horni a dolni listy) • Nekdy funguje dvoj-tap, ale ten zas otoci dve stran • Nekdy appka pri listovani spadne • Po kliku na ikonu knihy se zobrazi nahledy stranek vsechny s cislem 0 • Po kliku na stranku se casto zobrazi stranka "roztristena" img_3793 img_3794 img_3797

ovyhlidal commented 8 years ago

• Listovani tahanim soupatka je moc hrube, clovek se netrefi tam, kam chce (a pro posun na dalsi stranku je to uplne nepouzitelne) - posuvnik se nastavuje podle poctu stran - pokud ma nejaky titul hodne stranek, tak je posuvnik nepouzitelny.

• default velikost stranky je mala (mozna optimalizovana pro telefon?) - zkusim udelat nejake optimalice

• Prechod na dalsi stranku: pohyb prstem od boku do stredu displeje (tj. nativne dle papirove knihy) neprejde na dalsi stranku, ale posune stavajici po displeji - v soucasne verzi to nejde ani jinak udelat, jedna se o limitace prohlicky

• Tap na okraj stranky nekdy funguje, ale vetsinou ne (spis zafunguje jako schovani/zobrazeni horni a dolni listy) - zvetsim dotykove zony na krajich a tim by se tomuto melo predejit

• Nekdy funguje dvoj-tap, ale ten zas otoci dve stran - dvoj tap 2x klikne na otoceni stranky, proto se stranka otoci 2x.

• Nekdy appka pri listovani spadne - opraveno

• Po kliku na ikonu knihy se zobrazi nahledy stranek vsechny s cislem 0 - bude opraveno

• Po kliku na stranku se casto zobrazi stranka "roztristena" - pravdepodobne nejaky problem ve ctecte, zkusim se podivat proc se to deje

honza-rychtar commented 8 years ago

Listovani tahanim soupatka je moc hrube, clovek se netrefi tam, kam chce (a pro posun na dalsi stranku je to uplne nepouzitelne) - posuvnik se nastavuje podle poctu stran - pokud ma nejaky titul hodne stranek, tak je posuvnik nepouzitelny.

Soupatko nema slouzit k prechodu na predchozi/dalsi stranku - to u vicestrankovych dokumentu proste fungovat nebude. Slouzi jako info o aktualni pozici v dokumentu a k prechodu o velke useky, napr. nalistovani na konec dokumentu.

Nekdy funguje dvoj-tap, ale ten zas otoci dve stran - dvoj tap 2x klikne na otoceni stranky, proto se stranka otoci 2x.

Pokud je dvoj-tap matouci (ja bych treba cekal zoom), tak by bylo dobre nastavit nejaky threshold (napr. 500 milisekund), kdy se po tapu dalsi tap ignoruje.

Prechod na dalsi stranku: pohyb prstem od boku do stredu displeje (tj. nativne dle papirove knihy) neprejde na dalsi stranku, ale posune stavajici po displeji - v soucasne verzi to nejde ani jinak udelat, jedna se o limitace prohlicky

Nevim jak to funguje na iOS, ale urcite je tam nejaka view hierarchy a callback propagation, kdy v konteineru, ktery obsahuje prohlizecku, muzu zachytavat gesta a pokud simuluji swipe (tah zleva do prava), tak zastavit propagaci gesta do prohlizecky a interpretovat to tak, ze otocim stranku. Na Androidu taky v prvnich verzich nebyla podpora swipe, takze to neni tak nutne, zejmena pokud se bude prohlizecka prepisovat za nativni. Nicmene, otaceni pres swipe byla nejpozadovanejsi nova vlastnost. Uplny swipe, tj. nacteni i stranky pred a za a hladky posun pri posunu muze byt slozitejsi. Ale i jednoduche zachyceni gesta a nacteni predchozi/dalsi stranky bez vizualne naznaceneho prechodu by mohlo stacit.

ovyhlidal commented 8 years ago

Odstranil jsem label bug z tohoto issue, protoze vetsina problemu byla opravena. Swipe nechavam jako enhancement.

ovyhlidal commented 8 years ago

Tohle bude vyreseno v okamziku, kdy se dodela ios nativni ctecka, takze si myslim, ze muzeme tohle issue uzavrit.