View Issue Details

IDProjectCategoryView StatusLast Update
0021807mantisbtuipublic2017-03-29 08:34
ReportermcmoAssigned To 
PrioritynormalSeverityminorReproducibilityalways
Status confirmedResolutionopen 
Product Version2.0.0-beta.1 
Target Version2.3.0Fixed in Version 
Summary0021807: The required fields are not explicitly visible when updating, resolving or closing an issue
Description

The red asterisks are not displayed when updating, resolving or closing an issue.
They are displayed when creating an issue.
This is annoying as if you have several fields that are required on closing, then you may have to try several times before you can effectively submit the form.

Steps To Reproduce

create a custom field
make it mandatory on reporting, updating. resolving and closing an issue
create an issue
update it: the red asterisk does not appear for the custom field
_ do not enter a value in the custom field and submit the change.
for each field that is required and not filled in, you would get an application error #11 message to enter that value

Additional Information

I have checked the code and the <SPAN> elements that previously contained that asterisk are missing.
For instance, in bug_update_page.php, in 1.3.0 RC1
line 685 '<span class="required">*</span>
that disappeared in 1.3.2 and 2.0.0b3

Tagsmodern-ui

Relationships

related to 0021694 confirmed inconsistent presentation of required fields 

Activities

mcmo

mcmo

2016-10-19 03:48

reporter  

mcmo

mcmo

2016-10-19 03:49

reporter  

mcmo

mcmo

2016-10-19 03:50

reporter   ~0054265

Can someone remove the 2nd attachment, please?
It's useless.
Sorry for that mistake.

atrol

atrol

2016-10-19 04:16

developer   ~0054266

Seems I removed the wrong one as I didn't notice that dregad did already remove another attachment.

dregad

dregad

2016-10-19 04:33

developer   ~0054267

In 1.3.x branch, everything works fine as far as I can tell. I see the red * before required custom fields in bug report and update pages, as well as on change status pages.

For the record, the <span class="required">*</span> is no longer needed, because we rely on CSS to display the asterisk [1]

That being said, I confirm that this is broken in 2.0 (the .required:before class is gone). It also appears that in at least some cases (e.g. bug_report_page.php), the code reverts to using spans to display the asterisk.

[1] https://github.com/mantisbt/mantisbt/blob/master-1.3.x/css/default.css#L536

mcmo

mcmo

2016-10-19 04:40

reporter  

mcmo

mcmo

2016-10-19 04:57

reporter   ~0054269

If the "<span class="required">*</span>" is no longer needed, then I would suggest that you apply the CSS class also to the "normal" fields like "Summary" when reporting an issue.
that would solve the issue 0021694

mcmo

mcmo

2016-10-19 05:10

reporter   ~0054270

I changed /css/default.css by adding the missing lines, and that solved the issue when editing the issue.
However that does not work when changing the status.

mcmo

mcmo

2016-10-19 05:49

reporter   ~0054271

Last edited: 2017-03-29 08:34

View 2 revisions

That works now when resolving or closing an issue after changing bug_change_status_page.php

--- a/<html>bug_change_status_page.php (<b>Yesterday 12:41:20</b>)</html>
+++ b/<html><b>Current File</b></html>
@@ -291,6 +291,7 @@
 ?>
    <tr>
        <th class="category">
+                    <label <?php echo $t_class_required ?>></label>
            <?php echo lang_get_defaulted( $t_def['name'] ) ?>
        </th>
        <td>

thanks for your help

EDIT (dregad): fixed markdown

fman

fman

2017-03-29 08:31

reporter   ~0056262

My 5 cents using info I've found in this ticket

just added following lines to default.css on version 2.1.0 and Custom Fields have * when editing issue

/ from 1.3.7 /
div.form-container fieldset label.required:before,
th label.required:before,
td label.required:before {
font-size: 8pt;
content: '* ';
color: red;
}

Issue History

Date Modified Username Field Change
2016-10-19 03:48 mcmo New Issue
2016-10-19 03:48 mcmo File Added: 2016-10-19 09_40_07-MantisBT.png
2016-10-19 03:48 mcmo File Added: 2016-10-19 09_40_07-MantisBT-2.png
2016-10-19 03:48 mcmo File Added: 2016-10-19 09_46_59-bug_update_page.php - KDiff3.png
2016-10-19 03:49 mcmo File Added: 2016-10-19 09_39_56-0000002_ TEST 1 - MantisBT.png
2016-10-19 03:50 mcmo Note Added: 0054265
2016-10-19 03:53 dregad File Deleted: 2016-10-19 09_40_07-MantisBT-2.png
2016-10-19 04:13 atrol File Deleted: 2016-10-19 09_46_59-bug_update_page.php - KDiff3.png
2016-10-19 04:16 atrol Note Added: 0054266
2016-10-19 04:33 dregad Status new => confirmed
2016-10-19 04:33 dregad Product Version 2.0.0-beta.3 => 2.0.0-beta.1
2016-10-19 04:33 dregad Target Version => 2.0.0-rc.1
2016-10-19 04:33 dregad Note Added: 0054267
2016-10-19 04:40 mcmo File Added: 2016-10-19 09_46_59-bug_update_page.php - KDiff3.png
2016-10-19 04:57 mcmo Note Added: 0054269
2016-10-19 05:03 atrol Relationship added related to 0021694
2016-10-19 05:03 atrol Tag Attached: modern-ui
2016-10-19 05:04 atrol Severity major => minor
2016-10-19 05:10 mcmo Note Added: 0054270
2016-10-19 05:49 mcmo Note Added: 0054271
2016-10-30 23:23 vboctor Target Version 2.0.0-rc.1 => 2.0.0-rc.2
2016-11-27 08:20 dregad Target Version 2.0.0-rc.2 => 2.0.0
2016-12-30 15:56 vboctor Target Version 2.0.0 => 2.0.1
2017-02-01 22:49 vboctor Target Version 2.0.1 => 2.2.0
2017-02-26 21:19 vboctor Target Version 2.2.0 => 2.3.0
2017-03-29 08:31 fman Note Added: 0056262
2017-03-29 08:34 dregad Note Edited: 0054271 View Revisions