Fix formatting of reply text with the latest Markdown changes.

Review Request #6566 — Created Nov. 7, 2014 and submitted

Information

Review Board
release-2.0.x
ecfa021...

Reviewers

The reply code wasn't updated to properly handle plain text or to
correctly render provided HTML. This led to showing tags instead of
rendering those tags, and resulted in the Markdown checkbox doing
nothing.

This updates the reply code to work with the new Markdown changes.

Tested loading and saving rich text and plain text. Didn't see any
formatting issues anymore.

Wrote new unit tests, which pass.

reviewbot
  1. Tool: Pyflakes
    Ignored Files:
        reviewboard/static/rb/js/models/tests/reviewReplyEditorModelTests.js
        reviewboard/static/rb/js/views/reviewReplyEditorView.js
        reviewboard/static/rb/js/views/tests/reviewReplyEditorViewTests.js
        reviewboard/static/rb/js/models/reviewReplyEditorModel.js
    
    
    
    Tool: PEP8 Style Checker
    Ignored Files:
        reviewboard/static/rb/js/models/tests/reviewReplyEditorModelTests.js
        reviewboard/static/rb/js/views/reviewReplyEditorView.js
        reviewboard/static/rb/js/views/tests/reviewReplyEditorViewTests.js
        reviewboard/static/rb/js/models/reviewReplyEditorModel.js
    
    
  2. 
      
david
  1. Ship It!

  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-2.0.x (e99488e)
Loading...