-
Yes this was fixed.
-
Previous Value Current Value Open
Closed
| Type |
Enhancement
|
| Priority |
Minor
|
| Assignee |
Issue Votes (0)
Yes this was fixed.
| Previous Value | Current Value |
|---|---|
Open | Closed |
| Type |
Enhancement
|
| Priority |
Minor
|
| Assignee |
I am not sure if this would be possible but during code review collapsed code lines often result in presenting an irritating context.
For example consider the code
This would result in a collapsed code review view that looks like
So now you might think the code has been edited in
method2()while in reality it was inmethod4(). This can be irritating sometimes.I am wondering if it would be possible for OneDev to make sure to always show the enclosing context of a modified code section, e.g. the method that contains the modified code?
Also as a side note I noticed some
skipped 1 linesections which seem useless and you could just render the code line instead. Maybe theskipped X linesblock should have a minimum amount of lines before using it. I feel like hiding 1, 2 or maybe even 3 lines does not provide too much value.