proarc / proarc-client

3 stars 0 forks source link

Zvýraznění repre strany #485

Open luckajirku opened 10 months ago

luckajirku commented 10 months ago

Bylo by dobré, kdyby bylo vidět v tabulce i náhledech, že daná strana je označená jako reprezentativní - teď to nezjistím jinak než ve formuláři té strany. Třeba ji vytučnit - jak řádek v tabulce, tak ten zkrácený popis u náhledů... - nebo jestli někdo vymyslí něco lepšího...

ZdenkaSera commented 10 months ago

@luckajirku @pkudela Přidávám label redesign

ZdenkaSera commented 5 months ago

@pkudela Pošle návrhy

SykoraLukas commented 5 months ago

Přidat pole do indexu SOLRu a na jeho základě zvýraznit řádek ve stromě a v náhledech.

SykoraLukas commented 5 months ago

Na API hotové, nutné doplnit pole do Klienta a na jeho základě zvýraznit řádek. @pkudela vymyslí, jak se to udělá.

SykoraLukas commented 4 months ago

@luckajirku navrhuje tučné písmo.

ZdenkaSera commented 1 month ago

@pkudela Prosím, podíval by ses na tohle issue? Děkuji moc.

pkudela commented 1 month ago

hotovo

image image
pkudela commented 1 month ago

@pkudela Prosím, podíval by ses na tohle issue? Děkuji moc.

uz se stalo, ani jsem si nevsimnul, ze jsi pred tim psala, je to hotove

ZdenkaSera commented 1 month ago

@pkudela Díky :-) Nechci mluvit za @luckajirku, ale možná by se to zvýraznění hodilo i v úložišti ...?

ZdenkaSera commented 1 month ago

@pkudela @luckajirku Beru zpátky, je to i tam, byla jsem přihlášená na visk2024 místo na masteru. Super

ZdenkaSera commented 1 month ago

@pkudela Naopak to není v paginaci v importu - myslela jsem si, že Tvé ukázky byly odtud.

pkudela commented 1 month ago

@pkudela Naopak to není v paginaci v importu - myslela jsem si, že Tvé ukázky byly odtud.

Ted jsem zmaten, posli mi konkretni url. @ZdenkaSera mas na mysli toto viz. screenshot?

image

ZdenkaSera commented 1 month ago

Vysvětleno po telefonu - v paginaci sice můžu nastavit reprezentativní stranu (to už je tam dlouho), ale zatím nelze zvýraznit tučně v přehledu skenů, probíhá úprava na api, která to umožní.

SykoraLukas commented 1 month ago

Podpora na API při importu, @pkudela doplníš do importu?

pkudela commented 1 month ago

Podpora na API při importu, @pkudela doplníš do importu?

Jasne, jdu na to.

pkudela commented 1 month ago

hotovo, je nasazeno na server k testovani, uz se oznacuje repre strana i v importu

image image
ZdenkaSera commented 1 month ago

Prima, moc díky. Konečně slovo k testu budou mít zkušení uživatelé :-)

ZdenkaSera commented 1 month ago

Tohle se do vydané verze 2.3.1 nedostalo (nasazeno dnes v KNAV).

ZdenkaSera commented 1 month ago

Doplňuji, že v úložišti je to ok, zvýraznění chybí jen v paginaci v importu.

pkudela commented 1 month ago

Doplňuji, že v úložišti je to ok, zvýraznění chybí jen v paginaci v importu.

Ahoj Zdenko, dival jsem se na to jeste s Lukasem, jestli oba chapeme to same a v importu to je viz., screenshot nize z masteru, kde je posledni vydana verze. Pokud nemas na mysli toto a mas na mysli neco jineho, posli prosim nejaky screenshot, at to dokazeme s Lukasem nasimulovat.

image

image

ZdenkaSera commented 1 month ago

@pkudela Ano, jde o tohle. V pátek jsem jen v rychlosti testovala nějaký základ a zřejmě jsem si nastavení pod rukou smazala - teď jsem to zkusila znovu a zvýraznění je vidět i v importu. Díky, omlouvám se.

luckajirku commented 1 month ago

za nás OK, možno zavřít

ZdenkaSera commented 1 month ago

@pkudela Připravit návrh - zvýraznění rámečku kolem náhledu reprezentativní strany.

pkudela commented 1 month ago

Ahoj, posilam par screenshotu jak resit zobrazeni repre strany bez vytucneneho pisma zatim jen pro obrazkove nahledy, dejte vedet, ktery smer by vam vyhovoval, zatim to nenasazuji na server, ale mam to pripravene. Pravda je, ze najit, ktera strana je reprezintativni podle tucneho pisma da nekdy docela praci, prave jsem si to vyzkousel, takze dle meho, jine reseni nez jen tucne pismo je spravne, aby to bylo na prvni pohled videt. Osobne se klonim k variante 2 a 3, ale rozhodnuti je na vas.

image

image

image

image9)

luckajirku commented 1 month ago

@pkudela za nás je to tučné písmo ok, tohle je to fakt divočina. Takhle výrazné označení evokuje stranu, se kterou se právě pracuje (nakliknutá, která je ve velkém náhledu). Schválně vyzkoušej, jak to bude vypadat v importu v dávce periodika, kde bude repre třeba každá pátá strana...

pkudela commented 1 month ago

@pkudela za nás je to tučné písmo ok, tohle je to fakt divočina. Takhle výrazné označení evokuje stranu, se kterou se právě pracuje (nakliknutá, která je ve velkém náhledu). Schválně vyzkoušej, jak to bude vypadat v importu v dávce periodika, kde bude repre třeba každá pátá strana...

Aha, to jsem nevedel, ze muze byt oznaceno vice repre stran, v tom pripade ano, zkusim vymyslet jeste neco jineho, decentnejsiho.

luckajirku commented 1 month ago

ono v úložišti je jen jedna na objekt - ale v importu, když máš dávku periodika zatím nerozřazenou do čísel, jich může být fakt hodně - míváme běžně 4stránková čísla...

pkudela commented 1 month ago

OK, bude to muze byt jen takovy marker - ruzek, ktery jsem tedy zmensil, muze byt v jakekoliv barve, nebo to muze byt treba nejaka ikona, muze by i jina, neco co nebude rusit, ale na prvni pohled budes videt, ze je to reprezentativni strana, jeste se zkusim zamyslet nad jinou variatntou, pripadne muze zustat jen tucne pismo.

image

image

luckajirku commented 1 month ago

tyhle menší, decentnější rožky by šly. a ještě kdybys je zkusil tou tmavě šedou jako jsou ikony...

pkudela commented 1 month ago

tyhle menší, decentnější rožky by šly. a ještě kdybys je zkusil tou tmavě šedou jako jsou ikony...

Co takto? Roh jsem dal sedivou barvou a aby navazoval na hranu nahledu a nebyl trochu mimo obrazek a ty jednotlive polozky byly vice oddelena dal jsem tam ohraniceni jemnym odstinem sedive, tak dej vedet, jestli vyhovuje.Tucne pismo jsem dal pryc, aby bylo videt, jestli je to dostatecne poznat.

image

a jeste svetlejsi sediva, ale to uz se mi zda, ze neni moc zretelne videt image

pkudela commented 1 month ago

a takto to vypada u tech nahledu na sirku

image

pkudela commented 1 month ago

a jeste s decentejsi sedivou ohraniceni, to je posledni

image

luckajirku commented 4 weeks ago

...Tucne pismo jsem dal pryc, aby bylo videt, jestli je to dostatecne poznat.

a můžeš ho tam prosím tě vrátit? aby se na něj pak nezapomnělo:-). my to další zvýraznění (rožky, rámečky) bereme jako něco navíc (pro ty, kterým nestačí tučné písmo) k tomu tučnému, ale ne místo tučného. u toho je fajn, že se dá použít na obou místech - jak v řádkovém zobrazení, tak v náhledech - vidím tučné písmo, vím, že je to repre.

pkudela commented 4 weeks ago

...Tucne pismo jsem dal pryc, aby bylo videt, jestli je to dostatecne poznat.

a můžeš ho tam prosím tě vrátit? aby se na něj pak nezapomnělo:-). my to další zvýraznění (rožky, rámečky) bereme jako něco navíc (pro ty, kterým nestačí tučné písmo) k tomu tučnému, ale ne místo tučného. u toho je fajn, že se dá použít na obou místech - jak v řádkovém zobrazení, tak v náhledech - vidím tučné písmo, vím, že je to repre.

Neni problem, je to tam nazpatek viz. screenshot, kterou variantu z tech odstinu sediveho ruzku a ohraniceni preferujes z toho co jsem poslal na screenshotech vyse? Ze bych to pak dal uz k testovani.

image

luckajirku commented 4 weeks ago

za nás ta poslední nejsvětlejší (ale to asi nepřekvapí:-))

pkudela commented 4 weeks ago

za nás ta poslední nejsvětlejší (ale to asi nepřekvapí:-))

Ne, neprekvapuje :-), cili zapracuju do globals a dam na server k testovani.

pkudela commented 4 weeks ago

Hotovo, nasazeno, muzete testovat, dejte vedet, jestli takto vyhovuje a pokud ano, prosim zavrit toto issue.

https://proarc-master.inovatika.dev/repository/uuid:46132287-9d88-4536-a09f-bf36d1a4aff5 https://proarc-master.inovatika.dev/import/edit/825 https://proarc-master.inovatika.dev/import/edit/810

nezbedova commented 4 weeks ago

@pkudela je to nastavené i pro STT modely? Na našich "specialitkách" to nevidím.

https://proarc-master.inovatika.dev/import/edit/829

obrazek

obrazek

pkudela commented 4 weeks ago

@pkudela je to nastavené i pro STT modely? Na našich "specialitkách" to nevidím.

https://proarc-master.inovatika.dev/import/edit/829

obrazek

obrazek

Asi je to tim, ze mi to nevraci api, @SykoraLukas muzes se na to prosim mrknout? Nevidim to tam viz. screenshot.

image