View Issue Details

IDProjectCategoryView StatusLast Update
0022204mantisbtmarkdownpublic2017-01-31 04:02
Reporterbadfiles Assigned Tovboctor  
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionfixed 
Product Version2.0.0 
Target Version2.1.0Fixed in Version2.1.0 
Summary0022204: News headlines are parsed with markdown, though they should not be
Description

Markdown ON:

<h4 class="widget-title lighter">
                &lt;i class=&quot;ace-icon fa fa-edit&quot;></i>
                <p>Test headline</p>            &lt;/h4>

Markdown OFF:

&lt;h4 class=&quot;widget-title lighter&quot;>
                &lt;i class=&quot;ace-icon fa fa-edit&quot;></i>
                Test headline           &lt;/h4>
TagsNo tags attached.
Attached Files

Relationships

child of 0022180 new Markdown issues following implementation in 0017920 

Activities

dregad

dregad

2017-01-20 09:54

developer   ~0055241

@joel see attached screenshot (from main_page.php, with $g_news_enabled = ON) for illustration of what this issue is about.

main_no_markdown.png (2,369 bytes)   
main_no_markdown.png (2,369 bytes)   
main_with_markdown.png (2,581 bytes)   
main_with_markdown.png (2,581 bytes)   
joel

joel

2017-01-20 22:17

developer   ~0055246

Last edited: 2017-01-21 01:42

Thanks @dregad, I think its because headlines is being process with the text Mantis Core Formatting plugin, since Markdown is part of it.

 $t_headline = string_display_links( $p_headline );

solution: (remove the string_display_links helper)

$t_headline =  $p_headline;
joel

joel

2017-01-21 01:38

developer   ~0055247

Last edited: 2017-01-21 01:41

We can do it by tweaking the css as well without altering the code.

#news-items p { margin: 0 0 10px; all: inherit; }
joel

joel

2017-01-21 02:29

developer   ~0055248

Last edited: 2017-01-21 02:30

Ok, here is the PR for this issue https://github.com/mantisbt/mantisbt/pull/1003

Related Changesets

MantisBT: master f1aaddfb

2017-01-22 13:01

vboctor


Details Diff
Markdown: Fix broken layout in news page

Fixes 0022204
Affected Issues
0022204
mod - core/print_api.php Diff File