Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

v zobrazení kalendáře nefungují fasety #124

Closed
zabak opened this issue Jun 28, 2020 · 32 comments
Closed

v zobrazení kalendáře nefungují fasety #124

zabak opened this issue Jun 28, 2020 · 32 comments
Labels
chybí komentář NLCR Dodat data k issue

Comments

@zabak
Copy link
Collaborator

zabak commented Jun 28, 2020

Pokud mám titul zobrazený jako kalendář, tak systém ignoruje klikání na fasety. Navíc není možné zobrazit všechny položky v rámci dne. Vzhledem k #123 navrhuji toto zobrazení zcela zrušit, pokud nepřijdeme na to, jak v něm zobrazit potřebné informace a nebude jasný use-case.

image

@zabak
Copy link
Collaborator Author

zabak commented Jun 28, 2020

Kdyby fungovaly fasety a bylo vidět všechny řádky v rámci dne, tak by to bylo ideální pro kontrolu. Pak už zbývá jen dořešit #123

@albertoh
Copy link
Contributor

albertoh commented Jul 8, 2020

Facety bych tady schoval. Kalendar je vazan na metatitul

albertoh pushed a commit that referenced this issue Jul 8, 2020
@zabak
Copy link
Collaborator Author

zabak commented Jul 8, 2020

@Foltom @korhon prosím zamyslete se nad tímto víc. To co navrhuje Alberto sice řeší problém, ale pokud se #123 vyřeší odstraněním menu a tady se odstraní fasety, tak ten kalendář nebude k ničemu, protože se nezobrazí ke každému dni všechno. Potřebujeme pak ale takový kalendář?

@zabak
Copy link
Collaborator Author

zabak commented Aug 19, 2020

V Chrome se nezobrazuje info typu "2 další vydání", zatímco v Opeře to funguje

@korhon
Copy link

korhon commented Aug 19, 2020

kalendar

@zabak
Copy link
Collaborator Author

zabak commented Aug 19, 2020

image
takhle to vypadá v Chrome

@korhon
Copy link

korhon commented Aug 19, 2020

Odkaz "2 další vydání" nic neukazuje...

@korhon
Copy link

korhon commented Aug 19, 2020

kalendar 2

@zabak
Copy link
Collaborator Author

zabak commented Aug 19, 2020

Bylo by myslím lepší, kdyby kalendář neměl fixní výšku řádků ale řádky byly vysoké tak, aby se tem vešla všechna vydání. Zároveň prosím o vysvětlení, co znamenají barvy položek v kalendáři (resp. podle jakých pravidel se obarvuje) a podle jakých pravidel se vytvářejí samostatné řádky.

@Kretiss
Copy link
Collaborator

Kretiss commented Sep 2, 2022

@zabak Nasazeno na našem testu. Opravil jsem výšku buněk, aby byly vidět všechny řádky, včetně "2 další vydání". U toho prokliku byl problém v tom, že se použilo špatné datum, tím pádem to nemělo co zobrazit (jde to vidět i na tom screenu, že je tam 2020). Každý den v sobě má vypsané jeho issues, přičemž zelené řádky jsou issues, které jsou označeny stavem "OK".

@zabak
Copy link
Collaborator Author

zabak commented Sep 2, 2022

@Kretiss teď se mi v kalendáři nezobrazuje nic - testováno na https://permonik.inqool.cz/permonik/calendar/2081c9cc60d1101b53c4a0ce348195ee/month/19530101

@zabak zabak added the Opravit Bylo otestováno. Fukcionalita je připravena nekorektně, či nefunguje na 100%. label Sep 2, 2022
@Kretiss
Copy link
Collaborator

Kretiss commented Sep 2, 2022

@zabak To se mi tak zobrazovalo taky a popravdě momentálně nevím, proč tomu tak je.. ale tady to vidět jde https://permonik.inqool.cz/permonik/calendar/2081c9cc60d1101b53c4a0ce348195ee/month/19660120

EDIT: na produkci se to zobrazuje taktéž špatně: http://195.113.132.165:8080/permonik/calendar/2081c9cc60d1101b53c4a0ce348195ee/month/19530103, přičemž jsme kolem kalendáře zatím nic nenasazovali

@zabak
Copy link
Collaborator Author

zabak commented Sep 2, 2022

Aha, tady vidím problém - netušil jsem, že je tam prolink na x dalších vydání a jiná forma zobrazení. Osobně bych preferoval, aby se v tom kalendáři ve formě tabulky (1 týden 1 řádek) zobrazily všechna vydání (aby se výška řádku přizpůsobila). Ta forma zobrazení s každým dnem na novém řádku mi připadá nadbytečná a nepřehledná. Prosím o vyjádření alespoň @korhon a @BrozekAles

@zabak
Copy link
Collaborator Author

zabak commented Sep 2, 2022

Znovu jsem otevřel #123 ale pokud bychom se rozhodli zrušit tu stránku kde je jeden den na řádku, tak se to #123 může zase zavřít.

@BrozekAles
Copy link

Kalendář ve formě tabulky nevyužívám, resp. jsem ani o něm dosud nevěděl, a když se na to teď dívám, tak ani nechápu, co lze z kalendáře ve formě tabulky vyčíst, co znamenají barvy položek apod.

@zabak
Copy link
Collaborator Author

zabak commented Sep 3, 2022

@BrozekAles barvy jsou vysvětlené zde: #124 (comment) - myslím, že po úpravě zobrazení by to kalendářové zobrazení mohlo být vhodným výchozím zobrazením pro uživatele. Kdyby tam byla vidět všechna ten den vyšlá vydání s možností rozkliknout si kde dané vydání mají a další detaily, tak vlastně nepotřebujeme nic víc.

@BrozekAles
Copy link

Aha, zmátl mne ten výraz issues, který užívám jen pro issues na Githubu :-) Takže zelená barva značí, že daný výtisk je O.K. :-)

@Kretiss
Copy link
Collaborator

Kretiss commented Sep 8, 2022

@BrozekAles @korhon @zabak Jaké je teda finální vyjádření? Ať s tím issue nějak pohnem. Viděl bych to tak, že se všechny vydání zobrazí v tom kalendáři a ten prolink na další vydání se zruší. Svým způsobem je to taková zbytečná duplikace. Funkcionalitu tam můžeme nechat, kdyby se v budoucnu něco změnilo.

@korhon
Copy link

korhon commented Sep 9, 2022

Mně připadá kalendář velmi smysluplný, proklik na další vydání nevidím jako potřebný. Podle mě by to mělo fungovat tak, že se přímo v kalendáři vypíší všechna dostupná vydání včetně zobrazení symbolů mutačních vydání.
V tom barevném řádku by potom byla informace ve tvaru:
číslo vydání - mutace - značky mutačních vydání - počet exemplářů.
Např.
3 Praha * 2
3 Praha ** 1
3 Ostrava 3
3 Ostrava *** 1
3 Brno * 3
atd.
Pak by to bylo zdaleka nejpřehlednější zobrazenáí informace o tom, kolik je toho dochováno.
Kam by měl vést proklik z těchto řádků, to zatím nevím. Mohlo by se zobrazit okno, kde se zobrazí čárové kódy exemplářů (pokud je jich víc) a pak proklik do tabulky editace svazku.

@BrozekAles
Copy link

Mně připadá kalendář také velmi smysluplný a připojuji se k názoru kol. Korhoně. Jen mi připomněla jeho zmínka o čárových kódech exemplářů, že - snad se nepletu - nelze zatím podle čárových kódů vyhledávat a při opravách a běžném prohlížení dosavadních výsledků naší práce by pomohlo, kdybychom nemuseli hledat jen podle let a vlastníka, ale i podle čárového kódu.

@zabak
Copy link
Collaborator Author

zabak commented Sep 9, 2022

@BrozekAles pokud není na to vyhledávání založené issue, tak ho prosím založte.
@Kretiss udělejte to dle vašeho návrhu, textaci a linkování dle toho co píše @korhon

@Kretiss
Copy link
Collaborator

Kretiss commented Sep 15, 2022

@zabak @korhon @BrozekAles Nasadil jsem na náš test, prosím o kontrolu, jestli to může být tak jak to je. Mám ale dotaz k těm datům. Ty exempláře byly vždycky vidět hned po tom, co se vytvořil nějaký svazek a exempláře se vyplnily? Příjde mi totiž, jak kdyby se to předtím ukládalo ještě dodatečně, aby to šlo v kalendáři vidět, protože se data vytahují z databáze issue, ale mnou vytvořený nový metatiul, včetně svazku a exemplářů je uložený jinak, v issue ho nenajdu. Poslední záznam z issue je 10. 8. 2020, a to Lidové noviny - Vydání Brno, viz https://permonik.inqool.cz/permonik/calendar/uuid:b29ea210-2651-11dd-8334-000d606f5dc6/month/19360105. Neměnila se kolem tohoto data nějaká funkcionalita?

@Kretiss Kretiss added ready for test and removed Opravit Bylo otestováno. Fukcionalita je připravena nekorektně, či nefunguje na 100%. labels Sep 16, 2022
@zabak
Copy link
Collaborator Author

zabak commented Sep 18, 2022

@Kretiss Je to divné - Když se podívám na LN na tom linku, tak to něco zobrazuje, ale na hlavní stránce to zobrazuje, že Lidové noviny mají 0 exemplářů. Vypadá to na nějakou nekonzistenci dat. Je tam i nekonzistence druhým směrem: Československý sport: Když ho rozkliknu z hlavní stránky, tak se mi zobrazí výpis čísel, ale tam když kliknu na datum, tak kalendář (rok 1953) se zobrazí prázdný. Rok 1966 tam ale je.

Mimo to pár věcí:

  • zobrazení v kalendáři přestalo deduplikovat. Pokud mám identické řádky, měl by být zobrazen jen jeden se dvojkou na konci. Identické je myšleno i co se barvy týče, takže v přiloženém příkladu by se sloučily jen řádky
    1,2 a 4. Po kliknutí na takový řádek uživatel uvidí, kde se daná čísla nacházejí a případně si některé rozklikne. image
  • Z důvodu přehlednosti by měly být řádky v kalendáři řazeny podle abecedy, aby nebylo promíchané Brno a Praha navzájem
  • ČS sport, 3.1.1966 - první řádek nemá žádné čislo a je zelený - to je divné, očekával bych že tam bude číslo, když je tam ten řádek. Je to chyba v datech?
  • Domluvili jsme se, že zobrazení kalendáře v podobě seznamu odstraníme - je proto potřeba odstranit i jeho ikonu z menu v kalendáři (prostřední ikona) -

image

  • V menu výše je tlačítko na přechod do přehledu exemplářů. Když na ně kliknu, tak se dostanu do seznamu, který je ale prázdný a musím dát reload, aby se mi něco zobrazilo.
  • V menu v přehledu exemplářů ale není tlačítko na zobrazení kalendáře (to by mělo přejít na měsíc ve kterém je vydaný první exemplář ze zobrazeného seznamu
  • Týden musí začínat pondělkem - teď je to tak, že když přejdu na další týden a ten začíná středou, tak prostě daný řádek začíná středou.
  • Bude možno listovat dopředu a dozadu nejen o měsíc ale i o rok a 10 let, podobně jako při zobrazení tabulky, viz Šipky doprava a doleva vedle počtu nalezených záznamů #73

@zabak zabak added the Opravit Bylo otestováno. Fukcionalita je připravena nekorektně, či nefunguje na 100%. label Sep 18, 2022
jahhoo pushed a commit that referenced this issue Oct 6, 2022
jahhoo pushed a commit that referenced this issue Oct 6, 2022
@Kretiss Kretiss added the odloženo Bude pravděpodobně řešeno v budoucnu. Realizace je na něčem závislá. label Oct 7, 2022
jahhoo pushed a commit that referenced this issue Nov 4, 2022
@Kretiss Kretiss removed the Opravit Bylo otestováno. Fukcionalita je připravena nekorektně, či nefunguje na 100%. label Nov 25, 2022
@zabak
Copy link
Collaborator Author

zabak commented Jan 17, 2023

Mělo by být v nové verzi

@zabak zabak added ready to implement and removed odloženo Bude pravděpodobně řešeno v budoucnu. Realizace je na něčem závislá. labels Jan 17, 2023
@Kretiss
Copy link
Collaborator

Kretiss commented Jul 17, 2023

@zabak @korhon @AnnaCajthamlova potřeboval bych přesně specifikovat, co má být v novém kalendáři zobrazeno v modálu po kliknutí na daný řádek dne (např řádek, ve kterém jsou 2 exempláře).

Momentálně mám nachystané takové okno:
obrazek

Co dále tam teda má být?

  • Odkaz na katalog (jaký?), odkaz na digitalizát -> potřebuji dodat url adresy
  • Přehled svazku, tak jak je tomu ve stávajícím permoníkovi, ale pro každý exemplář (svazek) zvlášť? Viz:
obrazek

@Kretiss Kretiss added chybí komentář NLCR Dodat data k issue and removed ready to implement labels Jul 17, 2023
@zabak
Copy link
Collaborator Author

zabak commented Jul 17, 2023

Link na digitalizát určitě ano, pokud už víme jak/kam ho budeme ukládat. Ale digitalizovaný dokument je samostatná entita, z našeho hlediska. Z hledika jednoznačnosti by mělo smysl aby v záhlaví modalu byl metatitul, pod tím datum a pak řádek pro každý výtisk, kde by byla Mutace - vydání - název - podnázev - zkratka knihovny jako link na objednání na knihovny.cz pomocí čárového kódu. @korhon @AnnaCajthamlova @BrozekAles co říkáte?

@annavandasova
Copy link
Collaborator

Ano souhlasím s @zabak a prosím ještě o vyjádření i J. Ludvíčka

@Kretiss
Copy link
Collaborator

Kretiss commented Jul 18, 2023

@zabak Můžu poprosit o ten link na knihovny.cz? Zkoušel jsem to jejich hledání např takhle: https://www.knihovny.cz/Search/Results?lookfor=314800329591&type=AllFields&limit=20, ale nic mi to nenajde

@annavandasova
Copy link
Collaborator

@zabak ověřit propojení na knihovny.cz

@Kretiss
Copy link
Collaborator

Kretiss commented Oct 7, 2024

@zabak ping

@zabak
Copy link
Collaborator Author

zabak commented Oct 7, 2024

prověřím

@Kretiss
Copy link
Collaborator

Kretiss commented Oct 14, 2024

@zabak Zavírám toto issue, protože nabylo a řeší se v něm hodně věcí najednou. Zůstává prověřit link na knihovny.cz, založím na to nové issue.

@Kretiss Kretiss closed this as completed Oct 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chybí komentář NLCR Dodat data k issue
Projects
None yet
Development

No branches or pull requests

7 participants