Edgewall Software
Modify

Opened 17 years ago

Closed 17 years ago

#5795 closed defect (fixed)

No diff output when "White space changes" is checked

Reported by: r.sokoll@… Owned by: Tim Hatch
Priority: normal Milestone: 0.11
Component: wiki system Version: devel
Severity: trivial Keywords:
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

I changed a wiki page from

* some text

to

 * some test (note the leading space)

Now, if I want to see the diff and "White space changes" is checked, there is no diff at all.

Trac 0.11dev-r5883

Attachments (1)

Diff.png (7.0 KB ) - added by r.sokoll@… 17 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 by Tim Hatch, 17 years ago

Component: version controlwiki
Resolution: invalid
Status: newclosed

Assuming the 'text' → 'test' is a typo in the report, this is precisely what "ignore whitespace changes" is supposed to do.

Closing as invalid, feel free to reopen if there's a larger issue here.

comment:2 by r.sokoll@…, 17 years ago

Resolution: invalid
Status: closedreopened

It was a typo, of course. But, if this behavior is expected, then the diff output (without "White space changes") is wrong, see the screenshot. The leading space is missed. You may adopt the subject of this ticket, of course.

by r.sokoll@…, 17 years ago

Attachment: Diff.png added

comment:3 by Tim Hatch, 17 years ago

Oh, indeed it's wrong for me too (Firefox 2). It is only wrong in the tabular view, it's correct in the unified view. The source looks okay too, there's a <ins> </ins> in there. I'm not quite sure where it's getting lost.

comment:4 by r.sokoll@…, 17 years ago

What about <ins>&nbsp;</ins> instead of <ins> </ins>?

You may handle this as a cosmetic issue :-)

comment:5 by ThurnerRupert, 17 years ago

Milestone: 0.11
Severity: minortrivial

change to trivial if this is cosmetic …

comment:6 by Tim Hatch, 17 years ago

Owner: changed from Christian Boos to Tim Hatch
Status: reopenednew

comment:7 by Tim Hatch, 17 years ago

Resolution: fixed
Status: newclosed

Fixed in r5888 for 0.11-dev

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Tim Hatch.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Tim Hatch to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.