Crucible like diff -> fold/hide unmodified pieces of code

When i review changes done in huge files, i waste to ime on navigation. Crucible allows to focus only on modified pieces of code. I found recently, for me it's easier to make review in crucible to verify changes instead of review them within idea. Is it possible to bring such feature in idea?

thank you, Alex

6 comments
Comment actions Permalink

Hello Alexander,

this feature is already implemented and soon will be available in IntelliJ IDEA 11 EAP!
I think in a couple of days we will publish a post about it in our blog.

A screenshot: (also it will be available in Changes | Local)
commit.jpg

0
Comment actions Permalink

I always knew you Jerbrain can read minds. Will wait for EAP.
Thank you, Alex

0
Comment actions Permalink

Hm, with that new feature can I really hide or fold unchanged parts of a changed file?
For example it would be great to be able to collapse all methods that have not been changed.

0
Comment actions Permalink

Hi, Friedrich.
As i see on screenshots, idea will collapse unmodified piesces of code  without providing controls to investigate hided code.
regards, Alex

0
Comment actions Permalink

Hi, Irina.
Thank you for promising news. I have some points to recall, comparing with crucible. In case of review changes it's important to realize context of change. Crucible adresses this by allowing specify so called "context" parameter, which is number  of lines to dislplay near the modified line. So, in crucible world,  full context is a full diff, and context 3 diff - is a 3 unmodified  lines to dislplay around modified line. Also, i found very useful provided by crucible unfied diff. De facto, i refused side by side diff to review changes and use only unified diff.  Another one useful crucible option - soft  wrap in diff. Imaging 2 modified long lines - unified diff is much more usable compare to side by side diff, especially with soft wrap on.
So the question - will it be possible to manage hided code and do you consider to implement unified diff?

best regards, Alex

0
Comment actions Permalink

Hello Alexander,

1) With digest changes view, it is possible to change the number of context lines displayed around changed ones.
2) Soft wrap option is available and soft wraps are enabled by default in digest changes view
3) as for unified diff, I will consider the idea. Especially for the case of applying old patches, that have lost their context
Could you please create an issue for that? (ot sure if we already have one)
Thank you

0

Please sign in to leave a comment.