search

WildAid / o-fish-android

Android app for the Officer's Fishery Information Sharing Hub (O-FISH). The mobile app allows fisheries officers to document and share critical information gathered during a routine vessel inspection.
Apache License 2.0
22 stars 24 forks source link

Dark Mode for draft boardings page #371

Closed o-fish-wildaid closed 3 years ago

o-fish-wildaid commented 3 years ago

This issue depends on #368 being completed.

When Dark Mode is on - the draft boardings page should look like this: https://mongodb.invisionapp.com/share/PCYZ6ZW7FR8#/screens/433591780_Draft_Boardings

Color key for light mode: https://mongodb.invisionapp.com/share/PCYZ6ZW7FR8#/screens/433591768_Color_Key_-_Light_Mode Color key for dark mode: https://mongodb.invisionapp.com/share/PCYZ6ZW7FR8#/screens/433591769_Color_Key_-_Dark_Mode Component Color specs for light mode: https://mongodb.invisionapp.com/share/PCYZ6ZW7FR8#/screens/433591770_Component_Colors_-_Light_Mode Component Color specs for dark mode: https://mongodb.invisionapp.com/share/PCYZ6ZW7FR8#/screens/433591771_Component_Colors_-_Dark_Mode

Both dark and light modes should follow the color key and specs above (it is possible that the current light mode is not), and colors should be stored in: app/src/main/res/values-night/colors.xml app/src/main/res/values/colors.xml

Please make sure to TEST your build on a device or emulator to verify everything has changed - you can get a login and realm-app-id with easy instructions at https://wildaid.github.io/contribute/sandbox.html

ippschi commented 3 years ago

I would like to tackle this one

ippschi commented 3 years ago

Actually, after seeing what the previous contributors did, I would like you to consider to not let multiple people do those dark mode tasks. It seems that the available colors from the design library are not reused but always defined again and again.

Sheeri commented 3 years ago

@ippschi yes, I realized that mistake too late. I have cleaned up some of it, and added to the description:

Both dark and light modes should follow the color key and specs above (it is possible that the current light mode is not), and colors should be stored in: app/src/main/res/values-night/colors.xml app/src/main/res/values/colors.xml

If you would like to pick this up again, it would be great - since you're already aware of the pitfalls and will make sure to code it optimally, reusing the available colors.

ippschi commented 3 years ago

Yes, I will work on this.

Sheeri commented 3 years ago

Thanks, Andre, I have assigned this issue to you.

ippschi commented 3 years ago

Since I wrote about my concerns regarding the colors and the future use of them I just created a pull request with the proposal of having a separate colors file for the "brand" colors. Basically all the colors defined by the designers in invisionapp. (This can also be exported if not mistaken).

ippschi commented 3 years ago

The PR for the fix was also added