Angular Action/Trigger Event Definition screen clone dialog buttons should be re-labeled
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Evergreen |
Confirmed
|
Undecided
|
Unassigned |
Bug Description
In the new Angular Action/Trigger Event Definition grid, you can clone an event definition by selecting it, then selecting "Clone selected" from the actions menu.
This brings up a dialog asking whether or not you'd like to bring the environment along with you. It gives two options -- not "Yes" and "No", which would make more sense, but "OK" and "Cancel". "OK" has the effect of bringing the environment, while "Cancel" proceeds with the clone, but does not clone the environment.
I'd recommend that the buttons be relabeled from "OK" to "Yes" and "Cancel" to "No", to better describe the outcome of clicking the button. It could also be nice to have an actual "Cancel" button that stops the cloning process on this modal.
Changed in evergreen: | |
status: | New → Confirmed |
Looking at the code, it looks like that dialog is linked to the eg-confirm-dialog component (Open-ILS/ src/eg2/ src/app/ share/dialog/ comfirm. component. ts). I'm not sure, but I think using an eg-dialog component (.../dialog. component. ts) might allow the desired functionality, if there's an observable that can be passed. Even then, it only allows for two options that I see.