diff --git a/app/assets/stylesheets/application.scss b/app/assets/stylesheets/application.scss index 2e7657d10..8f9deb906 100644 --- a/app/assets/stylesheets/application.scss +++ b/app/assets/stylesheets/application.scss @@ -385,3 +385,7 @@ div.form-subcommunities .form-check{ table.dataTable { overflow-wrap: anywhere } + +.user-edit-long-field { + width: 400px; +} diff --git a/app/views/users/_form.html.erb b/app/views/users/_form.html.erb index fe03a55e1..d8735b3e8 100644 --- a/app/views/users/_form.html.erb +++ b/app/views/users/_form.html.erb @@ -9,7 +9,7 @@ -
+

<%= t('users.form.group') %>

<% @user.submitter_or_admin_groups.each do |group| %> @@ -75,14 +75,19 @@ <%= form.text_field :orcid, placeholder: "0000-0000-0000-0000" %> +
+ Email
+ +
+
Princeton Net ID
- +
Default group
- +
diff --git a/spec/system/user_edit_spec.rb b/spec/system/user_edit_spec.rb index 1c76da5d2..1ca719c16 100644 --- a/spec/system/user_edit_spec.rb +++ b/spec/system/user_edit_spec.rb @@ -16,6 +16,16 @@ fill_in "user_orcid", with: orcid click_on "Save" end + + it "shows user fields", js: true do + visit edit_user_path(user_admin) + # These fields are disabled so we cannot target them with + # RSpec `have_field`, but once we make them editable we + # could use the more common syntax. + expect(page.html.include?(user_admin.email)).to be true + expect(page.html.include?(user_admin.uid)).to be true + expect(page.html.include?(user_admin.default_group.title)).to be true + end end describe "Non-admin users cannot access others people data" do