Issue 3200:"Depends on" field now uses strikethrough for submitted dependencies.

Review Request #5348 — Created Jan. 29, 2014 and submitted

Information

Review Board
master

Reviewers

The "Depends on" field is used to indicate when one change depends on another. It would be very useful if it could react when those dependencies are marked as submitted by setting a strikethrough on the text. That way, users don't have to click through to see whether or not the dependent change has been committed yet.

Manual testing: marked a dependency submitted, observed it show up with a linethrough in the "Depends On" list for another review.
All tests in djblets and reviewboard pass.

Description From Last Updated

Indentation/alignment's a bit off now. Since this is going to end up being over 80 if you fix the alignment, …

chipx86chipx86

After thinking about this and doing some reading, I think this should be <s>: <ins> and <del> are for showing …

chipx86chipx86
OL
chipx86
  1. 
      
  2. reviewboard/reviews/builtin_fields.py (Diff revision 1)
     
     
     
    Show all issues

    Indentation/alignment's a bit off now.

    Since this is going to end up being over 80 if you fix the alignment, I'd suggest:

    rendered_item = (
        '<a href ...'
        % (...))
    
  3. 
      
chipx86
  1. 
      
  2. reviewboard/reviews/builtin_fields.py (Diff revision 1)
     
     
    Show all issues

    After thinking about this and doing some reading, I think this should be <s>:

    <ins> and <del> are for showing editorial changes, generally to an already published page. The <s> element is for indicating information that is no longer relevant or accurate, but still provides some value, and is generally used to create a document with both old and new information at the same time.

    • http://html5doctor.com/ins-del-s/
  3. 
      
OL
david
  1. Ship It!

  2. 
      
OL
Review request changed
Status:
Completed
Change Summary:
Pushed to master (9e57cc2). Thanks!