Commit f8e8541d authored by lpsolit%gmail.com's avatar lpsolit%gmail.com

Bug 357490: Reorganize fields in show_bug.cgi and group links together - Patch…

Bug 357490: Reorganize fields in show_bug.cgi and group links together - Patch by Frédéric Buclin <LpSolit@gmail.com> r=bkor a=justdave
parent c9f72537
...@@ -147,7 +147,7 @@ ...@@ -147,7 +147,7 @@
[% IF Param('useclassification') && bug.classification_id != 1 %] [% IF Param('useclassification') && bug.classification_id != 1 %]
<b>[[% bug.classification FILTER html %]]</b> <b>[[% bug.classification FILTER html %]]</b>
[% END %] [% END %]
<b>[% terms.Bug %]#:</b> <b>[% terms.Bug %]#</b>:
</td> </td>
<td> <td>
<a href="[% Param('urlbase') %]show_bug.cgi?id=[% bug.bug_id %]"> <a href="[% Param('urlbase') %]show_bug.cgi?id=[% bug.bug_id %]">
...@@ -182,18 +182,14 @@ ...@@ -182,18 +182,14 @@
<tr> <tr>
<td align="right"> <td align="right">
<b> <b><a href="page.cgi?id=fields.html#status">Status</a></b>:
<a href="page.cgi?id=fields.html#status">Status</a>:
</b>
</td> </td>
<td>[% status_descs.${bug.bug_status} FILTER html %]</td> <td>[% status_descs.${bug.bug_status} FILTER html %]</td>
</tr> </tr>
<tr> <tr>
<td align="right"> <td align="right">
<b> <b><a href="page.cgi?id=fields.html#resolution">Resolution</a></b>:
<a href="page.cgi?id=fields.html#resolution">Resolution</a>:
</b>
</td> </td>
<td> <td>
[% get_resolution(bug.resolution) FILTER html %] [% get_resolution(bug.resolution) FILTER html %]
...@@ -203,20 +199,9 @@ ...@@ -203,20 +199,9 @@
</td> </td>
</tr> </tr>
<tr>
<td align="right">
<b>
<a href="page.cgi?id=fields.html#assigned_to">Assigned&nbsp;To</a>:
</b>
</td>
<td>[% bug.assigned_to.identity FILTER html %]</td>
</tr>
</table> </table>
</td> </td>
<td valign="top"> <td valign="top">
<table cellspacing="1" cellpadding="1"> <table cellspacing="1" cellpadding="1">
<tr> <tr>
...@@ -272,12 +257,11 @@ ...@@ -272,12 +257,11 @@
</td> </td>
<td valign="top"> <td valign="top">
<table cellpadding="1" cellspacing="1"> <table cellpadding="1" cellspacing="1">
<tr> <tr>
<td align="right"> <td align="right">
<b>Reporter:</b> <b>Reporter</b>:
</td> </td>
<td> <td>
<a href="mailto:[% bug.reporter.email FILTER html %]"> <a href="mailto:[% bug.reporter.email FILTER html %]">
...@@ -286,6 +270,47 @@ ...@@ -286,6 +270,47 @@
</tr> </tr>
<tr> <tr>
<td align="right">
<b><a href="page.cgi?id=fields.html#assigned_to">Assigned&nbsp;To</a></b>:
</td>
<td>
<a href="mailto:[% bug.assigned_to.email FILTER html %]">
[% bug.assigned_to.identity FILTER html %]</a>
</td>
</tr>
[% IF Param('useqacontact') %]
<tr>
<td align="right">
<label for="qa_contact" accesskey="q"><b><u>Q</u>A Contact</b></label>:
</td>
<td colspan="7">
[% IF bug.check_can_change_field("qa_contact", 0, 1) || !user.id %]
[% INCLUDE global/userselect.html.tmpl
id => "qa_contact"
name => "qa_contact"
value => bug.qa_contact.login
size => 30
emptyok => 1
%]
[% ELSE %]
<input type="hidden" name="qa_contact"
value="[% bug.qa_contact.login FILTER html %]">
<a href="mailto:[% bug.qa_contact.email FILTER html %]">
[% IF bug.qa_contact.login && bug.qa_contact.login.length > 30 %]
<span title="[% bug.qa_contact.login FILTER html %]">
[% bug.qa_contact.identity FILTER truncate(30) FILTER html %]
</span>
[% ELSE %]
[% bug.qa_contact.identity FILTER html %]
[% END %]
</a>
[% END %]
</td>
</tr>
[% END %]
<tr>
<td align="right" valign="top"> <td align="right" valign="top">
<label for="newcc" accesskey="a"><b><u>A</u>dd&nbsp;CC</b></label>: <label for="newcc" accesskey="a"><b><u>A</u>dd&nbsp;CC</b></label>:
</td> </td>
...@@ -322,7 +347,6 @@ ...@@ -322,7 +347,6 @@
</tr> </tr>
</table> </table>
</td> </td>
</tr> </tr>
...@@ -330,36 +354,15 @@ ...@@ -330,36 +354,15 @@
<td colspan="2"> <td colspan="2">
<table cellspacing="1" cellpadding="1"> <table cellspacing="1" cellpadding="1">
[%# *** QAContact URL Requests Summary Whiteboard Keywords *** %] [%# *** URL Requests Summary Whiteboard Keywords *** %]
[% IF Param('useqacontact') %]
<tr> <tr>
<td align="right"> <td align="right">
<label for="qa_contact" accesskey="q"><b><u>Q</u>A Contact</b></label>: <label for="short_desc" accesskey="s"><b><u>S</u>ummary</b></label>:
</td>
<td colspan="7">
[% IF bug.check_can_change_field("qa_contact", 0, 1) || !user.id %]
[% INCLUDE global/userselect.html.tmpl
id => "qa_contact"
name => "qa_contact"
value => bug.qa_contact.login
size => 60
emptyok => 1
%]
[% ELSE %]
<input type="hidden" name="qa_contact"
value="[% bug.qa_contact.login FILTER html %]">
[% IF bug.qa_contact.login && bug.qa_contact.login.length > 60 %]
<span title="[% bug.qa_contact.login FILTER html %]">
[% bug.qa_contact.login FILTER truncate(60) FILTER html %]
</span>
[% ELSE %]
[% bug.qa_contact.login FILTER html %]
[% END %]
[% END %]
</td> </td>
[% PROCESS input inputname => "short_desc" size => "60" colspan => 2
maxlength => 255 %]
</tr> </tr>
[% END %]
<tr> <tr>
<td align="right"> <td align="right">
...@@ -372,25 +375,16 @@ ...@@ -372,25 +375,16 @@
[% END %] [% END %]
[%%]</b></label>: [%%]</b></label>:
</td> </td>
[% PROCESS input inputname => "bug_file_loc" size => "60" colspan => 5 %] [% PROCESS input inputname => "bug_file_loc" size => "60" colspan => 2 %]
</tr>
<tr>
<td align="right">
<label for="short_desc" accesskey="s"><b><u>S</u>ummary</b></label>:
</td>
[% PROCESS input inputname => "short_desc" size => "60" colspan => 5
maxlength => 255 %]
</tr> </tr>
[% IF Param('usestatuswhiteboard') %] [% IF Param('usestatuswhiteboard') %]
<tr> <tr>
<td align="right"> <td align="right">
<label for="status_whiteboard" accesskey="w"><b>Status <label for="status_whiteboard" accesskey="w"><b><u>W</u>hiteboard</b></label>:
<u>W</u>hiteboard</b></label>:
</td> </td>
[% PROCESS input inputname => "status_whiteboard" size => "60" [% PROCESS input inputname => "status_whiteboard" size => "60"
colspan => 5 %] colspan => 2 %]
</tr> </tr>
[% END %] [% END %]
...@@ -398,7 +392,7 @@ ...@@ -398,7 +392,7 @@
<tr> <tr>
<td align="right"> <td align="right">
<label for="keywords" accesskey="k"><b><a href="describekeywords.cgi"><u>K</u>eywords</a></b></label>: <label for="keywords" accesskey="k"><b><a href="describekeywords.cgi"><u>K</u>eywords</a></b></label>:
[% PROCESS input inputname => "keywords" size => 60 colspan => 5 [% PROCESS input inputname => "keywords" size => 60 colspan => 2
value => bug.keywords.join(', ') %] value => bug.keywords.join(', ') %]
</tr> </tr>
[% END %] [% END %]
...@@ -411,11 +405,35 @@ ...@@ -411,11 +405,35 @@
[% FOREACH field = fields %] [% FOREACH field = fields %]
<tr> <tr>
[% PROCESS bug/field.html.tmpl value=bug.${field.name} [% PROCESS bug/field.html.tmpl value=bug.${field.name}
editable=1 %] editable = 1
value_span = 2 %]
</tr> </tr>
[% END %] [% END %]
[% END %] [% END %]
<tr>
[% PROCESS dependencies
dep = { title => "Depends on", fieldname => "dependson" } %]
</tr>
<tr>
[% PROCESS dependencies accesskey = "b"
dep = { title => "<u>B</u>locks", fieldname => "blocked" } %]
</tr>
<tr>
<th>&nbsp;</th>
<td colspan="2">
<a href="showdependencytree.cgi?id=[% bug.bug_id %]&amp;hide_resolved=1">Show
dependency tree</a>
[% IF Param('webdotbase') %]
- <a href="showdependencygraph.cgi?id=[% bug.bug_id %]">Show
dependency graph</a>
[% END %]
</td>
</tr>
</table> </table>
</td> </td>
...@@ -516,50 +534,13 @@ ...@@ -516,50 +534,13 @@
show_attachment_flags = bug.show_attachment_flags show_attachment_flags = bug.show_attachment_flags
%] %]
[%# *** Dependencies Votes *** %]
<table> [%# *** Comments Groups *** %]
<tr>
[% PROCESS dependencies
dep = { title => "depends on", fieldname => "dependson" } %]
<td rowspan="2">
<a href="showdependencytree.cgi?id=[% bug.bug_id %]&amp;hide_resolved=1">Show
dependency tree</a>
[% IF Param('webdotbase') %]
<br> <br>
<a href="showdependencygraph.cgi?id=[% bug.bug_id %]">Show <table cellpadding="1" cellspacing="1">
dependency graph</a>
[% END %]
</td>
</tr>
<tr>
[% PROCESS dependencies accesskey = "b"
dep = { title => "<u>b</u>locks", fieldname => "blocked" } %]
</tr>
</table>
[% IF bug.use_votes %]
<table>
<tr> <tr>
<th>
<a href="page.cgi?id=voting.html">Votes</a>:
</th>
<td> <td>
[% bug.votes %]&nbsp;&nbsp;&nbsp;
<a href="votes.cgi?action=show_bug&amp;bug_id=[% bug.bug_id %]">Show
votes for this [% terms.bug %]</a>&nbsp;&nbsp;&nbsp;
<a href="votes.cgi?action=show_user&amp;bug_id=[% bug.bug_id %]#vote_[% bug.bug_id %]">Vote
for this [% terms.bug %]</a>
</td>
</tr>
</table>
[% END %]
[%# *** Comments Groups *** %]
<br>
<label for="comment" accesskey="c"><b>Additional <u>C</u>omments</b></label>: <label for="comment" accesskey="c"><b>Additional <u>C</u>omments</b></label>:
[% IF Param("insidergroup") && user.in_group(Param("insidergroup")) %] [% IF Param("insidergroup") && user.in_group(Param("insidergroup")) %]
<input type="checkbox" name="commentprivacy" value="1" <input type="checkbox" name="commentprivacy" value="1"
...@@ -592,6 +573,35 @@ ...@@ -592,6 +573,35 @@
&& user.settings.state_addselfcc.value == 'cc_unless_role') %]> && user.settings.state_addselfcc.value == 'cc_unless_role') %]>
<label for="addselfcc">Add [% user.identity FILTER html %] to CC list</label> <label for="addselfcc">Add [% user.identity FILTER html %] to CC list</label>
[% END %] [% END %]
</td>
<td valign="top">
<fieldset>
<legend>Related actions</legend>
<ul>
<li><a href="show_activity.cgi?id=[% bug.bug_id %]">View [% terms.Bug %] Activity</a></li>
<li><a href="show_bug.cgi?format=multiple&amp;id=[% bug.bug_id %]">Format For Printing</a></li>
<li><a href="show_bug.cgi?ctype=xml&amp;id=[% bug.bug_id %]">XML</a></li>
<li><a href="enter_bug.cgi?cloned_bug_id=[% bug.bug_id %]">Clone This [% terms.Bug %]</a></li>
</ul>
[% IF bug.use_votes %]
<br>
<b><a href="page.cgi?id=voting.html">Votes</a></b>: [% bug.votes %]
<ul>
<li><a href="votes.cgi?action=show_bug&amp;bug_id=[% bug.bug_id %]">Show
votes for this [% terms.bug %]</a></li>
<li><a href="votes.cgi?action=show_user&amp;bug_id=[% bug.bug_id %]#vote_[% bug.bug_id %]">Vote
for this [% terms.bug %]</a></li>
</ul>
[% END %]
[%# Links to more things users can do with this bug. %]
[% Hook.process("links") %]
</fieldset>
</td>
</tr>
</table>
<br> <br>
[% IF bug.groups.size > 0 %] [% IF bug.groups.size > 0 %]
...@@ -681,7 +691,7 @@ ...@@ -681,7 +691,7 @@
[% BLOCK dependencies %] [% BLOCK dependencies %]
<th align="right"> <th align="right">
<label for="[% dep.fieldname %]"[% " accesskey=\"$accesskey\"" IF accesskey %]> <label for="[% dep.fieldname %]"[% " accesskey=\"$accesskey\"" IF accesskey %]>
[% terms.Bug %] [%+ bug.bug_id %] [%+ dep.title %]</label>: [% dep.title %]</label>:
</th> </th>
<td> <td>
[% FOREACH depbug = bug.${dep.fieldname} %] [% FOREACH depbug = bug.${dep.fieldname} %]
...@@ -726,7 +736,7 @@ ...@@ -726,7 +736,7 @@
[%############################################################################%] [%############################################################################%]
[% BLOCK input %] [% BLOCK input %]
<td[% " colspan=\"$colspan\"" IF $colspan %]> <td[% " colspan=\"$colspan\"" IF colspan %]>
[% val = value ? value : bug.$inputname %] [% val = value ? value : bug.$inputname %]
[% IF bug.check_can_change_field(inputname, 0, 1) || !user.id %] [% IF bug.check_can_change_field(inputname, 0, 1) || !user.id %]
<input id="[% inputname %]" name="[% inputname %]" <input id="[% inputname %]" name="[% inputname %]"
......
...@@ -161,28 +161,11 @@ ...@@ -161,28 +161,11 @@
<div id="knob-buttons"> <div id="knob-buttons">
<input type="submit" value="Commit" id="commit"> <input type="submit" value="Commit" id="commit">
<p>
<font size="+1">
<b>
<a href="show_activity.cgi?id=[% bug.bug_id %]">View [% terms.Bug %] Activity</a>
&nbsp; | &nbsp;
<a href="show_bug.cgi?format=multiple&amp;id=[% bug.bug_id %]">Format For Printing</a>
&nbsp; | &nbsp;
<a href="show_bug.cgi?ctype=xml&amp;id=[% bug.bug_id %]">XML</a>
&nbsp; | &nbsp;
<a href="enter_bug.cgi?cloned_bug_id=[% bug.bug_id %]">Clone This [% terms.Bug %]</a>
[%# Links to more things users can do with this bug. %]
[% Hook.process("links") %]
</b>
</font>
[% IF bug.user.canmove %] [% IF bug.user.canmove %]
&nbsp; <font size="+1"><b> | </b></font> &nbsp; &nbsp; <font size="+1"><b> | </b></font> &nbsp;
<input type="submit" name="action" id="action" <input type="submit" name="action" id="action"
value="[% Param("move-button-text") %]"> value="[% Param("move-button-text") %]">
[% END %] [% END %]
</p>
</div> </div>
</div> </div>
......
...@@ -318,13 +318,12 @@ ...@@ -318,13 +318,12 @@
'selname', 'selname',
'" accesskey=\"$accesskey\"" IF accesskey', '" accesskey=\"$accesskey\"" IF accesskey',
'inputname', 'inputname',
'" colspan=\"$colspan\"" IF $colspan', '" colspan=\"$colspan\"" IF colspan',
'" size=\"$size\"" IF size', '" size=\"$size\"" IF size',
'" maxlength=\"$maxlength\"" IF maxlength' '" maxlength=\"$maxlength\"" IF maxlength'
], ],
'bug/knob.html.tmpl' => [ 'bug/knob.html.tmpl' => [
'bug.bug_id',
'knum', 'knum',
], ],
......
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