Skip to content

Conversation

@liveHarshit
Copy link
Member

@liveHarshit liveHarshit commented Jul 26, 2019

Fixes #2175
Fixes #2184

@auto-label auto-label bot added the fix label Jul 26, 2019
ShridharGoel
ShridharGoel previously approved these changes Jul 26, 2019
@ShridharGoel ShridharGoel dismissed their stale review July 26, 2019 21:22

Changes required

@liveHarshit liveHarshit changed the title fix: Feedback button fix: Feedback - button and crashes Jul 27, 2019
@liveHarshit
Copy link
Member Author

Updated

<Button
<com.google.android.material.button.MaterialButton
android:id="@+id/feedbackBtn"
style="@style/ThreeDS2TextButtonStyle"
Copy link
Member

Choose a reason for hiding this comment

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

What's this?

Copy link
Member Author

Choose a reason for hiding this comment

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

For UI.

Copy link
Member

Choose a reason for hiding this comment

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

What does it do which is not in default style?

Copy link
Member Author

@liveHarshit liveHarshit Jul 27, 2019

Choose a reason for hiding this comment

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

It shows elevation and boundary line. Even if a transparent background is set.

Copy link
Member

Choose a reason for hiding this comment

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

Merging this. But I see no documentation for this

Copy link
Member Author

Choose a reason for hiding this comment

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

Found it. It is from the stripe library.

Copy link
Member

Choose a reason for hiding this comment

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

Wow. Please revert it in the next development iteration ASAP.

Copy link
Member Author

Choose a reason for hiding this comment

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

Wow. Please revert it in the next development iteration ASAP.

Sure 👍

tools:visibility="visible"/>

<Button
<com.google.android.material.button.MaterialButton
Copy link
Member

Choose a reason for hiding this comment

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

Why? It happens automatically when android inflates the layout

<string name="what_type">What Sounds good?</string>
<string name="feedback">Feedback</string>
<string name="write_feedback">Write Feedback</string>
<string name="write_feedback">WRITE A FEEDBACK</string>
Copy link
Member

Choose a reason for hiding this comment

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

Revert

Copy link
Member Author

Choose a reason for hiding this comment

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

Okay. I just follow the play store for UI.

Copy link
Member

Choose a reason for hiding this comment

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

Button text is automatically capitalized

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes right sorry. I have changed it for text view.

Copy link
Member Author

Choose a reason for hiding this comment

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

Not capitalizing with that style. Added an extra attribute.

@liveHarshit
Copy link
Member Author

Updated

android:id="@+id/feedbackBtn"
style="@style/ThreeDS2TextButtonStyle"
android:textAllCaps="true"
android:textAlignment="center"
Copy link
Member

Choose a reason for hiding this comment

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

Unneeded

Copy link
Member Author

Choose a reason for hiding this comment

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

removed

@iamareebjamal iamareebjamal merged commit 506a3cf into fossasia:development Jul 27, 2019
@liveHarshit liveHarshit deleted the 2175 branch July 27, 2019 20:04
liveHarshit added a commit to liveHarshit/open-event-attendee-android that referenced this pull request Aug 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

App crashes on writing feedback after 2nd UI overlapping in event details

4 participants