[Enhancements] Phases
Closed this issue · 2 comments
mateoclarke commented
These are follow-up tasks and feature enhancement requests that spawned from cityofaustin/atd-moped#187
Quick fixes
- Can you transform the UI values for "Active?" to Yes/No instead of True/False? Fine with true/false in the database but I think generally yes/no is preferable for user-facing copy. (also fine with changing this in the database if that's easier, will let you decide that). Actually, a toggle between yes/no would be very nice, but no worries if that's difficult.
- Default sorting: can you make this sort chronologically from newest to oldest phase instead of by phase name?
Larger enhancements
- User cannot add a phase name that's already been added. [Pending]
- If a user adds an active phase, set any other active phases to inactive. (I'm not positive of this requirement, let's discuss it as a group, but my instinct is to say that two phases shouldn't be simultaneously active)
- Current phase stored in moped_projects should automatically reflect the newest phase chronologically where "active is true" [#5309]
- What do you think about moving the "+" from the upper right to below the table as a FAB? (Similar to the "Assign team" page...)
- it would be nice if the date-picker calendar opened automatically when a date field is active. if y'all agree, we might move this into a separate issue to address it globally since we need this enhancement in Sergio's advanced table filters and will need it elsewhere.
Other feedback
- When I first clicked the add button to add in a phase, I stupidly forgot to click the checkmark to save and lost the row. Totally user error but I'm just flagging it in case we run into that error during user testing.
- The slight lag between entering a new row and saving it, and then the row appearing is something I'm noting. Is this because we're waiting to see if the mutation is successful (otherwise it would display an error)? Really not a big deal but just thought I'd point out if anyone else thought anything of it.
- I'm unsure about all the pagination and search controls, and think users may want to see modified by/modified date columns here—but we can table that pending user testing.
johnclary commented
Related to #5309, I find it confusing to select the "Current Phase" during project creation. Possible alternatives:
- Remove this entirely. The user can add a phase later.
- Automatically create a new phase record based on the value the user selects during project creation
- Add a step in the new project workflow which enables the user to add one or more phases
amenity commented
- User cannot add a phase name that's already been added.
We're opting to let users add phases multiple times for now. No extra business logic unless warranted by user feedback or usability testing. - If a user adds an active phase, set any other active phases to inactive.
Addressed in cityofaustin/atd-moped#274 - Current phase stored in moped_projects should automatically reflect the newest phase chronologically where "active is true" [#5309]
- What do you think about moving the "+" from the upper right to below the table as a [FAB]
This will be covered by #5375 - it would be nice if the date-picker calendar opened automatically when a date field is active. if y'all agree, we might move this into a separate issue to address it globally since we need this enhancement in Sergio's advanced table filters and will need it elsewhere.
Adding this as a suggestion to #5375
Other feedback
- When I first clicked the add button to add in a phase, I stupidly forgot to click the checkmark to save and lost the row. Totally user error but I'm just flagging it in case we run into that error during user testing.
This was covered in cityofaustin/atd-moped#268 - The slight lag between entering a new row and saving it, and then the row appearing is something I'm noting. Is this because we're waiting to see if the mutation is successful (otherwise it would display an error)? Really not a big deal but just thought I'd point out if anyone else thought anything of it.
Adding this as a suggestion to #5375 - I'm unsure about all the pagination and search controls, and think users may want to see modified by/modified date columns here—but we can table that pending user testing.
Adding this as a suggestion to #5375