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

[FIX] Oval Shape button to a Circle in the What's New screen section #3364

Merged
merged 2 commits into from
Sep 1, 2021

Conversation

sashpawar11
Copy link
Contributor

Related Issues

App: #3351

Changed the previous oval shape to what it was intended to be, a Circular round button on the What's New Activity Screen after it reaches the final screen.
Attached screenshot below.

CircularRoundButton


…lcome section - Changed layout width and height of the button to 64dp ( Basically a square).
@CLAassistant
Copy link

CLAassistant commented Aug 31, 2021

CLA assistant check
All committers have signed the CLA.

@JuancaG05 JuancaG05 linked an issue Aug 31, 2021 that may be closed by this pull request
@abelgardep
Copy link
Contributor

Thanks for the contribution @sashpawar11 !! we will take care of it soon 👍

@JuancaG05 JuancaG05 changed the title [FIX] Oval Shape button to a Circle in the What's New screen section. Issue #3351 [FIX] Oval Shape button to a Circle in the What's New screen section Aug 31, 2021
@JuancaG05 JuancaG05 self-requested a review August 31, 2021 15:31
Copy link
Collaborator

@JuancaG05 JuancaG05 left a comment

Choose a reason for hiding this comment

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

Thanks a lot for your contribution @sashpawar11! Please take a look at this other XML file, it's the same layout you modified but for other screen resolution. If you could test it in some bigger device and see if it needs changes, it would be great and very complete! 👍

@sashpawar11
Copy link
Contributor Author

Thanks a lot for your contribution @sashpawar11! Please take a look at this other XML file, it's the same layout you modified but for other screen resolution. If you could test it in some bigger device and see if it needs changes, it would be great and very complete! 👍

Sure, I'll test it out

@abelgardep
Copy link
Contributor

Hi @sashpawar11, I have checked it with some devices with different screen sizes. 64dp is too big for smaller devices like Nexus S. BTW, this activity has 2 layouts, as @JuancaG05 stated here, and we need to update the other one too because it is used for bigger devices like Nexus 10, which still shows an oval.

Nexus S Nexus 10
Screenshot_1630479356 Screenshot_1630480129

A first solution could be reducing the size from 64dp to 48dp in https://github.com/owncloud/android/blob/master/owncloudApp/src/main/res/layout/whats_new_activity.xml and add 64dp in https://github.com/owncloud/android/blob/master/owncloudApp/src/main/res/layout-sw600dp/whats_new_activity.xml

Just some changes and we will move it forward 👍

@sashpawar11
Copy link
Contributor Author

sashpawar11 commented Sep 1, 2021

Thanks a lot for that @abelgardep! I'll do the changes immediately

@sashpawar11
Copy link
Contributor Author

sashpawar11 commented Sep 1, 2021

Made the changes @JuancaG05 @abelgardep ! Tested it on both Small and Large screens, works absolutely fine.

NexusS:
NexusS
Nexus 10:
nexus10

Copy link
Collaborator

@JuancaG05 JuancaG05 left a comment

Choose a reason for hiding this comment

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

Great!! Thanks a lot again for your contribution @sashpawar11!! See you!! 🥳

@abelgardep
Copy link
Contributor

Checked with different devices:

Nexus S
Nexus 5X
Pixel 3XL
Nexus 10

LGTM, thanks for the contribution @sashpawar11

@abelgardep abelgardep merged commit 47dca64 into owncloud:master Sep 1, 2021
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.

[BUG] Button in Welcome Wizard is oval
4 participants