Adium

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#8042 closed defect (fixed)

Display Pictures Get Stuck On Screen

Reported by: timhuff Owned by: evands
Milestone: Component: Message View
Version: Severity: normal
Keywords: Cc: timhuff@…
Patch Status:

Description

Just a heads up, not really a huge issue, but if you click a display picture in chat to make it larger, then press command+w to close out of the window, the larger display picture stays and you have to restart adium to get it to go away. again, not a huge problem, all you have to do is... not do that. but still a small bug.

Attachments (1)

Picture 3.png (14.6 KB) - added by timhuff 13 years ago.

Download all attachments as: .zip

Change History (5)

Changed 13 years ago by timhuff

Attachment: Picture 3.png added

comment:1 Changed 13 years ago by Evan Schoenberg

It seems this could only happen if AIImageButton is leaking since it should close the image on dealloc.

comment:2 Changed 13 years ago by Evan Schoenberg

Owner: set to Evan Schoenberg
Status: newassigned

comment:3 Changed 13 years ago by Evan Schoenberg

Resolution: fixed
Status: assignedclosed

(In [21218]) NSToolbarItem leaks its view, it appears. If we call setView: for a toolbar itme, be sure to do setView:nil when it is removed from the toolbar (as notified by NSToolbarDidRemoveItemNotification). Fixes #8042 which discussed the user icon toolbar button leaving the icon on screen after the window closed, which could only occur if it was never deallocated.

comment:4 Changed 13 years ago by Evan Schoenberg

(In [21219]) Merged [21218]: NSToolbarItem leaks its view, it appears. If we call setView: for a toolbar itme, be sure to do setView:nil when it is removed from the toolbar (as notified by NSToolbarDidRemoveItemNotification). Fixes #8042 which discussed the user icon toolbar button leaving the icon on screen after the window closed, which could only occur if it was never deallocated.

Note: See TracTickets for help on using tickets.