#5389 Code browser diff improvement - offer option for side by side

v1.0.0
closed
nobody
42cc (432)
General
Cory Johns
2015-08-20
2012-11-30
No

See [forge:feature-requests:#6]

Related

Tickets: #5389

Discussion

  • Dave Brondsema

    Dave Brondsema - 2012-11-30

    ViewVC is the old system that can be used for comparison. It is BSD-style licensed, so if it makes sense to re-use their code I think that should be ok. We should try to track it clearly in a separate file if so though.

     
  • Igor Bondarenko - 2012-12-03

    Created #229: [#5389] Code browser diff improvement - offer option for side by side

    Before implementing it we need some details:

    Do we need to have this option globally for the tracker (i.e. project admin will enable that option for the entire tracker, and all the diffs will be shown side by side)?

    Or maybe we just need the ability to switch between side by side and default mode in a diff-viewer?

    Or both?

     

    Related

    Tickets: #5389

  • Dave Brondsema

    Dave Brondsema - 2012-12-03

    I think it should be an option for the end-user to switch between the views. It depends on the specific commit and user preference as to which is better. Perhaps store the current mode in the session, so that when you go to a new commit it uses the mode you were using last time.

     
  • Igor Bondarenko - 2012-12-04
    • status: open --> in-progress
     
  • Igor Bondarenko - 2012-12-11

    Closed #229.

    Created #235: [#5389] side-by-side diff: improve styles and check in other browsers (1cp)

    allura:42cc_5389

     

    Related

    Tickets: #5389

  • Igor Bondarenko - 2012-12-12
    • status: in-progress --> code-review
     
  • Igor Bondarenko - 2012-12-12

    Closed #235.

    All changes are in allura:42cc_5389a

     
  • Cory Johns - 2012-12-14
    • qa: Cory Johns
     
  • Cory Johns - 2012-12-17

    Excellent. I made a couple of very minor UI tweaks but it was very well done.

     
  • Cory Johns - 2012-12-17
    • status: code-review --> closed
    • milestone: forge-backlog --> forge-dec-14
     
  • Cory Johns - 2012-12-18
    • milestone: forge-dec-14 --> forge-jan-11
     

Log in to post a comment.