changeset 3814:8854ddd51b48 beta

pullrequests: reorganize PR creation page - make it more like the view page
author Mads Kiilerich <madski@unity3d.com>
date Mon, 08 Apr 2013 23:34:02 +0200
parents dca89d578c70
children 8b1f64722d46
files rhodecode/templates/pullrequests/pullrequest.html
diffstat 1 files changed, 83 insertions(+), 63 deletions(-) [+]
line wrap: on
line diff
--- a/rhodecode/templates/pullrequests/pullrequest.html	Mon Apr 08 22:29:29 2013 +0200
+++ b/rhodecode/templates/pullrequests/pullrequest.html	Mon Apr 08 23:34:02 2013 +0200
@@ -19,73 +19,15 @@
     <div class="title">
         ${self.breadcrumbs()}
     </div>
-    ${h.form(url('pullrequest', repo_name=c.repo_name), method='post', id='pull_request_form')}
-    <div style="float:left;padding:0px 30px 30px 30px">
-        ##ORG
-        <div style="float:left">
-            <div>
-                <span style="font-size: 20px">
-                ${h.select('org_repo','',c.org_repos,class_='refs')}:${h.select('org_ref',c.default_org_ref,c.org_refs,class_='refs')}
-                </span>
-                 <div style="padding:5px 3px 3px 20px;">${c.rhodecode_db_repo.description}</div>
-            </div>
-            <div style="clear:both;padding-top: 10px"></div>
-        </div>
-          <div style="float:left;font-size:24px;padding:0px 20px">
-          <img height=32 width=32 src="${h.url('/images/arrow_right_64.png')}"/>
-          </div>
 
-        ##OTHER, most Probably the PARENT OF THIS FORK
-        <div style="float:left">
-            <div>
-                <span style="font-size: 20px">
-                ${h.select('other_repo',c.default_other_repo,c.other_repos,class_='refs')}:${c.default_other_repo_info['revs']}
-                </span>
-                 <div id="other_repo_desc" style="padding:5px 3px 3px 20px;"></div>
-            </div>
-            <div style="clear:both;padding-top: 10px"></div>
-        </div>
-       <div style="clear:both;padding-top: 10px"></div>
-       ## overview pulled by ajax
-       <div style="float:left" id="pull_request_overview"></div>
-       <div style="float:left;clear:both;padding:10px 10px 10px 0px;display:none">
-            <a id="pull_request_overview_url" href="#">${_('Detailed compare view')}</a>
-       </div>
-     </div>
-    <div style="float:left; border-left:1px dashed #eee">
-        <h4>${_('Pull request reviewers')}</h4>
-        <div id="reviewers" style="padding:0px 0px 0px 15px">
-          ## members goes here !
-          <div class="group_members_wrap">
-            <ul id="review_members" class="group_members">
-            %for member in [c.default_other_repo_info['user']]:
-              <li id="reviewer_${member['user_id']}">
-                <div class="reviewers_member">
-                  <div class="gravatar"><img alt="gravatar" src="${member['gravatar_link']}"/> </div>
-                  <div style="float:left">${member['firstname']} ${member['lastname']} (${_('owner')})</div>
-                  <input type="hidden" value="${member['user_id']}" name="review_members" />
-                  <span class="delete_icon action_button" onclick="removeReviewMember(${member['user_id']})"></span>
-                </div>
-              </li>
-            %endfor
-            </ul>
-          </div>
+    ${h.form(url('pullrequest', repo_name=c.repo_name), method='post', id='pull_request_form')}
 
-          <div class='ac'>
-            <div class="reviewer_ac">
-               ${h.text('user', class_='yui-ac-input')}
-               <span class="help-block">${_('Add reviewer to this pull request.')}</span>
-               <div id="reviewers_container"></div>
-            </div>
-          </div>
-        </div>
-    </div>
     <h3>${_('Create new pull request')}</h3>
 
     <div class="form">
         <!-- fields -->
 
-        <div class="fields">
+        <div class="fields" style="float:left;width:50%;padding-right:30px;">
 
              <div class="field">
                 <div class="label">
@@ -105,12 +47,90 @@
                 </div>
             </div>
 
-            <div class="buttons">
-                ${h.submit('save',_('Send pull request'),class_="ui-btn large")}
-                ${h.reset('reset',_('Reset'),class_="ui-btn large")}
+            <div class="field">
+                <div class="label label-textarea">
+                    <label for="pullrequest_desc">${_('Changeset flow')}:</label>
+                </div>
+                <div class="input">
+                    ##ORG
+                    <div style="float:left">
+                        <div>
+                            <span style="font-size: 20px">
+                            ${h.select('org_repo','',c.org_repos,class_='refs')}:${h.select('org_ref',c.default_org_ref,c.org_refs,class_='refs')}
+                            </span>
+                            <div style="padding:5px 3px 3px 20px;">${c.rhodecode_db_repo.description}</div>
+                        </div>
+                    </div>
+                    <div style="float:left;font-size:24px;padding:0px 20px">
+                        <img height=32 width=32 src="${h.url('/images/arrow_right_64.png')}"/>
+                    </div>
+
+                    ##OTHER, most Probably the PARENT OF THIS FORK
+                    <div style="float:left">
+                        <div>
+                            <span style="font-size: 20px">
+                            ${h.select('other_repo',c.default_other_repo,c.other_repos,class_='refs')}:${c.default_other_repo_info['revs']}
+                            </span>
+                            <div id="other_repo_desc" style="padding:5px 3px 3px 20px;"></div>
+                        </div>
+                    </div>
+                    <div style="clear:both"></div>
+                </div>
+            </div>
+
+            <div class="field">
+                <div class="buttons">
+                    ${h.submit('save',_('Send pull request'),class_="ui-btn large")}
+                    ${h.reset('reset',_('Reset'),class_="ui-btn large")}
+               </div>
+            </div>
+
+        </div>
+
+        ## Reviewers
+        <div style="float:left; border-left:1px dashed #eee">
+            <h4>${_('Pull request reviewers')}</h4>
+            <div id="reviewers" style="padding:0px 0px 0px 15px">
+              ## members goes here !
+              <div class="group_members_wrap">
+                <ul id="review_members" class="group_members">
+                %for member in [c.default_other_repo_info['user']]:
+                  <li id="reviewer_${member['user_id']}">
+                    <div class="reviewers_member">
+                      <div class="gravatar"><img alt="gravatar" src="${member['gravatar_link']}"/> </div>
+                      <div style="float:left">${member['firstname']} ${member['lastname']} (${_('owner')})</div>
+                      <input type="hidden" value="${member['user_id']}" name="review_members" />
+                      <span class="delete_icon action_button" onclick="removeReviewMember(${member['user_id']})"></span>
+                    </div>
+                  </li>
+                %endfor
+                </ul>
+              </div>
+
+              <div class='ac'>
+                <div class="reviewer_ac">
+                   ${h.text('user', class_='yui-ac-input')}
+                   <span class="help-block">${_('Add reviewer to this pull request.')}</span>
+                   <div id="reviewers_container"></div>
+                </div>
+              </div>
+            </div>
+        </div>
+
+        <div style="clear:both;padding: 0 0 30px 0;"></div>
+
+        <h4>${_('Changesets')}</h4>
+        <div style="float:left;padding:0px 30px 30px 30px">
+           ## overview pulled by ajax
+           <div style="float:left" id="pull_request_overview"></div>
+           <div style="float:left;clear:both;padding:10px 10px 10px 0px;display:none">
+                <a id="pull_request_overview_url" href="#">${_('Detailed compare view')}</a>
            </div>
         </div>
+        <div style="clear:both;"></div>
+
     </div>
+
     ${h.end_form()}
 
 </div>