View Issue Details

IDProjectCategoryView StatusLast Update
0022423mantisbthtmlpublic2017-04-01 00:13
Reporterlibregeek Assigned Tocommunity  
PrioritynormalSeverityfeatureReproducibilityhave not tried
Status closedResolutionfixed 
Target Version2.3.0Fixed in Version2.3.0 
Summary0022423: ID attribute for bugnote_text
Description

We have developed a plugin to integrate SimpleMDE (www.simplemde.com) for the textareas. All other controls except the note field in issue details page (view.php) has an ID attribute. Hence, it would be great help if an ID is assigned to the Note field in issue details page.

TagsNo tags attached.
Attached Files
Selection_527.png (14,638 bytes)   
Selection_527.png (14,638 bytes)   

Relationships

related to 0022548 closedcommunity Remove unnecessary 'center' class from textarea in bugnote edit page 

Activities

ajtruckle

ajtruckle

2017-03-02 11:04

reporter   ~0055873

I am interested in the answer to this. I have viewed the plugin code on GitHub and can see the commented out code:

// Note field in View Issue page
/ This conflicts with the bugnote_text with id. so disabling it for the time being
var bugNoteViewMde = new SimpleMDE({
element: document.getElementsByName("bugnote_text")[0],
autoDownloadFontAwesome: false,
toolbar: ["bold", "italic", "heading", "|", "unordered-list", "ordered-list", "|", "quote", "code", "table", "|", "preview", "guide"],
});
/

libregeek

libregeek

2017-03-14 06:31

reporter   ~0056071

I have submitted a pull request for this issue: https://github.com/mantisbt/mantisbt/pull/1054
Please accept.

libregeek

libregeek

2017-03-15 00:51

reporter   ~0056074

Just now I noticed that the ID attribute is missing in Edit Note page (bugnote_edit_page.php). I will submit another PR later today itself. Please include this in v2.3.0, otherwise we may not be able to completely utilize the editor plugin.

libregeek

libregeek

2017-03-15 06:45

reporter   ~0056076

Here is the PR for 0022423:0056074: https://github.com/mantisbt/mantisbt/pull/1055

libregeek

libregeek

2017-03-17 09:37

reporter   ~0056104

Reopening for PR: https://github.com/mantisbt/mantisbt/pull/1055

dregad

dregad

2017-03-17 12:04

developer   ~0056107

Sorry, it is not clear to me why you are reopening this.

I had a quick glance at the new PR, and see several commits, including some from the previous PR; if something was missing or improperly fixed in MantisBT master 45008cc8, then to avoid confusion please rebase your branch on top of master, instead of merging it.

libregeek

libregeek

2017-03-20 02:23

reporter   ~0056121

@dregard
I'm sorry for the confusion. I tried rebase and messed up my fork. So I forked the upstream repo again and created a new PR.

https://github.com/mantisbt/mantisbt/pull/1058

Please accept. I will make note of the rebase option for future PR.

Related Changesets

MantisBT: master 45008cc8

2017-03-14 09:05

libregeek

Committer: dregad


Details Diff
Add ID attribute to 'add note' textarea in bug_view page

Fixes 0022423

Signed-off-by: Damien Regad <dregad@mantisbt.org>
Affected Issues
0022423
mod - bugnote_add_inc.php Diff File

MantisBT: master 835e2460

2017-03-20 04:46

libregeek

Committer: dregad


Details Diff
Added ID attribute to textarea in bugnote edit page

Fixes 0022423

Signed-off-by: Damien Regad <dregad@mantisbt.org>
Affected Issues
0022423
mod - bugnote_edit_page.php Diff File