Diff viewer demo

Review Request #2 — Created Sept. 30, 2023 and updated

Information

Review Board

Reviewers

The Review Board diff viewer is one of the best on the market. It has
many ways to help you look through a change and focus on what's
important.

Here's just a few of the features we provide:

  • Detailed file index

    The file index lists all the files modified in the diff, along with a
    graph showing the relative number of inserts, deletes, and replaces.

    The thickness of the ring shows how much of the file is modified (and
    will be a full red or green circle, if the file is deleted or added).

    The colored dots on the side link to the matching changed chunks for
    that file.

  • Multi-line commenting

    Click a line number, or click and drag down several lines to comment
    on those lines.

  • Syntax highlighting

    Every diff is syntax-highlighted, making it easier to see, for
    instance, what's a string and what's a reserved keyword.

    This supports highlighting file formats of all kinds.

  • Collapsable diffs

    Diffs are shown collapsed, only showing what changed and the
    immediate lines around it.

    A header is shown above each change, showing the nearest function or
    class. Clicking on the header will show all unchangedl ines up to
    that function or class.

    Alternatively, click the [+] 20 to show 20 more lines, or the [+]
    to show all the lines hidden behind that header.

  • Moved code detection

    Significant code changes that move around a file are specially shown
    with little move flags saying where they went to or where they came
    from. You can rely on these to know that you don't need to give that
    code another look.

    For example, see how the __init__ function was moved in /r/2/diff/

  • Indentation change indicators

    When code is indented or un-indented, we show it cleanly without
    showing a full line change. These are shown by >>>> and <<<<
    markers on lines, which you can see in /r/2/diff/ and in
    /r/2/diff/1-2/

    Unlike most diff viewers, you won't have to waste your time examining
    every line that was indented, looking for changes.

    This will even differentiate between spaces and tabs. If someone
    accidentally indents the wrong way, you'll know.

  • See changes made within a line

    When replacing just part of a line, we highlight the changes, making
    it much easier to see the addition of a word, renaming of a variable,
    or even the accidental removal of a semicolon.

  • Interdiffs

    When reviewing multiple iterations of a change, you can view an
    "interdiff," the diff between two diffs. It's easy to see what has
    actually changed. For example, see /r/2/diff/1-2/

  • Multi-commit diffs

    Whether one commit or a hundred, you can post it to Review Board.

    Reviewers can look at the entirety of your work as one change,
    dig into individual commits, or view any range of commits they want.

  • Download any file in the diff

    Simply click the little down arrow next to the "Revision" or "New
    Change" in the diff's header to download the full content of either
    the file on the left-hand side or right-hand side.

Click View Diff in the top-right to look at the diff for this review
request to play around with the diffs.


 
Description From Last Updated

general

guest1975guest1975

test

guest9839guest9839

testing diff

guest6690guest6690

god

guest1975guest1975

Nope

guest8531guest8531

fdfdsf

guest8531guest8531

121212

guest3933guest3933

nope

guest8531guest8531

Test this line

guest2461guest2461

Not bad

guest2436guest2436

We can then have a discussion on it. Maybe I think this code is better: if not (not value): ... …

chipx86chipx86

Comments can be made across multiple lines. You might even want to suggest code, like: if value is not None: …

chipx86chipx86

demo comment 4512

guest1049guest1049

Needs testing

guest9115guest9115

sdafsdf

guest7233guest7233

e

guest2672guest2672
Checks run (1 failed)
test check test check is here ... keep putting more test and nothing will happen no markfdown support here as such Diff Job
chipx86
  1. 
      
  2. reviewboard/reviews/fields.py (Diff revision 2)
     
     
     
     
     
     
     
     
    Show all issues

    Comments can be made across multiple lines.

    You might even want to suggest code, like:

    if value is not None:
        # The rest of the code would go here...
        value_html = ...
    

    Note also how this comment opens an issue. We can use this to track what work still needs to be done on the diff.

    1. We can then have a discussion on it. Maybe I think this code is better:

      if not (not value):
          ...
      

      (It's not.)

    2. test

    3. test

  3. 
      
guest3933
  1. 
      
  2. reviewboard/reviews/fields.py (Diff revision 2)
     
     
    Show all issues

    121212

  3. 
      
guest7823
  1. this

  2. 
      
guest2672
  1. 
      
  2. reviewboard/reviews/fields.py (Diff revision 2)
     
     
    Show all issues

    e

  3. 
      
guest9870
  1. 
      
  2. reviewboard/reviews/fields.py (Diff revision 1)
     
     
     
     
     
     

    test

  3. 
      
guest1975
  1. Ship It!
  2. Show all issues

    general

  3. reviewboard/reviews/fields.py (Diff revisions 1 - 2)
     
     
     
     
     
     
    Show all issues

    god

  4. 
      
guest1049
  1. 
      
  2. reviewboard/reviews/fields.py (Diff revision 2)
     
     
    Show all issues

    demo comment 4512

  3. 
      
guest9115
  1. Ship It!
  2. 
      
guest9115
  1. 
      
  2. reviewboard/reviews/fields.py (Diff revision 2)
     
     
    Show all issues

    Needs testing

  3. 
      
guest7233
  1. Ship It!
  2. reviewboard/reviews/fields.py (Diff revision 2)
     
     
    Show all issues

    sdafsdf

  3. 
      
guest9839
  1. 
      
  2. Show all issues

    test

  3. 
      
guest1189
  1. Ship It!
  2. 
      
guest3034
  1. Test132

  2. 
      
guest8531
  1. 
      
  2. reviewboard/reviews/fields.py (Diff revision 2)
     
     
    Show all issues

    Nope

  3. reviewboard/reviews/fields.py (Diff revision 2)
     
     
    Show all issues

    nope

  4. 
      
guest8531
  1. 
      
  2. reviewboard/reviews/fields.py (Diff revision 2)
     
     
    Show all issues

    fdfdsf

  3. 
      
guest5830
  1. Ship It!
  2. 
      
guest2436
  1. 
      
  2. reviewboard/reviews/fields.py (Diff revision 2)
     
     
    Show all issues

    Not bad

  3. 
      
guest2436
  1. Ship It!
  2. 
      
guest2461
  1. 
      
  2. reviewboard/reviews/fields.py (Diff revision 2)
     
     
    Show all issues

    Test this line

  3. 
      
guest6690
  1. 
      
  2. reviewboard/reviews/fields.py (Diff revision 1)
     
     
     
    Show all issues

    testing diff

  3. 
      
guest6690
  1. 
      
  2. reviewboard/reviews/fields.py (Diff revision 1)
     
     

    old?

  3. 
      
Loading...