Modify

Opened 10 years ago

Closed 9 years ago

#12819 closed defect (fixed)

Conflicts with ReadmePlugin - preventing markdown files from being rendered in source browser

Reported by: Edmund NI Owned by: Ryan J Ollos
Priority: normal Component: ReadmePlugin
Severity: normal Keywords:
Cc: Trac Release: 1.0

Description

When TracWysiwygPlugin is enabled ReadmePlugin will not render *.md files in source browser (attachment:wysiwyg-enabled_markdown-not-rendered.png).

When TracWysiwygPlugin is disabled the same *.md file is correctly rendered (attachment:wysiwyg-disabled_markdown-rendered.png).

Attachments (2)

wysiwyg-enabled_markdown-not-rendered.png (80.9 KB) - added by Edmund NI 10 years ago.
TracWysiwygPlugin enabled and markdown file not rendered
wysiwyg-disabled_markdown-rendered.png (65.1 KB) - added by Edmund NI 10 years ago.
TracWysiwygPlugin disabled and markdown file rendered

Download all attachments as: .zip

Change History (7)

Changed 10 years ago by Edmund NI

TracWysiwygPlugin enabled and markdown file not rendered

Changed 10 years ago by Edmund NI

TracWysiwygPlugin disabled and markdown file rendered

comment:1 Changed 10 years ago by Jun Omae

Please post trac.log and javascript console after enabling TracLogging and reproducing it.

comment:2 Changed 10 years ago by Jun Omae

Component: TracWysiwygPluginReadmePlugin
Owner: changed from Jun Omae to Southen

I get the following error on javascript console.

ReferenceError: jQuery is not defined

ReadmePlugin conflicts with any plugins which use add_script_data(). That is a ReadmePlugin issue, not TracWysiwygPlugin.

    <script type="text/javascript">
      var _tracwysiwyg={};
    </script><script type="text/javascript">jQuery(document).ready(function($) {  $('#preview').each(function() {    $(this).html(marked( $(this).children('pre').first().text() ));  });});</script>

comment:3 Changed 9 years ago by Ryan J Ollos

Owner: changed from Southen to Ryan J Ollos
Status: newaccepted

comment:4 Changed 9 years ago by Ryan J Ollos

I think it should be fixed by b38a6e24. Make sure to install from the source available in the trac-hacks org on GitHub.

Last edited 9 years ago by Ryan J Ollos (previous) (diff)

comment:5 Changed 9 years ago by Ryan J Ollos

Resolution: fixed
Status: acceptedclosed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Ryan J Ollos.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.