-
Notifications
You must be signed in to change notification settings - Fork 43
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
Redesign Survey and Table Top Appbar to meet ODK-X Design Guidelines #429
Comments
@Redeem-Grimm-Satoshi can this task be assigned to me? |
Yes, please go ahead to work on it @Chinex-Boroja |
Alright. Thank you, I will update you on the progress |
@Redeem-Grimm-Satoshi @wbrunette, I have set up my environment and cloned all four repositories. I am checking out the apps |
Hi @Cveman1 @Redeem-Grimm-Satoshi, currently on this issue and I have a few things I am confused about while inspecting the code base.
|
|
@Redeem-Grimm-Satoshi thank you. This provides a bit more context. |
Hi @Redeem-Grimm-Satoshi @Cveman1 and @wbrunette , I am stuck on the Top Appbar color not reflecting. I have added the color values in the But in Android Studio, I can see the colors properly reflected in the I have also checked the Manifest file in the survey folder to investigate the app-theme. Probably I am missing something |
Great, try to understand how appbars work : https://m2.material.io/components/app-bars-top/android#regular-top-app-bar |
Thank you. I still have some question @Redeem-Grimm-Satoshi
I will appreciate a feedback, thank you |
@Chinex-Boroja You did great |
Thank you @Redeem-Grimm-Satoshi, what should be my next step from here? Raising a Pull Request? |
Hi @Redeem-Grimm-Satoshi I am still expecting a response to the second question because it seems I did not touch any file from the |
Yes you didn't because Android Library is where other ODK-X apps pull their resources from ( eg theming, etc.. )... A PR can be sent to the Android Library Repo |
Thanks for the clarification @Redeem-Grimm-Satoshi |
@Redeem-Grimm-Satoshi I just raised a PR on this issue |
1. Clone all 4 repositories locally into Android Studio ( Tables, Survey, Services and Android Library ) into the same folder, check readme https://github.com/odk-x/survey and make sure they're running without any errors
Survey Top Appbar currently looks like this:
As you can see, It currently doesn't conform to ODK-X Design Guidelines
2. Make changes to the Top Appbar so it conform to ODK-X Design Guidelines,
3. Send a Pull Request to Survey repository ( Development Branch ) with the changes and also a screenshot
###Read the docs well so you know where to make required changes
The text was updated successfully, but these errors were encountered: