Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 370526 - ability to open two way compare from commit page
Summary: ability to open two way compare from commit page
Status: RESOLVED FIXED
Alias: None
Product: Orion
Classification: ECD
Component: Git (show other bugs)
Version: 0.4   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 0.4 RC1   Edit
Assignee: Szymon Brandys CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 370935
Blocks:
  Show dependency tree
 
Reported: 2012-02-03 02:58 EST by Susan McCourt CLA
Modified: 2012-02-09 05:30 EST (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Susan McCourt CLA 2012-02-03 02:58:01 EST
I am browsing a large commit to figure out if I just broke something, and I find it quite cumbersome.  I realize there are bugs for future releases for having the diff area in the commit page be configurable (and hopefully expandable/collapsible).  In the meantime though it would be really helpful to have:

- a link to open/edit the file (could just be the file name that is currently static text)
- a link to the two-way compare
Comment 1 Susan McCourt CLA 2012-02-03 02:59:00 EST
Szymon, do you think we could add some links in 0.4 and then revisit in 0.5 with collapsible/configurable diff areas?
Comment 2 Szymon Brandys CLA 2012-02-03 04:39:48 EST
I can add links for .4.
Comment 3 Szymon Brandys CLA 2012-02-08 06:10:22 EST
"open two way compare" added with  http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=dac251138110f404525c9b7fe7c3d9adbecadc72 
"open the current file state" blocked by bug 370935
Comment 4 Szymon Brandys CLA 2012-02-08 08:45:18 EST
Still blocked by bug 370935.
Comment 5 Szymon Brandys CLA 2012-02-09 05:30:30 EST
"open the working directory version" fixed with http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=567950ed647c3cb4a59cc261507b3f398a9953d6