Merge pull request #1672 from tsigo/milestone_closed_issues

Milestone "All Issues" filter

app/assets/javascripts/milestones.js.coffee

+$ ->
+  $('.milestone-issue-filter tr[data-closed]').addClass('hide')
+
+  $('.milestone-issue-filter ul.nav li a').click ->
+    $('.milestone-issue-filter li').toggleClass('active')
+    $('.milestone-issue-filter tr[data-closed]').toggleClass('hide')
+    false

app/assets/stylesheets/gitlab_bootstrap/tables.scss

@@ -11,6 +11,11 @@ table {
     border-bottom: 1px solid #bbb;
     text-shadow: 0 1px 1px #fff;
     @include bg-dark-gray-gradient;
+
+    ul.nav {
+      text-shadow: none;
+      margin: 0;
+    }
   }
 
   th, td {

app/controllers/milestones_controller.rb

@@ -30,7 +30,7 @@ class MilestonesController < ProjectResourceController
   end
 
   def show
-    @issues = @milestone.issues.opened.page(params[:page]).per(40)
+    @issues = @milestone.issues
     @users = @milestone.participants
 
     respond_to do |format|

app/views/milestones/show.html.haml

@@ -45,18 +45,20 @@
 
 .row
   .span6
-    %table
+    %table.milestone-issue-filter
       %thead
-        %th Open Issues
+        %th
+          %ul.nav.nav-pills
+            %li.active= link_to('Open Issues', '#')
+            %li=link_to('All Issues', '#')
       - @issues.each do |issue|
-        %tr
+        %tr{data: {closed: issue.closed}}
           %td
             = link_to [@project, issue] do
               %span.badge.badge-info ##{issue.id}
             –
             = link_to_gfm truncate(issue.title, length: 60), [@project, issue]
     %br
-    = paginate @issues, theme: "gitlab"
 
   .span6
     %table

features/project/issues/milestones.feature

@@ -16,3 +16,11 @@ Feature: Project Milestones
     Given I click link "New Milestone"
     And I submit new milestone "v2.3"
     Then I should see milestone "v2.3"
+
+  @javascript
+  Scenario: Listing closed issues
+    Given the milestone has open and closed issues
+    And I click link "v2.2"
+    Then I should see 3 issues
+    When I click link "All Issues"
+    Then I should see 4 issues

features/steps/project/project_milestones.rb

@@ -36,4 +36,26 @@ class ProjectMilestones < Spinach::FeatureSteps
 
     3.times { Factory :issue, :project => project, :milestone => milestone }
   end
+
+  Given 'the milestone has open and closed issues' do
+    project = Project.find_by_name("Shop")
+    milestone = project.milestones.find_by_title('v2.2')
+
+    # 3 Open issues created above; create one closed issue
+    create(:closed_issue, project: project, milestone: milestone)
+  end
+
+  When 'I click link "All Issues"' do
+    click_link 'All Issues'
+  end
+
+  Then "I should see 3 issues" do
+    page.should have_selector('.milestone-issue-filter tbody tr', count: 4)
+    page.should have_selector('.milestone-issue-filter tbody tr.hide', count: 1)
+  end
+
+  Then "I should see 4 issues" do
+    page.should have_selector('.milestone-issue-filter tbody tr', count: 4)
+    page.should_not have_selector('.milestone-issue-filter tbody tr.hide')
+  end
 end