-
Notifications
You must be signed in to change notification settings - Fork 38
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
Refactor RecyclerView item decoration in AboutFragment and SettingsFr… #180
Refactor RecyclerView item decoration in AboutFragment and SettingsFr… #180
Conversation
…agment - Extracted the custom RecyclerView item decoration logic into a separate class called CustomDividerItemDecoration. - Added the CustomDividerItemDecoration to the RecyclerViews in AboutFragment and SettingsFragment. - Applied a custom drawable as the divider with a width of 1dp and rounded corners. - Set the left and right margins of the dividers to 50dp. Fixes scribe-org#139
Thank you for the pull request!The Scribe team will do our best to address your contribution as soon as we can. The following is a checklist for maintainers to make sure this process goes as well as possible. Feel free to address the points below yourself in further commits if you realize that actions are needed :) If you're not already a member of our public Matrix community, please consider joining! We'd suggest using Element as your Matrix client, and definitely join the General and Android rooms once you're in. Also consider joining our bi-weekly Saturday dev syncs. It'd be great to have you! Maintainer checklist
|
do i have a permission to resolve conflicts ? |
You should be able to merge main into your branch in your IDE and then resolve them locally, @MostafaMohamed2002 :) |
Another quick thing here, @MostafaMohamed2002: On a Pixel 8 Pro emulator the lines between the languages in settings seem to be button outlines: Could we get that to just be an individual line and not having the sides as well? |
I'll check on it and fix it 😃 |
Thanks, @MostafaMohamed2002! |
Nice, looking great! Do you want to fix the merge conflict and I'll bring it in? :) |
thanks , but i need to take alook at language settings fragment 😊 |
it's ready to merge , you can have a final look if it's necessary 😊 |
84d0352
to
597876d
Compare
when resolving the conflict i should choose the right and left side , i accepted the left only 😂 |
So i should add |
No it's all ok now 👌 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking great, @MostafaMohamed2002! 😊 Really appreciate your continued support for the project 🚀
…agment
Fixes #139
Contributor checklist
Description
Related issue
#139