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

feat: DrawBoundary map uses trash bin icon stead of arrow #1306

Merged
merged 1 commit into from
Dec 4, 2022

Conversation

jessicamcinchak
Copy link
Member

User-research wasn't particularly conclusive here, so we made a decision to go this route as it's most distinct from current option:

What should we change draw outline icon from ↩️ to ?
Tested in R.45 with 6 users.
Reset icon: 3/6 users prefer this icon, but a couple of them expect it to only delete one stroke at a time. Also, a user thinks it looks like the refresh button refreshes the whole page
Trash icon: 2/6 prefer this. Users Understood the ‘trash icon’ as ‘Delete’/ ‘Restart’. It feels clear what the button does but a couple of users don’t like it as it feels ‘too serious’/' too strong’
Rubber icon: Only 1 user perfer this and understood this as a rubber. It is not clear what the symbol means to most users. (edited)

@jessicamcinchak
Copy link
Member Author

@github-actions
Copy link

github-actions bot commented Dec 4, 2022

Removed vultr server and associated DNS entries

@github-actions
Copy link

github-actions bot commented Dec 4, 2022

Lighthouse

https://1306.planx.pizza/buckinghamshire/find-out-if-you-need-planning-permission/preview

  • performance: 0.27
  • accessibility: 1
  • best-practices: 1
  • seo: 0.89
  • pwa: 0.7
  • Full report

https://1306.planx.pizza/testing/lighthouse-canary-flow/unpublished

  • performance: 0.28
  • accessibility: 1
  • best-practices: 0.92
  • seo: 0.92
  • pwa: 0.7
  • Full report

@jessicamcinchak jessicamcinchak merged commit eb23acc into main Dec 4, 2022
@jessicamcinchak jessicamcinchak deleted the jess/update-draw-map-resetControlImage branch December 4, 2022 19:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants