Upravy gui dle pripominek Kuby

[tomas-knap]

Kuba delal celkovou revizi GUI a neni jeste spokojen s tim jak vypada. Meli by to byt vsechno drobnosti. Vetsi veci jsem dal zvlast.

Execution monitor:

Schovat sloupce:

• ID - dozvim se z pipeline event (starting execution ID #1127)
• user - kdyz bude v budoucnu potreba, pridame
• Start (dozvim se z detailu)

Poradi sloupcu

• Tlacitka
• Status (uzky jen na iconu)
• Pipeline name
• Duration
• debugging, schedulled (uzky jen na iconu)

  Execution monitor detail:

Schovat sloupec "Logs", "Source"

Poradi sloupcu

• Typ (uzky jen na sirku icony)
• Datum
• Instance
• Message

Odstranit carku mezi tisicema - v pocitani page. Zvetsit okno pro ukazatel page. Aby se tam vesly i desitky tisic stranek

Schovat tlacitko "close", schovat "Download" (see #1130)

Pipeline list:

• schovat columns: id, owner, description
• Tlacitko id musi byt v detailu, tam ho prosim pridej (zobrazi se jen pro jiz existujici pipeline)

Poradi sloupcu:

• buttons (ikonky pro tlacitka - pouzij ikonky z exec monitoru/canvasu, texty z tlacitek v tooltipu)
• Name
• 3x last run..

  Scheduler:

• schovat sloupec ID v prehledu, schovat description, owner
• ID pridat do detailu eventy (zobrazi se jen pokud otviram uz existujici event)

Poradi sloupcu:

• tlacitka (ale ikony misto textu, text do tooltipu)
• Pipeline
• Rule
• Last, Next
• Last runtime
• Status (ale jen na sirku ikony)

  Settings:

Schovat tab "namespace prefixes"

[tomas-knap]

Pokud cokoliv neni jasne nebo se ti zda divne, pis

[bogo777]

Ok, neměl by tam být žádný zásadní problém.

Jen dotaz k těm sloupcům na šířku ikony - co se má stát s Headerem sloupce?

[tomas-knap]

Takhle, ta sirka by asi mela byt takova, aby se do combo boxu vesla ta ikonka + sipka pro zmenu vyberu. Kdyz se pak combo rozbali, tak tam bude ikonka a popisek, s tim, ze ten rozbaleny list klidne muze presahovat. Myslim ze u nejakeho combo boxu uz to tak i mame

[tomas-knap]

1) Sirka sloupce pro buttons by mela mit vzdy pevnou delku a to takovou, aby se tam akorat vesly buttons:

Podobne pro dalsi sloupce s jasnymi hodnotami a jasnou minimalni sirkou (trebas status, last status, atd.), ty by proste mely mit pevnou velikost. Nikdy by se automaticky nemely vykreslit sirsi, ikdyz mam 40ku monitor.

[tomas-knap]

2) Pipeline detail by mela obsahovat "Created by: {autorPipeline}". Pod visibility.

[tomas-knap]

3) Execution monitor - fixni velikost. Sirku jeste malinko vetsi, aby se tam vesel i label "STATUS" sloupce. Omlouvam se, spatna spec ;-)

Similarly for debug. Label for scheduled should be only "SCH."

[tomas-knap]

4) Protor nad tlacitky vypada hrozne prazdny:

Co tam dat label "Actions"? U exec monitoru, pipeline listu.

[tomas-knap]

5) Ten sloupec tlacitek "Detail" u pipeline name u exec monitoru -> dej tam prosim nejakou icon. A umisti ten detail button (aktualne ve sloupci pipeline name) jako posledni action mezi tlacitkove actions. Opet s nejakou icon. Koukam totiz, ze je to u toho sloupce name trochu divne a hlavne pak divne funguje klikani na record (kdyz kliknes na record mimo sloupec pipeline name, tak ok, kdyz kliknes na pipeline name, tak to nejde)

6) Pokazeny log message detail:

[tomas-knap]

7) pipeline exec detail -> zalozka log -> Dej prvni sloupec loglevel, prejmenuj prosim na "Type". Pridej tam taky ikonky jako u events.

8) Scheduller - viz zadani

[tomas-knap]

9) Ikony - exec monitor - debug data/show log (ruzne ikony), cancel. Text opet do tool tipu

[tomas-knap]

10) Zkontroluj na zaver prosim, ze vsechny sloupce maji minimalni sirku nastavenou tak, aby se tam vesel header (label) sloupce.

[tomas-knap]

11) Please add back to exec monitor table the column "Started", which should be before the column "Duration". Sorry for that. Please add also column "Executed By" (instead of the original column "Author") to execution monitor. It should be the last column. It is needed because execution monitor is the only reasonable place where the user who executed the pipeline should be shown

[tomas-knap]

12) Regarding the action button for pipeline list, please order them as: Run, debug, schedule, copy, edit, delete. Delete icon should be red (the trash bin)

[bogo777]

Mělo by to být, až na:

5) Už jsem to upravil tak, že execution detail se zobrazí i po kliknutí na pipeline name, ještě by šel přes css přidat kurzor, aby byl stejný jako pro zbytek řádku, změnil bych header jen na pipeline a tlačítko detail nechal tam se standardní ikonou pro detail

nebo tedy přesunout a přidat novou ikonu, ale nenapadá mě žádná vhodná...

obecně k těm ikonám bych potřeboval napsat jakou z existujících použít, případně přímo poslat novou, přijde mi jako ztráta času nějakou hledat, která se pak nelíbí a hledat znovu, když se dá rovnou najít vyhovující, ideálně .png 16x16

Tedy pro dokončení prosím o upřesnění/ikonky pro: 9), 7) TRACE, 5) pipeline detail?, 12) red trash bin?

[tomas-knap]

Ad 5) ok, zkusim, diky. Nech to zatim tedy u toho pipeline name se standardni ikonou pro detail. Uprav jeste prosim ten kurzor v CSS, at je to jednotne. Zkusime a uvidime.

9) Cancel - stejna jako ikona cancelled ve statusu, show log - http://findicons.com/icon/219135/view?id=219295 , debug view - to same co show log, ale s mensi "debug pipeline" icon v pravem dolnim rohu (tedy s tim broukem v pravem dolnim rohu, snad nebude moc maly). 7) trace - https://www.iconfinder.com/icons/193069/hand_hands_logs_palm_prints_steps_touch_trace_icon 12) https://www.iconfinder.com/icons/32104/recyclebin_trash_icon#size=64 Ale podbarvi prosim nejak cervene ten kos

[tomas-knap]

Bohuslave, diky, jeste prosim:

[tomas-knap]

B) Id u pipeline detail dej prosim az za uzivatele, neni to tak dulezite, aby to bylo prvni. Pokud delam pipeline, nezobrazuj tam "ID New", ale proste tenhle radek schovej (klidne si tam na nej rezervuj misto, to je jedno). Kdyz dam save, radek s ID se objevi.

U schedulled event detail dopln prosim "Schedulled by: {user}" a dej to za description. Az za to dej opet to IDcko. Opet pokud ID jeste neni prirazeno, radek s ID schovej.

C) U schedulleru dej prosim status sloupec hned za actions. Za to se omlouvam, to je moje chyba.

[tomas-knap]

E) V Exec monitoru, kdyz kliknu na pipeline name, tak se zobrazi spravny kurzor, zobrazi se detail, ale nevybere se radek v exec monitoru. Kdyz kliknu na started, tak se vybere! To klikani na pipeline name sloupec v Exec monitoru funguje divne, obcas jo, obcas ne; jine sloupce se chovaji korektne. Vyzkousej si opet na odcs-test

[tomas-knap]

G) WHen opening exec monitor detail, it may happen that the pipeline name column dissapears:

Solution: Pipeline name column should have certain minimal width

[tomas-knap]

H) When filtering logs in exec monitor detail, there should be icon + text in the combo box. Currently there is only text.

[tomas-knap]

I) In scheduller, there is no space between actions icons (but in pipeline list/exec monitor there is a space). Put space in scheduler.

[tomas-knap]

J) Execution monitor, sometimes, when resizing, the column for action buttons is resized so that not all actions are visible. Actions column's size should be fixed, if possible. Or at least, the minimum width should be specified so that the buttons are always all visible

[bogo777]

C, E, H, I, J - done

G - this a problem, I didn't find solution yet.

• https://vaadin.com/api/com/vaadin/ui/Table.html#setColumnExpandRatio(java.lang.Object, float)
• it looks like either expand ratio OR width can be set, NOT both

B - later tonight

[tomas-knap]

Ad G) Nepomohlo by nastavit minimalWidth cele tabulce? Tedy aby trebas vypadala takto nejhure:

V pripade, ze ma tabulka jeste mene horizontalniho prostoru, tak se objevi horizontalni posuvnik.

Jeste dotaz, executedBy je fixni sloupec?

Ono kdyz na to ted koukam, tak bez toho minWidth se to chova divne, viz niz, headers nesedi:

[tomas-knap]

Tu minimalWidth pro table prosim nastav nejak rozumne (aby byly vsechny sloupce aspon trochu videt, neposouvali se headers) take pro pipeline list, scheduller, exec record detail

[tomas-knap]

Jeste drobnost, slo by do tech zavrenych comboboxu u uzkych sloupcu dava proste jen tu icon, bez nasledneho textu? Az kdyz se combo rozbali, tak tam bude videt to co ted (icon + text)

[bogo777]

Pri nastaveni pevne sirky tabulky se v pripade, ze je k dispozici mene mista nez je sirka, zobrazi posuvnik pro cele okno/levou cast okna u execution listu a ne primo v tabulce, coz vypada take blbe.

Byl bych pro nastaveni pevne sirky u vsech sloupcu, kde je to rozumne - ikonky, tlacitka, duration, mozna i datumy...a pro zbytek sloupcu bych nechal "natural" width = min. sirka takova, aby se tam vesel obsah, pripadne misto navic se rovnomerne rozdeli mezi vsechny "natural" width sloupce v tabulce...

U comboboxu muze byt bud ikona + text vzdy nebo nikdy, nevim jak by to slo nastavit zvlast pro vybranou hodnotu...

[tomas-knap]

Bohuslave, k te sirce, urcite nastav prosim pevnou sirku vsude tam, kde to dava dobry smysl. To jsou vlastne vsechny sloupce (u pipelineList,execmonitor, scheduler) az na pipeline name, executedBy a rule u scheduleru. Bohuslave, ale kdyz ted koukam na tabulku exec monitor detail - events, tak tam se to chova rozumne. Jak je to tam nastavene? Bo standardne to vypada takto:

A kdyz tu tabulku zmensim prilis, tak se objevi posuvnik:

Nicmene povsimni si, ze i v tom normalnim zobrazeni tam ta short message neni cela, tedy dlouha short message si nevynuti scrollbar. Na druhou stranu se nikdy nestane, ze by short message sloupec byl extremne maly. Neco podobneho bychom chteli pro pipeline name i v hlavni tabulce exec monitoru.

[tomas-knap]

K te pevne sirce, urcite prosim zmensi status a debug:

[tomas-knap]

K tem combo boxum jen s ikonou, udelal jsem #1195 jako future work

[bogo777]

Upravil jsem sirky sloupcu, snad uz to vypada docela dobre. U te "natural" width, ktera se pouziva i v te Events table, se zjevne pri pocitani minimal width neberou v uvahu vsechny/aktualni radky.

Nevim presne jak to funguje, ale da se to snadno vyzkouset resizem okna. Muzu se zkusit podivat, jestli o tom nekde neco nepisou.

[tomas-knap]

Takhle, podstatne je, aby to fungovalo nejak rozumne, tedy sloupce, ktere mohou mit pevnou sirku, necht ji maji. Ale nesmi se stat toto (viz sloupec pipeline name):

[tomas-knap]

Bohuslave, jeste bych poprosil udelat tam konstanty, ktere budou drzet aktualni pozadovane sirky sloupcu. At nemusim v budoucnu resit, ze chci upravit sirku, ale zapomenu to upravit na vice mistech. Tedy necht je v uvodu tridy serie konstant typu "COLUMN_PIPELINE_NAME_WIDTH"

[tomas-knap]

Dodelej prosim ty konstanty, at muzem uzavrit. Jinak jsem spokojen.