• 
      

    Show a spinner before rendering markdown text or diff index.

    Review Request #4835 — Created Oct. 23, 2013 and submitted — Latest diff uploaded

    Information

    Review Board
    master

    Reviewers

    Show a spinner before rendering markdown text or diff index.

    When displaying a review request, there's a brief moment of time before
    Markdown gets rendered or before the diff table is loaded where things
    look wrong. For the Markdown fields (namely the Description and Testing
    Done fields), the raw Markdown is shown first, and then switches to the
    rendered text, which looks bad. The diff index shows a big line before
    displaying its contents.

    These now are marked as loading at first, and we'll show spinners. Once
    the page has successfully loaded, the spinners go away and the rendered
    content appears.

    We also try to render the text for these fields a bit sooner than we did
    before, hopefully reducing the load time.

    Tested the description, testing done, and change description fields,
    and the diff index, as the owner of the review request and as anonymous.

    The page started out with those fields showing spinners, and the approximate
    text size (since the raw text was loaded), and then soon after, it was replaced
    with the rendered text.

    Same with the diff field. It just showed a spinner roughly where the diff file
    entries would go, and then the whole thing loaded. It was a bit weird with the
    "Diff Revision ..." appearing, but not as bad as it was before.