Skip to content

Post visual update: Update the corner radius of buttons #6570

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

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

0nko
Copy link
Member

@0nko 0nko commented Aug 13, 2025

Task/Issue URL: https://app.asana.com/1/137249556945/project/715106103902962/task/1210511370341983?focus=true

Description

Update the corner radius of all buttons to 12dp.

Steps to test this PR

Check ADS and compare

  • Go to Settings -> ADS
  • Check that the buttons now have a corner radius of 12dp
  • Smoke test the app and verify the buttons have the correct radius

UI changes

image

@@ -63,6 +63,7 @@ open class DaxButton @JvmOverloads constructor(

minHeight = resources.getDimensionPixelSize(Size.dimension(buttonSize))
setPadding(sidePadding, topPadding, sidePadding, topPadding)
setCornerRadiusResource(R.dimen.mediumShapeCornerRadius)
Copy link
Contributor

Choose a reason for hiding this comment

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

Can't you change the style instead? /cc @nalcalag

Copy link
Member Author

Choose a reason for hiding this comment

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

I can. Done here.

Copy link
Contributor

@nalcalag nalcalag left a comment

Choose a reason for hiding this comment

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

LGTM

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