-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
fix: modified voice commands #2170
fix: modified voice commands #2170
Conversation
@iamareebjamal Please review this !! |
@iamareebjamal Please review this sir!! |
Peer reviews |
@atm1504 please review this for me!! |
@@ -16,7 +16,7 @@ | |||
<android.support.v7.widget.RecyclerView | |||
android:id="@+id/clickableCommands" | |||
android:layout_width="match_parent" | |||
android:layout_above="@id/linearLayout" | |||
android:layout_above="@id/txtChat" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
O, sorry I didn't notice it.
@@ -13,6 +13,6 @@ | |||
tools:text="Open WhatsApp" | |||
android:id="@+id/voiceCommand" | |||
android:padding="@dimen/padding_moderate" | |||
android:textSize="@dimen/text_size_large"/> | |||
android:textSize="@dimen/text_size_medium"/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is just changing the textsize doing the work?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes it is!
@iamareebjamal I have reviewed it. Its okk from my side. |
@iamareebjamal Please review this and merge this!! |
Can't merge until CircleCI works |
Any idea how to check for ci build failure? |
It's not even being triggered. Only happening on your PR apparently |
Yes!! Any idea how to see what the problem is? |
@batbrain7 @arundhati24 as the ci/circleci is failing to trigger can I make a new pr to solve this issue? |
Fixes #2150
Changes:When the user talks, the txtchat is hidden by the voice commands.
Modified it by reducing the size and making it top of txtChat
Screenshots for the change: