Edgewall Software
Modify

Opened 19 years ago

Closed 18 years ago

#2213 closed defect (worksforme)

#!diff preview in wiki hides previous line

Reported by: anonymous Owned by: Jonas Borgström
Priority: normal Milestone:
Component: wiki system Version: 0.9b2
Severity: normal Keywords: iexplorer
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

Rendering Wiki Preview with #!diff from IE6(WinXP), diff outputs hide previous line. Firefox 1.0.7 does not have this problem.

= header 1 =
this is content1.
== header 2 ==
this is content2.
{{{
#!diff
 ... some unified diff
}}}

Preview at wiki, "this is content2." overlapped diff output. After submit this, "this is content2." line can see even if I saw this page by IE6.

Attachments (1)

Panorama2.png.uue (209.4 KB ) - added by anonymous 18 years ago.
Difference between Windows Firefox 2 and Windows IE 6

Download all attachments as: .zip

Change History (7)

comment:1 by sid, 18 years ago

Keywords: iexplorer needinfo added

Does this only happen for the preview, but not the normal saved page rendering?

comment:2 by anonymous, 18 years ago

header 1

this is content1.

header 2

this is content2.

 ... some unified diff

comment:3 by anonymous, 18 years ago

header 1

this is content1.

header 2

this is content2.

  • trac/ticket/web_ui.py

     
    461461            ticket = ctx.shortname() + ': '
    462462            if status == 'closed':
    463463                ticket = tag.span(ticket, style="text-decoration: line-through")
    464             yield (ctx.resource_href(), ticket + ctx.summary(),
     464            yield (ctx.resource_href(),
     465                   ticket + ctx.format_summary(summary, status),
    465466                   datetime.fromtimestamp(ts, utc), author,
    466467                   shorten_result(desc, terms))

comment:4 by Christian Boos, 18 years ago

Keywords: needinfo removed
Resolution: worksforme
Status: newclosed

Well, whatever that error was, it seems to be working now.

comment:5 by anonymous, 18 years ago

Resolution: worksforme
Status: closedreopened

I tested this at SandBox on this server, here is test code.

= header 1 =
this is content1.
== header 2 ==
this is content2.
{{{
#!diff
--- /trunk/trac/htdocs/js/expand_dir.js (revision 5338)
+++ /trunk/trac/htdocs/js/expand_dir.js (revision 5346)
@@ -39,5 +39,5 @@
   // rows to be removed when collapsing that folder
   var folderid = "f" + counter++;
-  td.addClass(folderid);
+  td.addClass(folderid).addClass("expanded");
   var ancestor_folderids = $.grep(tr.attr("class").split(" "), 
 				  function(c) { return c.match(/^f\d+$/)});
}}}
  1. View this page in Windows IE6.
  2. Select & copy above code, then press Ctrl+C.
  3. Click here → [SandBox].
  4. Click [Edit this page] button.
  5. Select all existing wiki code, then press Ctrl+V.
  6. Click [Preview] button. "Header 2" and "this is content2." are hidden by diff table.

by anonymous, 18 years ago

Attachment: Panorama2.png.uue added

Difference between Windows Firefox 2 and Windows IE 6

comment:6 by hyuga <hyugaricdeau@…>, 18 years ago

Resolution: worksforme
Status: reopenedclosed

Well, I just tried it myself and was unable to reproduce it anywhere. It's possible that when you tried it in SandBox, some invalid wiki code somewhere (and there is a lot of invalid stuff on that page) might have messed things up. But there doesn't seem to be a problem here.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Jonas Borgström.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Jonas Borgström 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.