2941: "Show Whitespace changes" button is not operatonal
- Fixed
- Review Board
rad***@gmai***** (Google Code) (Is this you? Claim this profile.) | |
|
|
Aug. 7, 2013 |
What version are you running? 1.7.6 What's the URL of the page containing the problem? https://reviewboard-openlmi.rhcloud.com/r/54/diff/ What steps will reproduce the problem? Show/Hide Whitespace changes button doesn't do anything, just changes the button label This might hide some issues from reviewer (e.g. indentation changes in Python code) What is the expected output? What do you see instead? Whitespace changes will be highlighted as any other changes. What operating system are you using? What browser? Fedora 18, Firefox 19.0.2
Please point out which lines of the diff there should be displaying whitespace differences as well, please. It'll make it easier to locate the problem.
Line 123-124: self.print_qualifiers(p.qualifiers, param_indent+ " "*4) got indented (because of added condition) but not highlighted.
This was fixed in commit 5e67070, but Christian forgot to mark the bug as fixed.
-
- PendingReview + Fixed
What version of RB has this fix? I am seeing the problem on 1.7.28.