CrisisCleanup / crisiscleanup-android

Current Crisis Cleanup native Android
2 stars 0 forks source link

Case View Updates #5

Open aarontitus opened 1 year ago

aarontitus commented 1 year ago

See numbers corresponding to the red numbers on the screenshot.

  1. I like the "Offline" icon. Please tint it red. When you are online, please turn it into a green (or gray) cloud (or cloud with a checkbox, or something like that) to let people know they're online.
  2. Add the number of photos in parenthesis "Photos (0)" or "Photos (23)"
  3. Add the number of notes in parenthesis "Notes (0)" or "Notes (5)"
  4. When you click and hold down on the case number, automatically copy it to the clipboard. Add alert "Copied to Clipboard."
  5. When you click and hold down on the person's name, automatically copy it to the clipboard. Add alert "Copied to Clipboard."
  6. A. This is blank for some reason, but it shouldn't be. Also, Hold down to copy to clipboard. Add alert "Copied to Clipboard."
  7. Click to open in a map app; long click to copy to clipboard. Add alert "Copied to Clipboard."
  8. The "Back" button should include a left caret, and the link should be blue (see Figma).
  9. The top-right most button is an alert, so it looks like the the app is alerting that something is wrong. Switch positions and put the heart in the top-right.
  10. For some reason email is not showing up. Is that on purpose?

Image

hueachilles commented 1 year ago
  1. and 10. Are due to cached data not saving the phone and email. I'm currently trying to solve it with the paging endpoint providing that data so the entire worksite doesn't need to be cached ahead of time.
  2. I'll make the change. Should the designs in Figma be the source of truth for UI? Or will those be initial references and we'll make changes to the app directly?
aarontitus commented 1 year ago

Yes, the designs in Figma should be considered the source of truth. We can deviate if the designs are incomplete or insufficient.

arroyoDev commented 1 year ago

All other items are looking and working good.

hueachilles commented 1 year ago
  1. Copying address works it's just the icon needs to be long pressed. Long pressing the address text currently won't work because it opens to Maps when (single) pressed. There may be a way to configure long pressing a link/button but it will take some time.