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

Circle and isoline settings review #1627

Open
jfitzpatrick17 opened this issue Oct 30, 2024 · 3 comments
Open

Circle and isoline settings review #1627

jfitzpatrick17 opened this issue Oct 30, 2024 · 3 comments
Assignees
Labels
RFC Request for Comment or Change. UI

Comments

@jfitzpatrick17
Copy link

jfitzpatrick17 commented Oct 30, 2024

Description

What:
The way additional settings for circle and isoline creation (e.g. distance, drivetime in minutes) are accessed needs to be reviewed. The UI is very unclear currently - we could consider separating the settings out into a separate dialog (now we have the UI dialog elements in core).

Why:
Many users are not realising they can change the default settings of a catchment by expanding the drawer; your eyes are immediately drawn to the big 'create' button in the current layout. The expand option is easily missed.

Considerations:

  • If we split this out into a separate dialog, then how will the user know what settings they have applied.

Current UI for review:

Screenshot 2024-10-25 at 16 37 23 (2)

Type of Issue:

  • ✅ New Feature/Usability
@dbauszus-glx
Copy link
Member

Any utility solution must be optional, configurable, and available to any drawer element, to be called from the drawer element method itself.

@jfitzpatrick17
Copy link
Author

@RobAndrewHurst, I was just thinking about your comment this morning in regards to how we will know what settings have been applied if these are handled in a separate dialog - we chatted about the possibility of adding it to the Drawing Instructions dialog that appears on drawing interaction, but this could get confusing especially if there are multiple catchments being drawn.

More and more these days we are capturing user inputs for catchments in the database (e.g. see below), so it could be as simple as adding a hover on this field.

image

@jfitzpatrick17 jfitzpatrick17 linked a pull request Oct 30, 2024 that will close this issue
@cityremade
Copy link
Member

cityremade commented Oct 31, 2024

This is not meant to be a utility. These panels have poor accessibility.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RFC Request for Comment or Change. UI
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants