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 material switch padding on item_device_form layout #2106

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Divinelink
Copy link

Description

This pull request addresses the missing padding in the item_device_form.xml, which caused the Material Switch to appear too close to the edge of the screen. To resolve this, I added the same padding used by the other Material Switch components through their style.

Let me know if this works for you!

Changes

What changes are made in this PR

  • Add android:paddingEnd="@dimen/abc_action_bar_content_inset_material" on item_device_form.xml layout

Screenshots

In case of UI changes, add before and after screenshots

Before After
Screenshot 2024-09-24 at 5 17 00 PM image

@Divinelink Divinelink force-pushed the fix/material-switch-style-padding branch from 71f1a0d to 07b22f8 Compare September 30, 2024 15:56
@Divinelink Divinelink force-pushed the fix/material-switch-style-padding branch from 07b22f8 to d4bb9d9 Compare September 30, 2024 15:56
@imsodin
Copy link
Member

imsodin commented Sep 30, 2024

Thanks for contributing - it's on my radar (and looks good/unproblematic). Just other/unrelated (procedural) things going on at the moment, thus not merging right away.

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.

2 participants