techmatters / terraso-mobile-client

GNU Affero General Public License v3.0
2 stars 0 forks source link

Styling Feedback: for Transfer Sites workflow (Project view) #570

Open ltseng opened 8 months ago

ltseng commented 8 months ago

Updates to the experience for transferring sites to projects (from the project view)

To find this screen

  1. Open a project where you are a manager
  2. Select the "Sites" tab
  3. Select "Transfer Sites"

References

See Figma

Courtney's feedback can be found across two docs - this one and this one.

### Tasks
- [x] Main sites tab showing empty state or list of site cards background should be grey/300
- [ ] iOS checkboxes should be squares (align with design)
- [x] Transfer Sites screen with list of possible sites for transfer needs white background
- [x] Text description “Transfer LandPKS sites you manage to this project. When you transfer sites, they will use the data settings and team member roles from this project.”
- [ ] disabled button styling: button color #E0E0E0
- [ ] disabled button styling: Text Color #1A202C 38%
- [ ] disabled button styling: remove drop shadow
- [ ] Accordion headers should have 1px margin or border between them when closed.

iOS checkboxes

CourtneyLee333 commented 5 months ago

@ltseng The issue description is incorrect. The guiding principle here is that the screen background is grey when we will be showing site or project cards on it - the cards are white, so the grey background helps provide contrast to differentiate each card. So the first two items in your list should be:

Tooltip text was updated in the content audit since the document you linked. It should be "When you transfer sites, they will use the data settings and team member roles from this project."

Disabled button also needs the correct word: Transfer.

It might be better to link to my most recent QA document to avoid confusion: https://docs.google.com/document/d/1mnFPTAO0jrYgI0A-pxqGxKGmCzyD3qC7UxVzBAEYZOw/edit?usp=sharing

ltseng commented 5 months ago

Paul has expressly requested that I copy the exact changes to be made into the issue body itself instead of linking documents. I have made your edits to the issue description above though, thank you!