Commit 7bf779ab authored by Phil Hughes's avatar Phil Hughes

Merge branch 'fix-edit-merge-request-button-case' into 'master'

Fix edit merge request button inconsistent letter casing

Closes #34086

See merge request !13548
parents ef0e3c8e 71130ba7
= link_to params.merge(rss_url_options), class: 'btn btn-default append-right-10 has-tooltip', title: 'Subscribe' do = link_to params.merge(rss_url_options), class: 'btn btn-default append-right-10 has-tooltip', title: 'Subscribe' do
= icon('rss') = icon('rss')
- if @can_bulk_update - if @can_bulk_update
= button_tag "Edit Issues", class: "btn btn-default append-right-10 js-bulk-update-toggle" = button_tag "Edit issues", class: "btn btn-default append-right-10 js-bulk-update-toggle"
= link_to "New issue", new_project_issue_path(@project, = link_to "New issue", new_project_issue_path(@project,
issue: { assignee_id: issues_finder.assignee.try(:id), issue: { assignee_id: issues_finder.assignee.try(:id),
milestone_id: issues_finder.milestones.first.try(:id) }), milestone_id: issues_finder.milestones.first.try(:id) }),
......
- if @can_bulk_update - if @can_bulk_update
= button_tag "Edit Merge Requests", class: "btn js-bulk-update-toggle" = button_tag "Edit merge requests", class: "btn js-bulk-update-toggle"
- if merge_project - if merge_project
= link_to new_merge_request_path, class: "btn btn-new", title: "New merge request" do = link_to new_merge_request_path, class: "btn btn-new", title: "New merge request" do
New merge request New merge request
---
title: Fix edit merge request and issues button inconsistent letter casing
merge_request:
author:
type: fixed
...@@ -353,7 +353,7 @@ feature 'Issues > Labels bulk assignment' do ...@@ -353,7 +353,7 @@ feature 'Issues > Labels bulk assignment' do
context 'cannot bulk assign labels' do context 'cannot bulk assign labels' do
it do it do
expect(page).not_to have_button 'Edit Issues' expect(page).not_to have_button 'Edit issues'
expect(page).not_to have_css '.check-all-issues' expect(page).not_to have_css '.check-all-issues'
expect(page).not_to have_css '.issue-check' expect(page).not_to have_css '.issue-check'
end end
...@@ -411,7 +411,7 @@ feature 'Issues > Labels bulk assignment' do ...@@ -411,7 +411,7 @@ feature 'Issues > Labels bulk assignment' do
def enable_bulk_update def enable_bulk_update
visit project_issues_path(project) visit project_issues_path(project)
click_button 'Edit Issues' click_button 'Edit issues'
end end
def disable_bulk_update def disable_bulk_update
......
...@@ -14,7 +14,7 @@ feature 'Multiple issue updating from issues#index', :js do ...@@ -14,7 +14,7 @@ feature 'Multiple issue updating from issues#index', :js do
it 'sets to closed' do it 'sets to closed' do
visit project_issues_path(project) visit project_issues_path(project)
click_button 'Edit Issues' click_button 'Edit issues'
find('#check-all-issues').click find('#check-all-issues').click
find('.js-issue-status').click find('.js-issue-status').click
...@@ -27,7 +27,7 @@ feature 'Multiple issue updating from issues#index', :js do ...@@ -27,7 +27,7 @@ feature 'Multiple issue updating from issues#index', :js do
create_closed create_closed
visit project_issues_path(project, state: 'closed') visit project_issues_path(project, state: 'closed')
click_button 'Edit Issues' click_button 'Edit issues'
find('#check-all-issues').click find('#check-all-issues').click
find('.js-issue-status').click find('.js-issue-status').click
...@@ -41,7 +41,7 @@ feature 'Multiple issue updating from issues#index', :js do ...@@ -41,7 +41,7 @@ feature 'Multiple issue updating from issues#index', :js do
it 'updates to current user' do it 'updates to current user' do
visit project_issues_path(project) visit project_issues_path(project)
click_button 'Edit Issues' click_button 'Edit issues'
find('#check-all-issues').click find('#check-all-issues').click
click_update_assignee_button click_update_assignee_button
...@@ -57,7 +57,7 @@ feature 'Multiple issue updating from issues#index', :js do ...@@ -57,7 +57,7 @@ feature 'Multiple issue updating from issues#index', :js do
create_assigned create_assigned
visit project_issues_path(project) visit project_issues_path(project)
click_button 'Edit Issues' click_button 'Edit issues'
find('#check-all-issues').click find('#check-all-issues').click
click_update_assignee_button click_update_assignee_button
...@@ -73,7 +73,7 @@ feature 'Multiple issue updating from issues#index', :js do ...@@ -73,7 +73,7 @@ feature 'Multiple issue updating from issues#index', :js do
it 'updates milestone' do it 'updates milestone' do
visit project_issues_path(project) visit project_issues_path(project)
click_button 'Edit Issues' click_button 'Edit issues'
find('#check-all-issues').click find('#check-all-issues').click
find('.issues-bulk-update .js-milestone-select').click find('.issues-bulk-update .js-milestone-select').click
...@@ -89,7 +89,7 @@ feature 'Multiple issue updating from issues#index', :js do ...@@ -89,7 +89,7 @@ feature 'Multiple issue updating from issues#index', :js do
expect(first('.issue')).to have_content milestone.title expect(first('.issue')).to have_content milestone.title
click_button 'Edit Issues' click_button 'Edit issues'
find('#check-all-issues').click find('#check-all-issues').click
find('.issues-bulk-update .js-milestone-select').click find('.issues-bulk-update .js-milestone-select').click
......
...@@ -98,7 +98,7 @@ feature 'Multiple merge requests updating from merge_requests#index' do ...@@ -98,7 +98,7 @@ feature 'Multiple merge requests updating from merge_requests#index' do
end end
def change_status(text) def change_status(text)
click_button 'Edit Merge Requests' click_button 'Edit merge requests'
find('#check-all-issues').click find('#check-all-issues').click
find('.js-issue-status').click find('.js-issue-status').click
find('.dropdown-menu-status a', text: text).click find('.dropdown-menu-status a', text: text).click
...@@ -106,7 +106,7 @@ feature 'Multiple merge requests updating from merge_requests#index' do ...@@ -106,7 +106,7 @@ feature 'Multiple merge requests updating from merge_requests#index' do
end end
def change_assignee(text) def change_assignee(text)
click_button 'Edit Merge Requests' click_button 'Edit merge requests'
find('#check-all-issues').click find('#check-all-issues').click
find('.js-update-assignee').click find('.js-update-assignee').click
wait_for_requests wait_for_requests
...@@ -119,7 +119,7 @@ feature 'Multiple merge requests updating from merge_requests#index' do ...@@ -119,7 +119,7 @@ feature 'Multiple merge requests updating from merge_requests#index' do
end end
def change_milestone(text) def change_milestone(text)
click_button 'Edit Merge Requests' click_button 'Edit merge requests'
find('#check-all-issues').click find('#check-all-issues').click
find('.issues-bulk-update .js-milestone-select').click find('.issues-bulk-update .js-milestone-select').click
find('.dropdown-menu-milestone a', text: text).click find('.dropdown-menu-milestone a', text: text).click
......
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