proarc / proarc-client

3 stars 0 forks source link

Hledat - editace #370

Closed SykoraLukas closed 1 year ago

SykoraLukas commented 1 year ago

Tabulka

Titulní metadata

pkudela commented 1 year ago

@luckajirku ahoj, posilam odkaz na ikony, lze v nich i vyhledavat, narp. na slovo validace to vyhleda nekolik ikon, ale ne vzdy to vylozene souvisi s nazvem ikony -> https://fonts.google.com/icons?icon.style=Filled&icon.set=Material+Icons Prosim, dodejte ikonu pro tlacitko validace vzi. bod 6, pripadne specifikujte dalsi pripadne zmeny ikon jako napr. budiky v MODS XML viz. body.

Prozatim jsen vybral pro validaci tuto ikonu.

image
luckajirku commented 1 year ago

@pkudela tuhle ikonu určitě ne, nic dalšího se šipkama tam, kde to není nutné. Původně jsme mysleli, že se prohodí ty ikony pro validaci a vybrat vše. Ale kulatý tvar by byl možná fajn - pokud se tam nedá ta, co je teď pro vybrat vše, tak za nás pro validaci jedna z těchhle (nejlíp ta první, ale inverzně, aby výplň byla bílá):

validace

pkudela commented 1 year ago

ahoj @luckajirku supeer, nevadi, dulezite, ze jste si vybrali, cili dam tu prvni. Jeste se chci ujistit u tohoto bodu - "Tlačítko Vybrat vše bude nově ikona Validace" - to souvisi s bodem "Vybrat všechny objekty převést do 3 teček" - jestli je to to same, tak vybrat vse nebude mit zadnou ikonu, protoze je to v menu navigaci, kde se zobrazuji cele nazvy akce viz. screenshot, tak se jen ptam, jestli jsem to pochopil dobre, ze se jedna o tu samou polozku zminenou ve dvou vyse uvedenych bodech

image
luckajirku commented 1 year ago

... jestli je to to same, tak vybrat vse nebude mit zadnou ikonu, protoze je to v menu navigaci, kde se zobrazuji cele nazvy akce ...

no v Hledat ne, tam to má být schované, ale v Importu ikonu pro Vybrat vše potřebovat budeš, tam by měla být na ráně

pkudela commented 1 year ago

Jestli dobre rozumim, tak na adrese http://localhost:4200/repository/uuid: ...... bude "vybrat vsechny objekty" schovane pod 3 tecky, jak jsem poslal v tom screenshotu, ale na adrese http://localhost:4200/import/edit/.... ma byt viditelna? Pokud ano, tak v repo bude pod 3 teckama 2 polozky - "zmena pozice"a "vybrat vsechny objekty" a v importu jen 1 polozka - "zmena pozice", je to tak :-)?

luckajirku commented 1 year ago

ty odkazy mi nefungují, ale jo, to jsme se přeci bavili už na schůzce, že to teď tak vychází, že v importu bude pod ... jen 1 položka. Na druhou stranu nemyslím, že to tak bude navždycky, časem se vyvrbí nějaké fce, které tam přijdou.

pkudela commented 1 year ago

jo, to je v pohode, ja se jen chtel ujistit, jestli to chapu spravne, protoze obe url pouzivaji stejnou listu, takze to musim nejak v kodu podminit, jinak jak pises, casem tam muze byt neco navic, pripadne treba zjistite, ze se casem jeste neco pod 3 tecky presune

luckajirku commented 1 year ago

ale tak těch rozdílů tam máš přeci víc, v Hledat je převod stran, v Importu ne, naopak zas v Importu je Označit řadu, ale v Hledat ne...

pkudela commented 1 year ago

ikona validace vymenena

image
luckajirku commented 1 year ago

@pkudela jen taková drobnost:-) - není ta ikonka validace dole trochu uřízlá?

pkudela commented 1 year ago

ano, opraveno

image
luckajirku commented 1 year ago

a ještě návrh na změnu pořadí ikon + na změnu pluska k vytváření objektů, aby bylo výraznější - je to tady hodně důležitá fce, tak ať je pořádně vidět ikony_hledat

pkudela commented 1 year ago

hotovo, je to tak spravne?

image
luckajirku commented 1 year ago

A ještě bych se teda zbavila toho slova Home. Zbytečně zabírá místo - a o to míň znaků se tam pak vejde z názvu. Samotná ikona domečku úplně stačí - pokud už teda musí být na tomhle místě ta fce, když to samé dělá i klik na logo PA.

pkudela commented 1 year ago

zatim jsem odstranil jen text

pkudela commented 1 year ago

Posledni 4 body bych potreboval zitra osvezit.

katerinasirova commented 1 year ago

@nezbedova a @SmejkalovaAnna a já se domluvit na použití ikon budíčků

pkudela commented 1 year ago

@nezbedova a @SmejkalovaAnna, nejaky posun ohledne budicku?

SmejkalovaAnna commented 1 year ago

@pkudela Petře já se omlouvám, úplně se nepamatuji, co jsme přesně chtěli s těmi budíky, můžete mi zavolat prosím do Jenštejna na 315315382? Abychom si to ujasnili...

pkudela commented 1 year ago

@SmejkalovaAnna ahoj, dneska jsem Ti volal na toto cislo, ale nedovolal jsem se, zkusim to jeste zitra.

pkudela commented 1 year ago

Prosim, muzete mi nekdo poslat link na testovaci server, kde je problem v nize uvedenem bode?

Udělat klikací cestu o objektu (Monografie, Periodika, Zvuk)

Proklikaval jsem to, Alberto take a zda se mi, ze vsude klikaci cesta je, dekuji.

luckajirku commented 1 year ago

Jestli se nepletu, tak tam byl problém, jak se vracet na metadata titulu. Přijdu do periodika, vidím metadata titulu. Nakliknu nějaký ročník - a už vidím metadata ročníku a nemám jak se vrátit na metadata titulu. To samé u monografie, když nakliknu nějakou stranu, už nemám jak se dostat na metadata titulu. Svádí to k tomu kliknout na název - ale ten není v tu chvíli klikací, protože právě tu úroveň má člověk otevřenou. V těchhle případech funguje dát Obnovit - a metadata titulu jsou zpátky - ale není to moc intuitivní. Nemohlo by to fungovat tak, že ta titulová úroveň by byla neklikací pouze v případě, že by nebylo nakliknuté nic pod ní, a ve chvíli, kdy označím stranu, ročník atd. a dostanu se na metadata podřízeného modelu, už se odkaz na titul zaktivní?

pkudela commented 1 year ago

Udělat klikací cestu o objektu (Monografie, Periodika, Zvuk) - toto je take hotovo, nasazeno na serveru, prosim otestovat

luckajirku commented 1 year ago

ještě k bodu "Přidat šipku Dolu a upravit do pořadí jako ve starém klientovi" - šipka tam je, ale popisek při najetí myší je stejný jako u šipky nahoru:-).

SmejkalovaAnna commented 1 year ago

@pkudela Už asi vím ty budíky. Navrhuji:

obrazek Zároveň, by se při editaci formuláře podbarvoval element v MODSu a po uložení formuláře by se zároveň uložila i změna v MODSU.

luckajirku commented 1 year ago

jo a ještě jedna věc k té šipce dolů - na téhle liště zatím chybí - když už se přidává do těch panelů, tak by měla i sem spikadolu

pkudela commented 1 year ago

Provedl jsem nasledujici zmeny:

Co se tyce tech budiku, pockame s Albertem na vyjadreni @luckajirku , jinak stav ikony "save" je opraveny, nyni je disablovana jak si chtela v tom prvnim bode @SmejkalovaAnna

luckajirku commented 1 year ago

Budíky bych reklamovala u Lukáše, že to nedostatečně zapsal:-). Já se od nich distancuju, šlo to trochu mimo mě a nepamatuju si to. Byla tam nějaká diskuse k tomu, že se ty budíky pletou, taky jakou verzi přesně to načte atd. Myslím, že když se @SykoraLukas zasnaží, tak si to vybaví. A nebo to rozsekne @katerinasirova, ta se k tomu na poslední schůzce hlásila.

luckajirku commented 1 year ago

ještě k té ukládací ikoně - v metadatech strany u formuláře je taky vysvícená pořád, asi by se to mělo změnit i tam

SmejkalovaAnna commented 1 year ago

@luckajirku Dobře, díky, já si to právě taky moc nepamatovala, ale tohle jsem vymyslela, jak by to mohlo fungovat relativně intuitivně. Ještě poprosím zítra @ZdenkaSera, aby se mrkla, jestli jsem tam nevymyslela nějaký nesmysl....

SmejkalovaAnna commented 1 year ago

@pkudela Petře za mě perfektní 👍 Dokonce je to tak chytrý, že když něco smažu v MODSu a napíšu tam to samý, tak to pozná, že už to tam bylo a zneaktivní uložení, to jsem fakt nečekala 🥇

pkudela commented 1 year ago

Ja tedy budicky zatim neresim do doby, kdy dostaneme jasne zadani na kterem se vsichni shodnou, Alberto to od @SmejkalovaAnna procital a jestli to pochopil spravne, tak by to nebylo zas tak jednoduche, tak abysme nedelali neco zbytecne z nepochopeni, souhlasis Anicko?

Co se tyce te ukladaci ikony @luckajirku Lucko, tak na to se podivame a pripadne opravime, dik za upozorneni.

SmejkalovaAnna commented 1 year ago

@pkudela Souhlasím.

pkudela commented 1 year ago

Disable ikony "save" na strance Alberto opravil.

albertoh commented 1 year ago

Jsou k testovani

katerinasirova commented 1 year ago

K budíčkům - když jsme to testovali, tak mi přišlo, že tak jak to bylo nastavené všechny ikony dávaly smysl, ALE podle mě to nikdo takhle nepoužíval. Nevím, jestli někdo měl ideu, že se mu z nějakého důvodu změny nemají propisovat z XML do MODS. Ale za nás to nikdo nikdy nepoužil. Přemýšlím, jestli je dobré mít na první dobrou do kliknutí v XML možnost hned něco měnit. Tam se totiž může ukliknout a napsat něco někam, ani nevíš jak. Takže za mě by mohla zůstat jen ikona upravit a uložit (a ta disketa už se může chovat tak jak bylo popsáno výše). Pokud bude fungovat podbarvení, budeme jen rádi, práce se pěkně zpřehlední. Dávám dotaz na ostatní, jak s ikonou upravit? Jinak souhlasím s @SmejkalovaAnna - viz níže.

@pkudela Už asi vím ty budíky. Navrhuji:

  • Aby v okně Formuláře, bylo primárně nastavená ikona diskety jako neaktivní. Ve chvíli, kdy klepnu do formuláře, že chci něco upravit, tak by naskočila aktivní a po odklepnutí zase změnila na neaktivní (aby bylo jasno, že jsem to uložila nebo že je to v editaci):

obrazek Zároveň, by se při editaci formuláře podbarvoval element v MODSu a po uložení formuláře by se zároveň uložila i změna v MODSU.

  • v okně MODS - vyhodit oba budíky, nechat jen obnovit a disketku, která bude zase neaktivní pokud nezačnu měnit formulář nebo samotný MODS, zaktivní se pro uložení změn. Zároveň zase pokud bude něco opravováno v MODSU a bude na to chlíveček, promítne se to v reálném čase do formuláře. (teď to jde jen obráceně - z Formuláře se to v reálu propisuje do MODSU, když mám odklepnutý správný budík)
  • a co na to @luckajirku ?
SmejkalovaAnna commented 1 year ago

@katerinasirova s Katkou souhlasím, přesně, jak píše. Ikony disable save jsem otestovala, viz víše.

katerinasirova commented 1 year ago

Provedl jsem nasledujici zmeny:

  • tooltip pro ikonu dolu jsem zmenil, mrknete, jeslti text vyhovuje
  • ikonu dolu jsem pridal do toolbaru a dodrzel poradi ikon, jako je to v editaci
  • nasazeno na serveru - prosim tyto 2 body otestujte vcetne te ikony save, respektive jejiho zneaktivneni

Co se tyce tech budiku, pockame s Albertem na vyjadreni @luckajirku , jinak stav ikony "save" je opraveny, nyni je disablovana jak si chtela v tom prvnim bode @SmejkalovaAnna

Otestováno:

image

ZdenkaSera commented 1 year ago
  1. ze schůzky 30.5.: zatím ponecháme budíčky tak, jak jsou, tj. úpravy v reálném čase uvidíme pouze po cíleném zapnutí "budíčku"
  2. totéž ohledně podbarvení prováděných změn
  3. zvážit možnost zobrazit změyn prováděná v xml zároveň ve formuláři
katerinasirova commented 1 year ago

Při testování na notebooku jsem objevila, že ikony šipek jsou hodně široké a při běžném zobrazení se ta čtvrtá přidaná šipka posune dolů a zbytečně roztáhne toolbar na dva řádky. Dotaz na @pkudela - šířka/velikost těch ikon je daná nebo se dá upravit? Tady by totiž možná bylo vhodnější, aby ty 4 ikony byly víc u sebe... případně poprosím o komentář ostatní, jak to vidí oni.

luckajirku commented 1 year ago

o té šířce jsme se už bavili a Petr říkal, že to prověří

pkudela commented 1 year ago

ano, toto budu testovat az budu menit design cele aplikace, jak jsme se bavili na schuzce, kazdopadne to budu muset zmenit globalne, ale nikdy se nevyhneme tomu, ze nektera ikona bude mit kolem sebe vice mista, vsechny maji stejnou sirku, ale vizualne muzou nektere ikony pusobit tak, ze maji kolem sebe vice mista, protoze vypln kazde ikony je jina

luckajirku commented 1 year ago

no tady nejde ani tak o to, jak to působí vizuálně, ale z hlediska praktičnosti, je lepší, když jsou ty šipky blíž u sebe. Jako chápu, že nejde přizpůsobovat každé tlačítko ikoně a je to i nesmysl, ale kdyby byly dvě šířky (stávající pro normální tlačítka a užší pro ty šipky), to by se dalo, ne?

pkudela commented 1 year ago

@luckajirku asi by to slo, pokud by se to omezilo na 2 velikostii, udelam globalni class pro tyto ucely

katerinasirova commented 1 year ago

@luckajirku asi by to slo, pokud by se to omezilo na 2 velikostii, udelam globalni class pro tyto ucely

Právě, díky. U ostatních ikon to není tak výrazné a tam je to jedno, ale tady by se to vyloženě nabízelo.

ZdenkaSera commented 1 year ago

@luckajirku @katerinasirova Snažila jsem se zorientovat a otestotvat - prosím, je to kromě potíží se zobrazením na notebooku hotové? Zdá se mi, že ano, ale prosím o info, zda bychom tohle mohli navrhnout k uzavření. Děkuji.

luckajirku commented 1 year ago

No já původně čekala, že to ještě zkontroluju, až půjde vyzkoušet i reindexace. Jestli jde teď jen o to, jestli jsou tlačítka tam, kde mají být, a vypadají tak, jak mají vypadat, tak ano (resp. v tuto chvíli, kdy to píšu, jsou OK). Klikací cesta do objektu už je taky funkční včetně přístupu k titulovým metadatům.

ZdenkaSera commented 1 year ago

@luckajirku Děkuji za rychlou reakci. jestli správně chápu, tak reindexací máte na mysli to zobrazení indexů v přehledu závislé na akubře, reindexace jako taková je podle mého funkční.

luckajirku commented 1 year ago

jj, myslím otestování bez toho, aby člověk musel proklikávat jednotlivé strany, jestli ty indexy sedí.

ZdenkaSera commented 1 year ago

@albertoh @pkudela Musím to znovu otevřít - funkce neaktivní ikony pro ukládání změn má svá úskalí. 1.ve formuláři jsem vymazala celý element (např. note) - nemám jak uložit, ale element zmizel, v xml zůstává vidět

  1. použiju funkci obnovení na horní liště v editaci - element ve formuláři stále není, v xml ano
  2. vrátím se na obrazovku "hledat" a po zpětném otevření titulu v editaci je "vymazaný" element zpět i ve formuláři

Podle mého se stejného problému týkají i poslední poznámky od @kerschfilip zde https://github.com/proarc/proarc-client/issues/316.