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

Updated code to newer android compatibles, Updated error 9-patch #5

Merged
merged 1 commit into from
Oct 5, 2023

Conversation

OmarShoaib
Copy link
Contributor

Hi Ali,

First of all, this repository of yours is literally amazing and helpful. While using this library I encountered an issue in which error image you used messes up the UI as the field gets bigger because of this image on error prompt. So I've updated that image plus upgraded your project to run on newer android versions.

It'll be really grateful to you if you could accept this pull request.

Please let me know if you want to ask anything. I'm just a message away.

Thanks.

@AliAzaz
Copy link
Owner

AliAzaz commented Sep 27, 2023

Thanks a lot, @OmarShoaib for contributing to this library and your support in giving back to the community. Shoaib, it has been quite some time that I wasn't maintaining this library due to that you people are facing issues.
@OmarShoaib you mentioned that you are experiencing some UI distortion issues, can you please share any screenshots for better understanding?

@AliAzaz AliAzaz self-requested a review September 28, 2023 14:19
@AliAzaz AliAzaz added the good first issue Good for newcomers label Sep 28, 2023
@OmarShoaib
Copy link
Contributor Author

OmarShoaib commented Oct 2, 2023

Screenshot_20231002-112925_ITRIPP 1

For your reference.

@AliAzaz
Copy link
Owner

AliAzaz commented Oct 2, 2023

@OmarShoaib I got it and thanks a lot for pointing out this issue. Can you please remove .idea files from your PR so, I can merge it?

@AliAzaz AliAzaz self-assigned this Oct 2, 2023
@AliAzaz AliAzaz added the hacktoberfest-accepted This PR is accepted in hacktoberfest label Oct 5, 2023
@AliAzaz
Copy link
Owner

AliAzaz commented Oct 5, 2023

I am going this PR and will generate the release in a while

Copy link
Owner

@AliAzaz AliAzaz left a comment

Choose a reason for hiding this comment

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

.idea files are not to be added while creating PR in any project because these files are generated by Android Studio where it is installed.

@AliAzaz AliAzaz merged commit f92f09b into AliAzaz:master Oct 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers hacktoberfest-accepted This PR is accepted in hacktoberfest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants