While viewing a topic's comments, would it be technically feasible to bring up a file's diff (as shown when viewing the Topic Text and reviewing the code) in a popup window (as provided by the 'Parallel' link with SCM integration)? The shown link could be named differently (e.g., 'Show diff') to avoid confusion with the existent ones.
The diff information is there, the rendering is there, and the advantage would be complete independence from any SCM (in my environment, the Codestriker server does not have any SCM connection). Not being able to view the file via a direct link slows down the answering of comments considerably (because the contextual information is missing). Now, a second browser window has to be opened, and the browsing to the relevant file has to be done manually.
Thanks for your consideration,
Christian
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
This has been implemented - thankfully it was pretty easy. If you like, you can try it out using what is in CVS now, or wait until 1.9.7 in the next week or so.
Cheers,
David
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Hello,
While viewing a topic's comments, would it be technically feasible to bring up a file's diff (as shown when viewing the Topic Text and reviewing the code) in a popup window (as provided by the 'Parallel' link with SCM integration)? The shown link could be named differently (e.g., 'Show diff') to avoid confusion with the existent ones.
The diff information is there, the rendering is there, and the advantage would be complete independence from any SCM (in my environment, the Codestriker server does not have any SCM connection). Not being able to view the file via a direct link slows down the answering of comments considerably (because the contextual information is missing). Now, a second browser window has to be opened, and the browsing to the relevant file has to be done manually.
Thanks for your consideration,
Christian
Hi Christain,
Sorry for the delay - I've been away on holidays.
Your request makes sense, and it shouldn't be that hard to implement. I'll try and squeeze it in for the upcoming 1.9.7. release.
Cheers,
David
Hi Christain,
This has been implemented - thankfully it was pretty easy. If you like, you can try it out using what is in CVS now, or wait until 1.9.7 in the next week or so.
Cheers,
David
Dear David,
No problem :-)
I'm looking forward to version 1.9.7.
Cheers,
Christian