Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Vytvoření GUI #5

Closed
1 task done
jankaderabek opened this issue Mar 9, 2017 · 5 comments
Closed
1 task done

Vytvoření GUI #5

jankaderabek opened this issue Mar 9, 2017 · 5 comments
Assignees
Projects

Comments

@jankaderabek
Copy link
Owner

jankaderabek commented Mar 9, 2017

  • Vytvoření GUI na základě mockupu
@jankaderabek jankaderabek added this to Backlog in Aviatto Mar 9, 2017
@jankaderabek jankaderabek mentioned this issue Mar 9, 2017
10 tasks
@ondrejzemanek ondrejzemanek moved this from TODO to In progress in Aviatto Mar 9, 2017
@ondrejzemanek ondrejzemanek moved this from In progress to TODO in Aviatto Mar 21, 2017
@ondrejzemanek ondrejzemanek moved this from TODO to In progress in Aviatto Mar 24, 2017
ondrejzemanek added a commit that referenced this issue Mar 24, 2017
 - Create GUI from mockups screens
 - Create userControl for  functionkeys and specialkeys
 - Create control for keyboard, display and toolbar
 - Create Commands for usercontrols
 - Create Exec methods for commands
 - Create enums for keys
 - Create animations for keys
ondrejzemanek added a commit that referenced this issue Mar 27, 2017
 - Add contentcontrol for vector buttons
 - Add display buttons with functionality
 - Add functionality for copy and back button
 - Add tooltip for copy button
 - Modify keyboard for vector content buttons
 - Add numpad keys bindings
@ondrejzemanek
Copy link
Collaborator

Úkol realizován ve větvi 5-gui-implementation.
PR: #17

@Glooby5 prosím o proklikání aplikace a o review kódu v pull requestu.
@jelinet01 prosím o kontrolu návrhu na základě mockupů.

ondrejzemanek added a commit that referenced this issue Mar 27, 2017
#5 Implementace grafického rozhraní na základě mockupů
@daliborjelinek
Copy link
Collaborator

daliborjelinek commented Mar 28, 2017

Z grafického hlediska bych si dovolil poukázat na následující nepřesnosti oproti návrhu. Některé jsou spíše drobnosti, jiné dle mého názoru zasluhují vyšší pozornost.

  1. použitý font není open sans light
  2. tlačítka nejsou kruhová (což považuji za závažný problém) (jsou o 5px širší)
  3. tlačítka jsou u sebe dost natěsno.
  4. WPF používá jako defaultní jednotku normalizovaný pixel. To je důvod, proč je výsledek znatelně větší než grafický návrh přesto, že hodnoty odpovídají. Osobně bych okno rozhodně zmenšil.
  5. memory tlačítka jsou vyšší a mají větší border radius.
  6. přesto, že jsem převáděl všechny nestandardní symboly do svg tak žádný nebyl použit:
  • close button
  • x^y
  • root
  • divide
  • multiply
  1. close button je příliš daleko od otazníku.
  2. není to v mockupu, ale možná by v té horní liště mohlo být něco jako Aviatto Calculator či tak něco
  3. pozadí je místo obrázku gradient který nevypadá moc hezky.
  4. chybí backspace tlačítko
  5. chybí clear tlačítko

@ondrejzemanek
Copy link
Collaborator

ondrejzemanek commented Mar 28, 2017

@jelinet01 stáhni si prosím znovu repozitář z větve master. Určitě jsi testoval špatnou verzi aplikace. Většina problémů, které popisuješ je v nové verzi vyřešena. Napiš pak prosím, které problémy přetrvávají v nejnovější verzi a já je opravím.

@ondrejzemanek ondrejzemanek moved this from In progress to In review in Aviatto Mar 29, 2017
@ondrejzemanek ondrejzemanek moved this from In review to Merge pending in Aviatto Mar 29, 2017
@daliborjelinek
Copy link
Collaborator

K aktuální verzi dle mého mínění vyžadují pozornost následující úpravy:

  1. použít font open sans light
  2. grid systém tlačítek by měl být nasledující:
  • okraj od okna aplikace 17px
  • mezera horizontální i vertikální mezi tlačítky 16px
  • velikost tlačítka včetně jeho okraje 92px
  • ( 2x17 + 3x16 + 4x92 = 450 = šířka okna)
  1. border radius memory tlačítek je 10px a jejich výška je 23px
  2. backspace tlačítko je zdeformované (to vadí méně) není centrované vzhledem k "delete all" tlačítku (to vadí více)
  3. problém s tím, že okno je obrovské... ovšem nevím, zda to má nějaké rozumné řešení.
  4. horní lišta má následující problémy:
  • místo close ikonky je písmeno X
  • chybí minimalize button
  • chybí titulek aplikace
  • close button je příliš daleko od help button

@ondrejzemanek
Copy link
Collaborator

Díky za poznámky.
Moje zpětná odezva k připomínkám:

  1. Font jsem záměrně použil systémový, abychom předešli problémům s instalací fontu na jiných počítačích. Integrace do aplikace je možná, ale složitá a proto v rámci rozsahu projektu bych to zanedbal.
  2. Upravil jsem to v následujícím commitu tak, aby se s kalkulačkou dobře pracovalo, ale jelikož wpf používá jiný typ jednotek, tak je velice složité se trefit přesně do návrhu.
  3. Opraveno
  4. Opraveno
  5. Vyřešeno zmenšením celkového layoutu.
  6. Vše opraveno

ondrejzemanek added a commit that referenced this issue Mar 29, 2017
 - change layout size
 - create right size aspect ratio for buttons
 - fix special buttons
 - Change close button label for vector X
 - Change space between toptoolbar buttons
 - Add app label
 - fix display buttons functionality
ondrejzemanek added a commit that referenced this issue Mar 29, 2017
 - change special buttons for working with memory
ondrejzemanek added a commit that referenced this issue Mar 29, 2017
@ondrejzemanek ondrejzemanek moved this from Merge pending to Completed in Aviatto Mar 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Aviatto
Completed
Development

No branches or pull requests

3 participants