Move author/date info for Issue#show, MR#show pages to issue-box

Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>

Move author/date info for Issue#show, MR#show pages to issue-box
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
Dmitriy Zaporozhets
1 parent a718a9af
Showing 6 changed files with 31 additions and 25 deletions Show diff stats
app/assets/stylesheets/generic/issue_box.scss
app/views/projects/issues/_issue_context.html.haml
app/views/projects/issues/show.html.haml
app/views/projects/merge_requests/show/_context.html.haml
app/views/projects/merge_requests/show/_mr_box.html.haml
app/views/projects/merge_requests/show/_mr_title.html.haml
@@ -17,6 +17,13 @@
     margin-bottom: 0;
   }
+  .creator {
+    padding: 8px 25px;
+    background: #eee;
+    border-bottom: 1px solid #DDD;
+    color: #777;
+  }
+
   .title {
     font-size: 22px;
     font-weight: 500;
 = form_for [@project, @issue], remote: true, html: {class: 'edit-issue inline-update'} do |f|
-  Created by #{link_to_member(@project, issue.author)}&nbsp;
-  - if issue.assignee
-    \ and currently assigned to
+  %strong.append-right-10
+    Assignee:
   - if can?(current_user, :modify_issue, @issue)
     = project_users_select_tag('issue[assignee_id]', placeholder: 'Select assignee', class: 'custom-form-control', selected: @issue.assignee_id)
   - elsif issue.assignee
     = link_to_member(@project, @issue.assignee)
+  - else
+    None
-
-  .pull-right.hidden-sm.hidden-xs
-    - if issue.milestone
-      - milestone = issue.milestone
-      %cite.cgray Attached to milestone
-
+  .pull-right
+    %strong.append-right-10
+      Milestone:
     - if can?(current_user, :modify_issue, @issue)
       = f.select(:milestone_id, milestone_options(@issue), { include_blank: "Select milestone (none):" }, {class: 'select2 select2-compact'})
-
       = hidden_field_tag :issue_context
       = f.submit class: 'btn'
     - elsif issue.milestone
       = link_to issue.milestone.title, project_milestone_path
+    - else
+      None
 %h3.page-title
   Issue ##{@issue.iid}
-  %small
-    created #{time_ago_with_tooltip(@issue.created_at)}
-
   - if @issue.closed?
     %span.state-label.state-label-red Closed
   - else
@@ -38,6 +35,9 @@
         = @issue.milestone.title
 .issue-box
+  .creator
+    Created by #{link_to_member(@project, @issue.author)} #{time_ago_with_tooltip(@issue.created_at)}
+
   %h4.title
     = gfm escape_once(@issue.title)
 = form_for [@project, @merge_request], remote: true, html: {class: 'edit-merge_request inline-update'} do |f|
-  Created by #{link_to_member(@project, merge_request.author)}&nbsp;
-  - if merge_request.assignee
-    \ and currently assigned to
+  %strong.append-right-10
+    Assignee:
   - if can?(current_user, :modify_merge_request, @merge_request)
     = project_users_select_tag('merge_request[assignee_id]', placeholder: 'Select assignee', class: 'custom-form-control', selected: @merge_request.assignee_id)
   - elsif merge_request.assignee
     = link_to_member(@project, @merge_request.assignee)
+  - else
+    None
-
-  .pull-right.hidden-sm.hidden-xs
-    - if merge_request.milestone
-      - milestone = merge_request.milestone
-      %cite.cgray Attached to milestone
-
+  .pull-right
+    %strong.append-right-10
+      Milestone:
     - if can?(current_user, :modify_merge_request, @merge_request)
       = f.select(:milestone_id, milestone_options(@merge_request), { include_blank: "Select milestone (none):" }, {class: 'select2 select2-compact'})
-
       = hidden_field_tag :merge_request_context
       = f.submit class: 'btn'
     - elsif merge_request.milestone
       = link_to merge_request.milestone.title, project_milestone_path
+    - else
+      None
 .issue-box
+  .creator
+    Created by #{link_to_member(@project, @merge_request.author)} #{time_ago_with_tooltip(@merge_request.created_at)}
+
   %h4.title
     = gfm escape_once(@merge_request.title)
 %h3.page-title
   = "Merge Request ##{@merge_request.iid}"
-  %small
-    created #{time_ago_with_tooltip(@merge_request.created_at)}
   - if @merge_request.merged?
     %span.state-label.state-label-green
	@@ -17,6 +17,13 @@		@@ -17,6 +17,13 @@
17	margin-bottom: 0;	17	margin-bottom: 0;
18	}	18	}
19		19
		20	+ .creator {
		21	+ padding: 8px 25px;
		22	+ background: #eee;
		23	+ border-bottom: 1px solid #DDD;
		24	+ color: #777;
		25	+ }
		26	+
20	.title {	27	.title {
21	font-size: 22px;	28	font-size: 22px;
22	font-weight: 500;	29	font-weight: 500;
1	.issue-box	1	.issue-box
		2	+ .creator
		3	+ Created by #{link_to_member(@project, @merge_request.author)} #{time_ago_with_tooltip(@merge_request.created_at)}
		4	+
2	%h4.title	5	%h4.title
3	= gfm escape_once(@merge_request.title)	6	= gfm escape_once(@merge_request.title)
4		7
1	%h3.page-title	1	%h3.page-title
2	= "Merge Request ##{@merge_request.iid}"	2	= "Merge Request ##{@merge_request.iid}"
3	- %small
4	- created #{time_ago_with_tooltip(@merge_request.created_at)}
5		3
6	- if @merge_request.merged?	4	- if @merge_request.merged?
7	%span.state-label.state-label-green	5	%span.state-label.state-label-green