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

Redesign Survey and Table Top Appbar to meet ODK-X Design Guidelines #429

Open
Redeem-Grimm-Satoshi opened this issue Oct 5, 2023 · 16 comments

Comments

@Redeem-Grimm-Satoshi
Copy link
Member

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:

Screenshot_20231005_170535

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

@Chinex-Boroja
Copy link

@Redeem-Grimm-Satoshi can this task be assigned to me?

@Cveman1
Copy link
Member

Cveman1 commented Oct 6, 2023

Yes, please go ahead to work on it @Chinex-Boroja

@Chinex-Boroja
Copy link

Yes, please go ahead to work on it @Chinex-Boroja

Alright. Thank you, I will update you on the progress

@Chinex-Boroja
Copy link

@Redeem-Grimm-Satoshi @wbrunette, I have set up my environment and cloned all four repositories. I am checking out the apps

@Chinex-Boroja
Copy link

Hi @Cveman1 @Redeem-Grimm-Satoshi, currently on this issue and I have a few things I am confused about while inspecting the code base.

  • In the res folder 📁, checking the values sub-folder, I can't seem to locate the styles.xml where I can change the Top Appbar theme, also the same thing goes for the colors.xml file. I am a bit confused as to where to locate it, or am I to create them in the values folder?
  • Secondly, this is how the Survey Top Appbar looks after running it on my android device, the blue app bar colour is not showing, which is a bit different from the screenshot sent
    Screenshot_2023-10-12-09-05-10-892_org opendatakit survey

@Redeem-Grimm-Satoshi
Copy link
Member Author

Hi @Cveman1 @Redeem-Grimm-Satoshi, currently on this issue and I have a few things I am confused about while inspecting the code base.

* In the `res` folder 📁, checking the `values` sub-folder, I can't seem to locate the `styles.xml` where I can change the Top Appbar theme, also the same thing goes for the `colors.xml` file. I am a bit confused as to where to locate it, or am I to create them in the values folder?

* Secondly, this is how the Survey Top Appbar looks after running it on my android device, the blue app bar colour is not showing, which is a bit different from the screenshot sent
  ![Screenshot_2023-10-12-09-05-10-892_org opendatakit survey](https://user-images.githubusercontent.com/61055200/274667329-3958d265-8edb-426e-ba4c-aef45d3d3643.jpg)
  1. Great question, the app theming resides in the android library ( read more here )

    Android Library Repo

  2. Yeah, Mine was altered don't worry about that

@Chinex-Boroja
Copy link

@Redeem-Grimm-Satoshi thank you. This provides a bit more context.

@Chinex-Boroja
Copy link

Chinex-Boroja commented Oct 13, 2023

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 color.xml file, as described in the ODK-X Design. And also used them in the theme.xml file. But when running the app, the Appbar is not changing or showing the color changes. The color for the status bar is showing. This is the image screenshot
Screenshot_2023-10-14-00-32-46-801_org opendatakit survey

But in Android Studio, I can see the colors properly reflected in the sort.xml file in the menu folder. This is also a screenshot to provide a bit more context

App Bar shot

I have also checked the Manifest file in the survey folder to investigate the app-theme. Probably I am missing something

@Redeem-Grimm-Satoshi
Copy link
Member Author

Great, try to understand how appbars work : https://m2.material.io/components/app-bars-top/android#regular-top-app-bar

@Chinex-Boroja
Copy link

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 figured out how to change the topbar app colour from theme.xml in the androidlibrary module. This is the update so far with a screenshot
Screenshot_2023-10-20-18-44-24-313_org opendatakit survey

  • So is this what I am expected to do? Based on my understanding of the question.
  • Secondly, it seems I am not touching any file on the survey_app, and I just want to find out if I am doing it the right way

I will appreciate a feedback, thank you

@Redeem-Grimm-Satoshi
Copy link
Member Author

@Chinex-Boroja You did great

@Chinex-Boroja
Copy link

Chinex-Boroja commented Oct 23, 2023

@Chinex-Boroja You did great

Thank you @Redeem-Grimm-Satoshi, what should be my next step from here? Raising a Pull Request?

@Chinex-Boroja
Copy link

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 figured out how to change the topbar app colour from theme.xml in the androidlibrary module. This is the update so far with a screenshot Screenshot_2023-10-20-18-44-24-313_org opendatakit survey

  • So is this what I am expected to do? Based on my understanding of the question.
  • Secondly, it seems I am not touching any file on the survey_app, and I just want to find out if I am doing it the right way

I will appreciate a feedback, thank you

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 survey_app. I only made changes to the colors.xml and theme.xml in the androidLibrary. So not sure on how to go about raising the pull request

@Redeem-Grimm-Satoshi
Copy link
Member Author

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 figured out how to change the topbar app colour from theme.xml in the androidlibrary module. This is the update so far with a screenshot Screenshot_2023-10-20-18-44-24-313_org opendatakit survey

  • So is this what I am expected to do? Based on my understanding of the question.
  • Secondly, it seems I am not touching any file on the survey_app, and I just want to find out if I am doing it the right way

I will appreciate a feedback, thank you

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 survey_app. I only made changes to the colors.xml and theme.xml in the androidLibrary. So not sure on how to go about raising the pull request

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

@Chinex-Boroja
Copy link

Thanks for the clarification @Redeem-Grimm-Satoshi

@Chinex-Boroja
Copy link

Chinex-Boroja commented Oct 27, 2023

@Redeem-Grimm-Satoshi I just raised a PR on this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants