diff options
| -rw-r--r-- | skins/standard/IE-fixes.css | 4 | ||||
| -rw-r--r-- | skins/standard/global.css | 32 | ||||
| -rw-r--r-- | template/en/default/bug/create/create.html.tmpl | 81 | ||||
| -rw-r--r-- | template/en/default/bug/field.html.tmpl | 7 |
4 files changed, 80 insertions, 44 deletions
diff --git a/skins/standard/IE-fixes.css b/skins/standard/IE-fixes.css index f7e95f990..4c85340e7 100644 --- a/skins/standard/IE-fixes.css +++ b/skins/standard/IE-fixes.css @@ -22,6 +22,10 @@ margin-top: .5em; } +form#Create #comp_desc { + margin: .5em 1em; +} + #footer #useful-links li { padding-bottom: 0.8ex; } diff --git a/skins/standard/global.css b/skins/standard/global.css index 6e55385e7..2690b6e2d 100644 --- a/skins/standard/global.css +++ b/skins/standard/global.css @@ -497,18 +497,44 @@ input.required, select.required, span.required_explanation { list-style-type: none; } +/*************/ +/* enter_bug */ +/*************/ + +form#Create table { + border-spacing: 0; + border-width: 0; +} +form#Create td, form#Create th { + padding: .25em; +} form#Create th { text-align: right; } +/* This makes the "component" column as small as possible (since it + * contains only fixed-width content) and the Reporter column + * as large as possible, which makes the form not jump around + * when the Component Description changes size. This works + * pretty well on all browsers except IE 8. + */ +form#Create #field_container_component { width: 1px; } +form#Create #field_container_reporter { width: 100%; } + form#Create .comment { vertical-align: top; overflow: auto; color: green; - margin: 0 0.5em; - padding: 0.3em; - height: 8ex; } +form#Create #comp_desc_container td { padding: 0; } +form#Create #comp_desc { height: 11ex; } +form#Create #os_guess_note { + padding-top: 0; +} +form#Create #os_guess_note div { + max-width: 35em; +} + .image_button { background-repeat: no-repeat; diff --git a/template/en/default/bug/create/create.html.tmpl b/template/en/default/bug/create/create.html.tmpl index a5bfb0739..5db1eddf3 100644 --- a/template/en/default/bug/create/create.html.tmpl +++ b/template/en/default/bug/create/create.html.tmpl @@ -175,7 +175,7 @@ TUI_hide_default('expert_fields'); <input type="hidden" name="product" value="[% product.name FILTER html %]"> <input type="hidden" name="token" value="[% token FILTER html %]"> -<table cellspacing="4" cellpadding="2" border="0"> +<table> <tbody> <tr> <td colspan="4"> @@ -204,11 +204,12 @@ TUI_hide_default('expert_fields'); </tr> <tr> - <th>Product:</th> - <td width="10%">[% product.name FILTER html %]</td> - - <th>Reporter:</th> - <td width="100%">[% user.login FILTER html %]</td> + [% INCLUDE bug/field.html.tmpl + bug = default, field = bug_fields.product, editable = 0, + value = product.name %] + [% INCLUDE bug/field.html.tmpl + bug = default, field = bug_fields.reporter, editable = 0, + value = user.login %] </tr> [%# We can't use the select block in these two cases for various reasons. %] @@ -217,7 +218,7 @@ TUI_hide_default('expert_fields'); <a href="describecomponents.cgi?product=[% product.name FILTER url_quote %]"> Component</a>: </th> - <td> + <td id="field_container_component"> <select name="component" id="component" onchange="set_assign_to();" size="7" aria-required="true" class="required"> [%# Build the lists of assignees and QA contacts if "usemenuforusers" is enabled. %] @@ -254,7 +255,7 @@ TUI_hide_default('expert_fields'); </script> </td> - <td colspan="2"> + <td colspan="2" id="comp_desc_container"> [%# Enclose the fieldset in a nested table so that its width changes based # on the length on the component description. %] <table> @@ -271,7 +272,8 @@ TUI_hide_default('expert_fields'); </tr> <tr> - <th rowspan="3">Version:</th> + <th rowspan="3"><a href="page.cgi?id=fields.html#version"> + [%- field_descs.version FILTER html %]:</a></th> <td rowspan="3"> <select name="version" size="5"> [%- FOREACH v = version %] @@ -298,6 +300,22 @@ TUI_hide_default('expert_fields'); bug = default, field = bug_fields.op_sys, editable = 1, value = default.op_sys %] </tr> + [% IF !Param('defaultplatform') || !Param('defaultopsys') %] + <tr> + <td colspan="3"> </th> + <td id="os_guess_note" class="comment"> + <div>We've made a guess at your + [% IF Param('defaultplatform') %] + operating system. Please check it + [% ELSIF Param('defaultopsys') %] + platform. Please check it + [% ELSE %] + operating system and platform. Please check them + [% END %] + and make any corrections if necessary.</div> + </td> + </tr> + [% END %] </tbody> <tbody class="expert_fields"> @@ -319,25 +337,6 @@ TUI_hide_default('expert_fields'); </tr> </tbody> -[% IF !Param('defaultplatform') || !Param('defaultopsys') %] - <tbody> - <tr> - <th> </th> - <td colspan="3" class="comment"> - We've made a guess at your - [% IF Param('defaultplatform') %] - operating system. Please check it - [% ELSIF Param('defaultopsys') %] - platform. Please check it - [% ELSE %] - operating system and platform. Please check them - [% END %] - and make any corrections if necessary. - </td> - </tr> - </tbody> -[% END %] - <tbody class="expert_fields"> <tr> <td colspan="4"> </td> @@ -400,7 +399,8 @@ TUI_hide_default('expert_fields'); [% IF Param("useqacontact") %] <tr> - <th>QA Contact:</th> + <th><a href="page.cgi?id=fields.html#qa_contact"> + [%- field_descs.qa_contact FILTER html %]:</a></th> <td colspan="2"> [% INCLUDE global/userselect.html.tmpl id => "qa_contact" @@ -417,7 +417,8 @@ TUI_hide_default('expert_fields'); [% END %] <tr> - <th>CC:</th> + <th><a href="page.cgi?id=fields.html#cc"> + [%- field_descs.cc FILTER html %]:</a></th> <td colspan="2"> [% INCLUDE global/userselect.html.tmpl id => "cc" @@ -431,7 +432,7 @@ TUI_hide_default('expert_fields'); </tr> <tr> - <th>Default CC:</th> + <th>Default [% field_descs.cc FILTER html %]:</th> <td colspan="2"> <div id="initial_cc"> </div> @@ -444,7 +445,8 @@ TUI_hide_default('expert_fields'); [% IF user.is_timetracker %] <tr> - <th>Estimated Hours:</th> + <th><a href="page.cgi?id=fields.html#estimated_time">Estimated + Hours:</a></th> <td colspan="2"> <input name="estimated_time" size="6" maxlength="6" value="[% estimated_time FILTER html %]"> </td> @@ -462,7 +464,8 @@ TUI_hide_default('expert_fields'); [% IF Param("usebugaliases") %] <tr> - <th>Alias:</th> + <th><a href="page.cgi?id=fields.html#alias"> + [%- field_descs.alias FILTER html %]:</a></th> <td colspan="2"> <input name="alias" size="20" value="[% alias FILTER html %]"> </td> @@ -470,7 +473,8 @@ TUI_hide_default('expert_fields'); [% END %] <tr> - <th>URL:</th> + <th><a href="page.cgi?id=fields.html#bug_file_loc"> + [%- field_descs.bug_file_loc FILTER html %]:</a></th> <td colspan="2"> <input name="bug_file_loc" size="40" value="[% bug_file_loc FILTER html %]"> @@ -495,7 +499,8 @@ TUI_hide_default('expert_fields'); <tbody> <tr> - <th class="required">Summary:</th> + <th class="required"><a href="page.cgi?id=fields.html#short_desc"> + [%- field_descs.short_desc FILTER html %]:</a></th> <td colspan="3"> <input name="short_desc" size="70" value="[% short_desc FILTER html %]" maxlength="255" spellcheck="true" aria-required="true" @@ -592,13 +597,15 @@ TUI_hide_default('expert_fields'); [% END %] <tr> - <th>Depends on:</th> + <th><a href="page.cgi?id=fields.html#dependson"> + [%- field_descs.dependson FILTER html %]:</a></th> <td colspan="3"> <input name="dependson" accesskey="d" value="[% dependson FILTER html %]"> </td> </tr> <tr> - <th>Blocks:</th> + <th><a href="page.cgi?id=fields.html#blocked"> + [%- field_descs.blocked FILTER html %]:</a></th> <td colspan="3"> <input name="blocked" accesskey="b" value="[% blocked FILTER html %]"> </td> diff --git a/template/en/default/bug/field.html.tmpl b/template/en/default/bug/field.html.tmpl index 2f1ad35ca..0656996bd 100644 --- a/template/en/default/bug/field.html.tmpl +++ b/template/en/default/bug/field.html.tmpl @@ -54,11 +54,10 @@ [% END %] [% IF desc_url %] <a href="[% desc_url FILTER html %]"> - [% ELSIF !field.custom %] + [% ELSE %] <a href="page.cgi?id=fields.html#[% field.name FILTER url_quote %]"> - [% END -%] - [% field_descs.${field.name} FILTER html %]: - [%- '</a>' IF (!field.custom || desc_url) %] + [% END %] + [%- field_descs.${field.name} FILTER html %]:</a> [% '</label>' IF editable %] </th> [% END %] |