View Issue Details

IDProjectCategoryView StatusLast Update
0007121mantisbtbugtrackerpublic2010-04-23 23:22
ReporterchillaxAssigned Tovboctor  
PrioritynormalSeverityfeatureReproducibilityN/A
Status closedResolutionno change required 
Product Version1.0.3 
Summary0007121: CSS-combined Simple/Advanced Issue Reporting Page
Description

This is a request for feedback of an enhancement which combines the simple and advanced reporting page into one for inclusion into 1.1.0. Toggling between the simple and advanced options is accomplished using JS/CSS. A new file (bug_report_page_css.php) is included. This file replaces HTML tables with CSS and is based off of the CVS bug_report_advanced_page.php. Due to this, it requires the CVS version of the ./core/helper_api.php and ./config_defaults_inc.php (included in full in the zip) for the dynamic tabindex function and additional global variables respectively. Since it requires JS in order to work properly, it modifies ./core/string_api.php (unified diff patch file included) to check that JS is enabled for mantis before setting the URL for the bug report page. If you don't have JS enabled, you should still go to the normal, separate pages. There is also a unified diff patch file included for ./css/default.css for additional CSS styling needed.

Again, this is a request for feedback. I don't suggest using this in production yet. Looking to get any problems or suggestions for changes.

Additional Information

The bug_report_page_css.php drops the inclusion of the default_bug_reproducibility since it is not yet supported in the CVS config_defaults_inc.php file.

One thing still missing is dynamic changing of the alternating row color based on which options are visible.

Installation instructions:

  1. Backup any files that will be overwritten first (duh)
  2. Download and extract the zip file
  3. Copy bug_report_page_css.php to ./ of your mantis installation
  4. Make sure you did step one, backup your files first
  5. Merge the diff changes for ./css/default.css (default.css.patch) and ./core/string_api.php (string_api.php.patch)
  6. CVS users can skip this step. If using 1.0.3, you need to replace ./core/helper_api.php and ./config_defaults_inc.php files with the ones included
  7. Log in and test reporting an issue, try toggling between advanced and simple
TagsNo tags attached.
Attached Files
files_complete.zip (38,300 bytes)

Relationships

has duplicate 0007338 closedvboctor Keep inputs from simple report to advanced report 
has duplicate 0003643 closedvboctor Fields are cleared when you change view (simple/detailled) 
has duplicate 0008472 closedvboctor Switching between Simple/Advanced report does not preserve user input 
has duplicate 0008678 closedryandesign Enable "remember" filled fields while switching to advenced mode in reporting 
has duplicate 0007440 closedvboctor Switching between advanced and simple bug report and back again loses information 
has duplicate 0002121 closedvboctor switching between simple and advanced bug entry sheet should recover entered data 
has duplicate 0008985 closedryandesign Repost form data if switching from simple to extended reporting 
related to 0007559 closedvboctor Choosing active project clears news input 

Activities

vboctor

vboctor

2006-05-22 05:48

manager   ~0012885

Two comments about the attachment itself:

  1. It would make it easier if the files are stored in the zip file with the mantis directory structure. For example, core/strings_api.php, core/helper_api.php, etc.

  2. In addition to strings_api.php.patch and default.css.patch, I would attach the full files.

Please re-attach the zip once you apply these changes. I'll test once 2 is done.

chillax

chillax

2006-05-22 12:11

reporter   ~0012887

Last edited: 2006-06-23 21:43

I have attached the complete files in addition to the diff patch files and added directory structure to the zip.

vboctor

vboctor

2007-09-11 22:20

manager   ~0015616

Chillax, do you still have the patch for this. We seem to have lost it during the server migration.

chillax

chillax

2007-09-13 16:08

reporter   ~0015634

I can try to regenerate it for you. I'm running against an older version of mantis presently but it could be updated to work fairly easily I assume.

foo

foo

2007-10-19 15:30

reporter   ~0015912

I'd like to test this as well, if we can get new files attached. I suggested enhancing the separate forms in bug 0008472 - which woulsn't have to be J/S dependent - but Victor may be right that this is the way to go.

My first two thoughts, not having seen the patch yet:
1) J/S dependence may be bad for some folks.
2) If the above solution hides advanced fields from the user's view (not sure), but those fields can be populated manually when switching views, we'd want to make sure the data are handled properly. At the very least generate a warning message on submit, that says you should switch back to advanced view if you want to preserve any data in those fields.

grangeway

grangeway

2008-07-27 17:00

reporter   ~0018691

Chillax, Could you regenerate this ?

vboctor

vboctor

2009-07-28 09:03

manager   ~0022578

As of 1.2.0rc2 this will be no longer required, since we don't have the simple / advanced concept anymore.