Closed Bug 1096590 Opened 10 years ago Closed 10 years ago

Persist typing notifications over link displays in status panel

Categories

(Instantbird Graveyard :: Conversation, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: aleth, Assigned: aleth)

References

Details

(Keywords: regression)

Attachments

(1 file)

      No description provided.
Attached patch typing.diffSplinter Review
Attachment #8520198 - Flags: review?(florian)
Comment on attachment 8520198 [details] [diff] [review]
typing.diff

Review of attachment 8520198 [details] [diff] [review]:
-----------------------------------------------------------------

::: im/content/instantbird.js
@@ +133,5 @@
>    },
>  
>    updateStatusField: function () {
>      var text, type, types = ["overLink"];
> +    // if (this._busyUI)

Any reason this is still here? (Hmm...I seem to remember asking this once before! :-D)
Attachment #8520198 - Flags: review?(florian) → review-
(In reply to Patrick Cloke [:clokep] from comment #2)
> Any reason this is still here? (Hmm...I seem to remember asking this once
> before! :-D)

It's baaaack! Now the change is actually needed though. I kept the commented out line to aid future ports of changes to this ported code.
Attachment #8520198 - Flags: review- → review?(florian)
(In reply to aleth [:aleth] from comment #3)
> (In reply to Patrick Cloke [:clokep] from comment #2)
> > Any reason this is still here? (Hmm...I seem to remember asking this once
> > before! :-D)

To be clear, the alternative is setting the _busyUI flag, but that may end up having unexpected side-effects, if not now then in the future.
Blocks: 1105871
Attachment #8520198 - Flags: review?(florian) → review?(nhnt11)
Comment on attachment 8520198 [details] [diff] [review]
typing.diff

Review of attachment 8520198 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good. I suppose you'll want to change the commit message to r=nhnt11 :)
Attachment #8520198 - Flags: review?(nhnt11) → review+
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.6
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: