Adium

Opened 14 years ago

Closed 13 years ago

Last modified 13 years ago

#6396 closed defect (fixed)

“Hide contact list in the screen borders” animation doesn't move smoothly

Reported by: enkuturi Owned by: Catfish_Man
Milestone: Adium X 1.1 Component: None
Version: Severity: major
Keywords: Cc:
Patch Status:

Description

when the option to hide the contact list in the screen borders is turned on, when you show the contact list it moves fine, but the last in the last moment the animation stop for like 0.5 secs. and then continues.

Change History (8)

comment:1 Changed 14 years ago by Bastian

im using 10.3.9 and a 400mhz power pc machine

comment:2 Changed 14 years ago by Evan Schoenberg

Milestone: Adium X 1.0.2

This only happens when using the standard window style; borderless styles are not affected.

comment:3 Changed 14 years ago by David Smith

I think this code snippet from AIListWindowController may be the guilty party

int counter = 10000;
	AIListWindowController *windowController = (AIListWindowController *)[inWindow windowController];
	//snipped for brevity
	do 
	{
		//this is dumb, but I've seen situations where we get into an infinite loop here. 10k is probably enough, if it's not, up it a bit.
		if(counter <= 0) return;
		else counter--;

comment:4 Changed 14 years ago by Chris Forsythe

Owner: changed from nobody to David Smith
Severity: normalblocker

Setting as a blocker for 1.0.2 and assigning

comment:5 Changed 14 years ago by Chris Forsythe

Milestone: Adium X 1.0.2Adium X 1.0.3

comment:6 Changed 13 years ago by Eric Richie

Milestone: Adium X 1.0.3Adium X 1.1
Patch Status: None

comment:7 Changed 13 years ago by Evan Schoenberg

Resolution: fixed
Status: newclosed

(In [20012]) Used NSViewAnimation instead of custom animation code for showing/hiding the contact list on screen edges. Fixes #6396

comment:8 Changed 13 years ago by Evan Schoenberg

(In [20018]) Really fix the standard contact list's sliding on screen. The docking window code was grabbing it as it neared the edge and causing a stutter. Fixes #6396 for real.

Note: See TracTickets for help on using tickets.