diff --git a/app/assets/stylesheets/components/settings.css b/app/assets/stylesheets/components/settings.css index f6ac449..f544ebf 100644 --- a/app/assets/stylesheets/components/settings.css +++ b/app/assets/stylesheets/components/settings.css @@ -22,6 +22,22 @@ color: var(--color-text-primary); } +.settings-page .settings-hint { + margin-bottom: var(--spacing-4); + color: var(--color-text-secondary); + font-size: var(--font-size-sm); +} + +.settings-page fieldset[disabled] { + opacity: 0.6; + cursor: not-allowed; +} + +.settings-page fieldset[disabled] input, +.settings-page fieldset[disabled] button { + cursor: not-allowed; +} + .settings-warning { margin-bottom: var(--spacing-4); color: var(--color-warning-text); diff --git a/app/controllers/settings/teams_controller.rb b/app/controllers/settings/teams_controller.rb index 992f8f0..3c619ef 100644 --- a/app/controllers/settings/teams_controller.rb +++ b/app/controllers/settings/teams_controller.rb @@ -15,7 +15,7 @@ def index def show @team_members = @team.team_members.includes(:user) @can_manage = user_signed_in? && @team.admin?(current_user) - @can_invite = user_signed_in? && (@team.member?(current_user) || @team.admin?(current_user)) + @can_invite = user_signed_in? && @team.admin?(current_user) end def create diff --git a/app/views/settings/teams/show.html.slim b/app/views/settings/teams/show.html.slim index d444954..0d6cf0d 100644 --- a/app/views/settings/teams/show.html.slim +++ b/app/views/settings/teams/show.html.slim @@ -21,10 +21,12 @@ - else p No members yet. - - if @can_invite - .invite-section - h2 Add member - = form_with url: settings_team_team_members_path(@team), method: :post, local: true do |f| + .invite-section + h2 Add member + - unless @can_invite + p.settings-hint Only team admins can add members. + = form_with url: settings_team_team_members_path(@team), method: :post, local: true do |f| + fieldset disabled=(!@can_invite) .form-group = label_tag :username, "Username" = text_field_tag :username, nil, required: true, placeholder: "existing_username"