#9 Propojení GUI a knihovny #22
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Collaborator
ondrejzemanek
commented
Apr 18, 2017
ondrejzemanek
commented
- Modify history listview in dislay.xml
- Add Converters for ConsoleText
- Add MainModel for linking MathLib and ext. functionality
- Modify HistoryItem
- Update Enums with stringValues for display operation symbols
- Change keyboard to view communication style
- Modify history listview in dislay.xml - Add Converters for ConsoleText - Add MainModel for linking MathLib and ext. functionality - Modify HistoryItem - Update Enums with stringValues for display operation symbols - Change keyboard to view communication style
|
@jankaderabek @jelinet01 prosím Vás o code review a především o otestování propojení knihovny. Díky |
2 tasks
daliborjelinek
requested changes
Apr 18, 2017
There was a problem hiding this comment.
K propojení gui a knihovny mám následující výhrady:
- na displayi se nezobrazuje symbol operace např. u 5*5 není nikde vidět, že jsem stisknul *
- v případě, že začnu psát nové číslo poté, co jsem dostal výsledek předchozí operace, nově napsané číslice se připojují za již zobrazené číslo
- nevím zda to vyžaduje zadaní, ale naše aplikace nepodporuje zřetězené operace. Např 33+5.
V současné chvíli aplikace po napsaní tohoto výrazu spočítá 335, což rozhodně není dobré chování. - v případě binárních operací se na konci musí stisknout = v případě unárních nikoliv. Toto bych sjednotil (osobně jsem pro variantu vždy potvrdit výpočet stisknutím tlačítka = )
- historie se řadí chronologicky opačně. Bylo by dobré kdyby vždy byla vidět poslední operace. Posuvník vypadá poněkud nevzhledně. Asi bych zobrazil jen poslední dvě operace.
- nefunguje desetinná čárka
- přiliš dlouhá čísla se oříznou z obou stran (jak na displayi tak v paměti)
Jinak pěkný kus práce. Díky.
- Add MemoryItem class for working with memory - link MemoryItem to view
- Add selected operation to display - Add limitation to history - set to 2 items - Reversed history listview - newest on top - Remove auto complete for single operand operations - Fix problem with decimal symbol - Fix font size formula for displaying number on display - Minor changes
|
@jelinet01 @jankaderabek Přidal jsem požadované změny. Prosím o otestování. Nově jsem přidal i práci s pamětí. Díky |
- Fix problem with create operation with result - Fix delete last char problem
- Add full formula to display
- Reverse History order - display minor changes
- Add calculation when user selecting next operation.
|
@jelinet01 @jankaderabek Prosím o dokončení review. Přidal jsem novou funkcionalitu. Když uživatel píše další operaci, tak se předchozí operace vypočítá a počítá se s výsledkem. |
jankaderabek
approved these changes
Apr 19, 2017
daliborjelinek
approved these changes
Apr 19, 2017
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.