Opened 13 years ago

Closed 13 years ago

#803 closed Defect (Fixed)

JVStyleView should not use insertBefore: to append messages, switch to use outerHTML

Reported by: timothy Owned by: timothy
Component: Colloquy (Mac) Version: Local Build
Severity: Critical Keywords: webkit
Cc: kdecker@…

Description

Right now there is a bug in appendMessage that will pass a nil object to insertBefore:. This causes an assert in debug versions of WebKit?. This code is also nto ass efficent as it could be, we can just use outerHTML to insert HTML as needed.

Change History (2)

comment:1 Changed 13 years ago by timothy

  • Status changed from new to assigned

comment:2 Changed 13 years ago by timothy

  • Resolution set to fixed
  • Status changed from assigned to closed

Fixed in [3329]

Note: See TracTickets for help on using tickets.