Closed Bug 637630 Opened 12 years ago Closed 12 years ago

Compose throbber doesn't

Categories

(SeaMonkey :: MailNews: Composition, defect)

Product:
SeaMonkey
Component:
MailNews: Composition
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: neil, Assigned: neil)

Details

Attachments

(1 file)

Proposed patch
2.36 KB, patch
mnyromyr
: review+
Details | Diff | Splinter Review 
Steps to reproduce problem:
1. Send/post a message

Expected result: Compose throbber throbs while message sends

Actual result: Progress meter shows progress only
Attached patch Proposed patch — — Splinter Review 
Assignee: nobody → neil
Status: NEW → ASSIGNED

        Attachment #515885 -
        Flags: review?(mnyromyr)

    
    

    
  
Comment on attachment 515885 [details] [diff] [review]
Proposed patch

>+        document.getElementById('navigator-throbber').setAttribute( "busy", "true" );
...
>+        document.getElementById('navigator-throbber').removeAttribute( "busy" );

I don't think that losing the superfluous whitespace would be too severe a violation of the "adhere to local style" rule. ;-)

        Attachment #515885 -
        Flags: review?(mnyromyr) → review+

    
    

    
  
Pushed changeset 27f43f536cb2 to comm-central.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED

          You need to log in
          before you can comment on or make changes to this bug.