quickbox icon indicating copy to clipboard operation
quickbox copied to clipboard

Ghost button in latest dev build

Open jmacura opened this issue 3 years ago • 6 comments

There is a blank button in the top control panel which opens another blank drop-down. image image Nothing special in the log. image

Fresh new install. No event exists.


QE version: https://github.com/Quick-Event/quickbox/actions/runs/2556216254 OS version: Win 10 Pro 21H1

jmacura avatar Jul 10 '22 20:07 jmacura

Podobných chyb je v QE bohužel více - není ošetřen stav, kdy není otevřen žádný závod. Třeba po kliknutí na tlačítko přidání závodníka QE spadne, další chyby očekávám i při pokusu o export atd.

Současná situace není určitě ideální, ale nevím, jestli tohle bude mít úplně prioritu.

paukert avatar Jul 10 '22 21:07 paukert

Chápu. Na druhou stranu... ošetření chyb typu "nezobrazuj toto, když neexistuje tamto" zabere člověku, který se v kódu vyzná, asi tak... 5 minut? Včetně spuštění IDE a kompilace apky? To je docela "low-hanging fruit", ne? 😉

A pro BFU to má přínos nepopsatelný.

jmacura avatar Jul 10 '22 22:07 jmacura

Těch míst kde je to třeba ošetřit je hodně. Uživateli vyskočí upozornění že má nejdřív otevřít/vytvořit/importovat event. V menu jsou zakázané ostatní volby, aby to měl jednoduší. Je to edge case, a jiných issues je víc než dost.

otahirs avatar Jul 11 '22 05:07 otahirs

Jsem rad, ze uz se dostavame k reseni takovych corner-cases, znamena to, ze zbytek je funkcni, protoze uzivatele repotuji vzdy nejvetsi problem. Chyba to samozrejme je, jenom nesouhlasim s low-hanging frui. Tech veci, ktere postradaji smysl, kdyz neni otevren zadny zavod je mnohem vic a urcite je nema smysl resit pomoci nejakych 5-ti minutovych quick fixu. Pokud neni otevren zadny zavod, jsou ghost buttony ve vsech perspektivach a to nemluvim o prazdnych tabulkach, atd.

Takze pokud chceme resit toto, asi by bylo nejlepsi, nedat uzivateli moznost opustit connection wizard , dokud neotevre nebo nenaimportuje nejaky zavot a to urcite neni na 5 minut.

V kazdem pripade, jsem rad, ze se toto tema otevira. Zpusob pripojeni k databazi a otevreni zavodu vznikl dost zivelne a nabranil bych se nejakemu smysluplnejsimu redesignu, ktery by treba vyresil i ghost buttony.

fvacek avatar Jul 11 '22 10:07 fvacek

Jsem rad, ze uz se dostavame k reseni takovych corner-cases, znamena to, ze zbytek je funkcni, protoze uzivatele repotuji vzdy nejvetsi problem.

Nebo to znamená, že uživatelé se k větším problémům ani nedostanou. Z hlediska UX:

  1. BFU spustí program
  2. "tady je něco, co nic nedělá"
  3. "je to rozbitý"
  4. "tak tohle nechci"
  5. game over

jmacura avatar Jul 11 '22 10:07 jmacura

Ano, tohle je mozny scenar, pokousime se prozatim cilit na chytrejsi uzivatele, na usporadani zavodu je to beztak potreba. Jinak se ale mozne uprave/vylepseni vubec nevyhybam, viz muj predchozi post.

fvacek avatar Jul 11 '22 19:07 fvacek