Uploaded image for project: 'Crucible'
  1. Crucible
  2. CRUC-4184

Make Crucible reviews use progressive rendering of diffs

    • 3
    • Our product teams collect and evaluate feedback from a number of different sources. To learn more about how we use customer feedback in the planning process, check out our new feature policy.

      Currently, Crucible uses table based rendering, which could potentially blow out the DOM with large reviews (i.e., greater than 1000~ lines of diffs).

      The solution is to progressively render each line of diff, so that there isn't so many dom elements. This has been implemented in Fisheye's side-by-side view, and should be used in crucible.

      The problem may be in how we might have to render comments associated with the diff. Not sure how that can be solved yet.

            [CRUC-4184] Make Crucible reviews use progressive rendering of diffs

            Katherine Yabut made changes -
            Workflow Original: JAC Suggestion Workflow [ 3020619 ] New: JAC Suggestion Workflow 3 [ 3633046 ]
            Marek Parfianowicz made changes -
            Status Original: Reviewing [ 11773 ] New: Not Being Considered [ 11776 ]
            Marek Parfianowicz made changes -
            UIS New: 3
            Owen made changes -
            Workflow Original: Confluence Workflow - Public Facing v4 [ 2948020 ] New: JAC Suggestion Workflow [ 3020619 ]
            Owen made changes -
            Status Original: Gathering Interest [ 11772 ] New: Reviewing [ 11773 ]
            Owen made changes -
            Workflow Original: FECRU Development Workflow - Triage [ 940379 ] New: Confluence Workflow - Public Facing v4 [ 2948020 ]
            Status Original: Open [ 1 ] New: Gathering Interest [ 11772 ]
            Owen made changes -
            Backlog Order (Obsolete) Original: 3240000000
            Issue Type Original: Improvement [ 4 ] New: Suggestion [ 10000 ]
            Piotr Swiecicki made changes -
            Workflow Original: FECRU Development Workflow (Triage) [ 315662 ] New: FECRU Development Workflow - Triage [ 940379 ]
            Piotr Swiecicki made changes -
            Labels New: diff review-ux
            Patrick Hill made changes -
            Remote Link New: This issue links to "Wiki Page (Extranet)" [ 23520 ]

              Unassigned Unassigned
              jxie Chii (Inactive)
              Votes:
              20 Vote for this issue
              Watchers:
              21 Start watching this issue

                Created:
                Updated: