Closed Bug 330523 Opened 17 years ago Closed 17 years ago

Replace Animated scroll with a fade in for new mail alert notification

Categories

(Thunderbird :: Mail Window Front End, defect)

Product:
Thunderbird
Component:
Mail Window Front End
Platform:
x86
Windows XP
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
Thunderbird2.0

People

(Reporter: mscott, Assigned: mscott)

References

Details

(Keywords: fixed1.8.1)

Attachments

(1 file)

add the fade effect
11.09 KB, patch
Bienvenu
: superreview+
Details | Diff | Splinter Review 
Instead of sliding the new mail alert up into view and then sliding it back down, fade the alert in and then fade it away.

This looks a lot smoother than the scroll which tends to get a bit jerky.
Status: NEW → ASSIGNED
In addition to replacing the scrolling animation with fade in / fade out, I removed some dead code from newmailalert.js and renamed the animation boolean argument to aUserInitiated.

Behaviorally, the fade only occurs when the alert is automatically generated. If the user clicks on the biff icon to open the alert, we show it immediately without any fade.
Attachment #215085 - Flags: superreview?(bienvenu)
Comment on attachment 215085 [details] [diff] [review]
add the fade effect

cool
Attachment #215085 - Flags: superreview?(bienvenu) → superreview+
sweet!
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Keywords: fixed1.8.1
Resolution: --- → FIXED
The fix does not work for me.  I'm assuming that the whole box is supposed to fade out, but instead the text inside fades out, and then the box dissapears.
(In reply to comment #4)
> The fix does not work for me.  I'm assuming that the whole box is supposed to
> fade out, but instead the text inside fades out, and then the box dissapears.
> 

I see the same using current 2.0b1pre en-US nightlies on Windows XP. Is there a bug number for this?
You need to log in before you can comment on or make changes to this bug.