Started switching user data entry to use autocomplete #1035
Conversation
| User.all.map(&:uid) | ||
| end | ||
|
|
||
| # Users that can be project sponsors | ||
| def self.sponsor_users |
There was a problem hiding this comment.
self.sponsor_users and self.manager_users below now returns a list of User objects (rather than just their uids) so that we can display both the user name and their netid
…till need to fix a few more broken test
hectorcorrea
force-pushed
the
980-single-user-select
branch
from
93b6757
to
cb71b0c
Compare
| @@ -88,3 +88,11 @@ | |||
| end | |||
| end | |||
| end | |||
|
|
|||
There was a problem hiding this comment.
This helper removes the need of calling page.find("body").click as a separate step when validating textboxes since it forces the textbox to lose focus (via tabbing out)
| </div> | ||
| </div> <!-- id=edit --> | ||
|
|
||
| <script> |
There was a problem hiding this comment.
This is the biggest change in this PR: the JavaScript logic to get the autocomplete and the validation to work together.
| # Without removing the focus from the form field, the "change" event is not propagated for the DOM | ||
| page.find("body").click | ||
| click_on "btn-add-rw-user" |
There was a problem hiding this comment.
Notice that clicking on the button to add the user once selected is not longer necessary in the UI (and therefore on the test). The value selected is automatically added as a user.
Updates the user entry elements on the form to use an autocomplete textbox and mimics the validation requested on #980.
This animated GIF shows both the autocomplete and the validation in action:
Closes #980