diff --git a/app/jsx/account_course_user_search/UsersList.jsx b/app/jsx/account_course_user_search/UsersList.jsx index 007c014cdfa..bc70526828a 100644 --- a/app/jsx/account_course_user_search/UsersList.jsx +++ b/app/jsx/account_course_user_search/UsersList.jsx @@ -17,30 +17,25 @@ define([ var { users } = this.props; return ( -
- - - - - - - - - +
+
+
+ {I18n.t("Name")} +
+
+ {I18n.t("Email")} +
+
+ {I18n.t("SIS ID")} +
+
+ {I18n.t("Last Login")} +
+
-
+
{users.map((user) => )} -
-
- {I18n.t("Name")} - - {I18n.t("Email")} - - {I18n.t("SIS ID")} - - {I18n.t("Last Login")} - -
+
); } diff --git a/app/jsx/account_course_user_search/UsersListRow.jsx b/app/jsx/account_course_user_search/UsersListRow.jsx index 0ce570a0c04..bacf76bf148 100644 --- a/app/jsx/account_course_user_search/UsersListRow.jsx +++ b/app/jsx/account_course_user_search/UsersListRow.jsx @@ -19,31 +19,35 @@ define([ }, render() { - var { accountId, id, name, sis_user_id, email, avatar_url, last_login } = this.props; - var url = `/accounts/${accountId}/users/${id}`; + let { accountId, id, name, sis_user_id, email, avatar_url, last_login } = this.props; + let url = `/accounts/${accountId}/users/${id}`; return ( - - - {!!avatar_url && - - - - } - {name} - - +
+
+ + {!!avatar_url && + + + + } + + + {name} + +
+
{email} - - +
+ +
{sis_user_id} - - +
+ +
{$.datetimeString(last_login)} - - - - +
+
); } }); diff --git a/spec/selenium/admin/new_user_search_spec.rb b/spec/selenium/admin/new_user_search_spec.rb index 9748409f2ef..839f5333855 100644 --- a/spec/selenium/admin/new_user_search_spec.rb +++ b/spec/selenium/admin/new_user_search_spec.rb @@ -14,7 +14,7 @@ describe "new account user search" do end def get_rows - ff('.users-list > tbody > tr') + ff('.users-list div[role=row]') end def click_tab @@ -89,7 +89,7 @@ describe "new account user search" do it "should search by name" do match_user = user_with_pseudonym(:account => @account, :name => "user with a search term") - not_match_user = user_with_pseudonym(:account => @account, :name => "diffrient user") + user_with_pseudonym(:account => @account, :name => "diffrient user") get "/accounts/#{@account.id}" click_tab @@ -102,4 +102,4 @@ describe "new account user search" do expect(rows.count).to eq 1 expect(rows.first).to include_text(match_user.name) end -end \ No newline at end of file +end