Formulare - refactoring / redesign

pkudela commented 3 months ago

[x] odlisit hlavni oblasti od vnorenych - podbarveni, nebo ramecek, nebo barva pripadne tucne pismo
[x] zmensit radky jednotlivych fieldu
[x] zkuist zmensit vysku materialoveho fieldu
[x] vizualne oznacit nevalidni formular
[x] u disablovaneho fieldu sjednotit barvu dolni cary

pkudela commented 3 months ago

@erikamirova ahoj, ted testuju to co nasadil po oprave alberto na test u tohoto konkretniho objektu -> https://proarc-test.inovatika.dev/repository/uuid:cddd21bd-1726-4b6f-9d10-ecb4300df817 a vse se zobrazuje korektne, kdyz srolluju dolu a dojde na kartu formulare "Origin info", tak se mi zmeni i horni hodnota inputu "Zobrazit element" a kdyz jedu dal a narazim na "Location", tak se mi take horni hodnota spravne prepne viz. screenshot nize. Zkus si mozna vymazat cache prohlizece, jestli tam neco nevisi.

Zkousim to i opacne, ze zvolim hotnotu "Origin info" v poli "Zobrazit element" a chova se to stejne, skoci to vzdy na pozadovanou kartu formulare a hodnoty se shoduji.

erikamirova commented 3 months ago

@pkudela Jooo, už to funguje. Díky👍

pkudela commented 3 months ago

@katerinasirova diky za vytvoreni nove davky, dodelal jsem formular pro hromadnou upravu zvukovych nahravek, je to na testu -> https://proarc-test.inovatika.dev/

katerinasirova commented 3 months ago

Snažila jsem se odpoledne prokousat změnami a návrhy a za nás prosím o ty dva návrhy, jak psala Zdeňka - s tím, že pokud jsem to identifikovala správně, tak za nás by řešení, kdy stav před schůzkou by byl jako default a ta druhá výraznější varianta by byla na uživatelské preferenci (pokud by nešlo, tak preferenci instituce), bylo OK.

Jen při rychlém přetestování se mi teď na té verzi, na které nasadil Petr nezobrazuje červeně tučně zvýrazněný název nevalidního elementu, červeně je pouze v seznamu všech elementů v roletce "Zobrazit element":

Otázka je, jestli nestačí červeně jen ta část formuláře - viz obrázek. Nechávám na zvážení ostatních.

Při dnešním testování už se nezobrazuje červeně ani která hodnota v elementu je nevalidní:

katerinasirova commented 3 months ago

Jinak za nás podoba formulářů odsouhlasena, už se těšíme na nasazení. Kromě zobrazení nevalidních hodnot připomínky nemáme.

pkudela commented 3 months ago

Jinak za nás podoba formulářů odsouhlasena, už se těšíme na nasazení. Kromě zobrazení nevalidních hodnot připomínky nemáme.

Mas na mysli nevalidni hotnotu pole formulare? Prosim, posli mi uuid objektu ze screenshotu, dik!

pkudela commented 3 months ago

Jinak za nás podoba formulářů odsouhlasena, už se těšíme na nasazení. Kromě zobrazení nevalidních hodnot připomínky nemáme.

Mas na mysli nevalidni hotnotu pole formulare? Prosim, posli mi uuid objektu ze screenshotu, dik!

Uz nemusis, nasel jsem to, jedna se jen o nektere elementy, lisi se to i oproti puvodni verzi na masteru, to zitra musim probrat s @albertoh , jakmile tohle opravime, tak nasadime na master a naopak na test dam soucasnou verzi co je na mastru pred refactoringem formularu, abychom meli po nejakou dobu srovnani, kdyby se neco objevilo.

albertoh commented 3 months ago

Zobrazení nevalidních hodnot: opraveno

Nasazeno na testu

pkudela commented 3 months ago

ahoj vsem, po oprave chyby validace jsme udelali nasledujici:

na master odkazu je dostupna posledni aktualni verze s opravou chyb, ktere byly zmineny v tomto issue a s novym formularem
na testovacim odkazu je stale dostupna puvodni verze bez opravy chyb a pred refactoringem formulare pro ucely srovnani (necham dostupny cca mesic)
@albertoh dnes vyda novou verzi

pkudela commented 3 months ago

@ZdenkaSera prosim, potom nezapomen zavrit toto issue, nebo ho mam zavrit ja? Pokud by byly nejake dalsi dilci pozadavky na upravu formulare, tak uz bych to nevazal s timto issue, ale separatne.

ZdenkaSera commented 3 months ago

Navrhuji - vzhledem ke komplexnosti tohoto issue - pro přehlednost otevírat nová issues pro případné chyby/vylepšení. Toto issue zatím jen označím k uzavření.

luckajirku commented 3 months ago

a já to ještě přihodím sem, je to jen kosmetika, to si ani nezaslouží vlastní issue - u role term je ten text přiražený úplně na rámeček (nikde jinde jsem na to zatím nenarazila) a vypadá to divně. prirazenyautor

pkudela commented 3 months ago

a já to ještě přihodím sem, je to jen kosmetika, to si ani nezaslouží vlastní issue - u role term je ten text přiražený úplně na rámeček (nikde jinde jsem na to zatím nenarazila) a vypadá to divně.

Podivam se na to.

pkudela commented 3 months ago

@luckajirku fixnuto, nasazeno na masteru, mozna si budes muset udelat tvrdy reload prohizece aby se zmena nacetla

luckajirku commented 2 months ago

jo, to vypadá OK

SmejkalovaAnna commented 2 months ago

@pkudela Petře, když je to zvýraznění teď možné tak hezky uživatelsky zaškrtnout, chtěla bych tě po dohodě se @ZdenkaSera , @NeradovaK a @kerschfilip ještě poprosit, jestli by ses na tom zvýraznění ještě trochu víc odvázal ( sytější šedou? Silnější rámečky? ) - jde o to, že teď, jak to je, tak je ten rozdíl fakt strašně malý a skoro neviditelný.... dala jsem si k sobě oba formuláře a rozdíl je malinkatý. Půjde to? Teď bez a s: obrazek

pkudela commented 2 months ago

@SmejkalovaAnna neboj, ja se tedy odvazu :-), akorat to nebude hned, ted mam rozdelany asjournals, ktery potrebuju dodelat, ale do schuzky s tim snad neco udelam.

luckajirku commented 2 months ago

@pkudela jen pro jistotu, aby ses pod vlivem Aniččina fandění neodvázal tak, že se přestaneš kontrolovat:-) a půjde to i do defaultní verze bez zvýraznění :-))) - v Hradci nám tak, jak je teď, vyhovuje, my nepřepínáme:-) - za nás je ta verze nadmíru povedená, my jsme spokojení!

pkudela commented 2 months ago

@pkudela jen pro jistotu, aby ses pod vlivem Aniččina fandění neodvázal tak, že se přestaneš kontrolovat:-) a půjde to i do defaultní verze bez zvýraznění :-))) - v Hradci nám tak, jak je teď, vyhovuje, my nepřepínáme:-) - za nás je ta verze nadmíru povedená, my jsme spokojení!

Neboj, bude se to tykat pouze pokud budes mit aktivovano zvyrazneni, defaultni zobrazeni, ktere pouzivate, zustane beze zmeny.

pkudela commented 2 months ago

@SmejkalovaAnna zvysil jsem sytost barvy ohraniceni a stejne tak sytost stinnu, vyhovuje takto?

bez zvyrazneni a pod tim zvyraznene

pkudela commented 2 months ago

Jeste jsem zkousel zvetsit tloustku ohraniceni, ale do toho bych nesel, to mi prijde uz moc

SmejkalovaAnna commented 2 months ago

@pkudela Líbí se mi ten druhý a třetí obrázek :) Super!

SmejkalovaAnna commented 2 months ago

@pkudela pokud ti ten třetí už přijde moc, tak budu spokojená s druhým.

pkudela commented 2 months ago

@pkudela pokud ti ten třetí už přijde moc, tak budu spokojená s druhým.

No to je na vas :-) V klidu se na to podivejte a dejte mi vedet, dulezite je, co vam bude vyhovovat stran prace a ne to co se libi me. Nasadim ted tedy zatim verzi 2 na server.

pkudela commented 2 months ago

Nasazeno na serveru, v klidu na to mrknete a dejte vedet

NeradovaK commented 2 months ago

já teda souhlasím s @SmejkalovaAnna, mě se taky ten třetí navrh líbí, vypadá dobře :)

SmejkalovaAnna commented 2 months ago

@pkudela Petře prosím nasaď tedy ten třetí, nejvýraznější. Kačce se taky líbí nejvíc, tak chceme ten. Díky moc!

pkudela commented 2 months ago

@SmejkalovaAnna hotovo, je to na serveru, prosim otestujte a pak pripadne navrhnete k zavreni toto issue

SmejkalovaAnna commented 2 months ago

@pkudela Za věchny píšu, že je to pěkný a že s tím takto souhlasíme.

ZdenkaSera commented 2 months ago

@katerinasirova Katko, zatím nenavrhuji k uzavření, ještě na to prosím koukni Ty. Díky.

proarc / proarc-client

Formulare - refactoring / redesign #514