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

Upgrade app UI and address old issues #85

Merged
merged 7 commits into from
Feb 18, 2019
Merged

Upgrade app UI and address old issues #85

merged 7 commits into from
Feb 18, 2019

Conversation

nvarner
Copy link
Contributor

@nvarner nvarner commented Feb 18, 2019

Upgrade app UI and address old issues

This PR updates the app's UI to be more useful and solve several old issues.

Changes:

Screenshots

Microphone mode

screenshot_2019-02-18-13-35-42

Keyboard mode

screenshot_2019-02-18-13-36-02

Keyboard input is sent to Mycroft

screenshot_2019-02-18-13-36-10

Long press on Mycroft utterance

screenshot_2019-02-18-13-36-20

Long press on user utterance

screenshot_2019-02-18-13-36-15

Made the design more like a messaging app, including showing what the user said to Mycroft. Currently, those cards will only show up if the connection to Mycroft was successful.
+ Added lighter user cards displaying what the user said
+ Mycroft cards on the other side from the user cards
- Removed "Mycroft says:"
Long presses on utterances will show an option menu. Currently, the user's utterances can be copied and resent and Mycroft's utterances can be copied and shared. It is easy to add and implement new options. This addresses issue #53.
+ Options menu on long click
Changed the TTS switch to a button with icons. It should be more intuitive and look better.
* Changed the button
- Using Material Design icons
Create a button to toggle between keyboard mode and speech input mode. This will theoretically allow for a version without Google Play Services using a keyboard as input.
+ Keyboard toggle button
* Changed the FAB to another larger, centered button in microphone mode
@skraynick skraynick self-assigned this Feb 18, 2019
Copy link
Collaborator

@skraynick skraynick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please check individual comments. thanks

@nvarner
Copy link
Contributor Author

nvarner commented Feb 18, 2019

I changed line 123 to use utterance and renamed cb to clipboardManager.

@skraynick skraynick merged commit d9fde0e into MycroftAI:master Feb 18, 2019
@testman42
Copy link

I would just like to point out that Chuck Norris joke fact in screenshots above is very funny 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants