Commit fae42270 authored by Luke Bennett's avatar Luke Bennett Committed by Phil Hughes

Resolve "No UI element to edit group label"

parent e28896ce
...@@ -21,8 +21,9 @@ ...@@ -21,8 +21,9 @@
= sprite_icon('star-o') = sprite_icon('star-o')
%button.label-action.remove-priority.btn.btn-transparent.has-tooltip{ title: _('Remove priority'), type: 'button', data: { placement: 'top' }, aria_label: _('Deprioritize label') } %button.label-action.remove-priority.btn.btn-transparent.has-tooltip{ title: _('Remove priority'), type: 'button', data: { placement: 'top' }, aria_label: _('Deprioritize label') }
= sprite_icon('star') = sprite_icon('star')
- if can?(current_user, :admin_label, label)
%li.inline %li.inline
= link_to edit_label_path(label), class: 'btn btn-transparent label-action', aria_label: 'Edit label' do = link_to edit_label_path(label), class: 'btn btn-transparent label-action edit', aria_label: 'Edit label' do
= sprite_icon('pencil') = sprite_icon('pencil')
%li.inline %li.inline
.dropdown .dropdown
...@@ -42,6 +43,7 @@ ...@@ -42,6 +43,7 @@
container: 'body', container: 'body',
toggle: 'modal' } } toggle: 'modal' } }
= _('Promote to group label') = _('Promote to group label')
- if can?(current_user, :admin_label, label)
%li %li
%span{ data: { toggle: 'modal', target: "#modal-delete-label-#{label.id}" } } %span{ data: { toggle: 'modal', target: "#modal-delete-label-#{label.id}" } }
%button.text-danger.remove-row{ type: 'button' }= _('Delete') %button.text-danger.remove-row{ type: 'button' }= _('Delete')
......
require 'spec_helper'
feature 'Group labels' do
let(:user) { create(:user) }
let(:group) { create(:group) }
let!(:label) { create(:group_label, group: group) }
background do
group.add_owner(user)
sign_in(user)
visit group_labels_path(group)
end
scenario 'label has edit button', :js do
expect(page).to have_selector('.label-action.edit')
end
end
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment