Closed
Bug 1109073
Opened 9 years ago
Closed 9 years ago
Conversation name doesn't get bold in the Loop panel after starting a call
Categories
(Hello (Loop) :: Client, defect, P2)
Tracking
(firefox35 fixed, firefox36 fixed, firefox37 fixed)
backlog | Fx35+ |
People
(Reporter: pauly, Unassigned, Mentored)
References
Details
(Whiteboard: [good first verify])
Attachments
(1 file, 2 obsolete files)
4.54 KB,
patch
|
mikedeboer
:
review+
Sylvestre
:
approval-mozilla-aurora+
lsblakk
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
Press the "Start a conversation" button so that the call starts in a small window at the bottom of your screen Actual results: Conversation name doesn't get bold in the panel, while the Loop icon gets blue 37.0a1 (2014-12-09), Win 7
Updated•9 years ago
|
backlog: --- → Fx35+
Priority: -- → P1
Comment 1•9 years ago
|
||
Pauly, Can you retest with Nightly? Now that the panel closes automatically when a new conversation is opened, I think this bug is effectively fixed. (When I open the panel after the conversation view is opened, the current (active) conversation is bold in the list as it should be.)
Flags: needinfo?(paul.silaghi)
Reporter | ||
Comment 2•9 years ago
|
||
(In reply to Maire Reavy [:mreavy] (Plz needinfo me) from comment #1) > Pauly, Can you retest with Nightly? > > Now that the panel closes automatically when a new conversation is opened, I > think this bug is effectively fixed. The panel remains open to me after clicking on "start a new conversation" 37.0a1 (2014-12-12) win 7
Flags: needinfo?(paul.silaghi)
Comment 3•9 years ago
|
||
I still very much want this in fx35, but I wouldn't block the release on this.
Priority: P1 → P2
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → nperriault
Assignee | ||
Comment 4•9 years ago
|
||
There was a bug allowing to have the same room token being listed twice in the roomStore list, making react only using one of the dupes, hence sometimes having obsolete information about it.
Attachment #8538494 -
Flags: review?(standard8)
Comment 5•9 years ago
|
||
Comment on attachment 8538494 [details] [diff] [review] Ensure a Loop room entry is highlighted when opened. Review of attachment 8538494 [details] [diff] [review]: ----------------------------------------------------------------- Somehow this breaks unit tests: #getAllRooms should order the room list using ctime desc ‣ AssertionError: expected 1418937570302 to deeply equal 1405518241 I've done a bit of experimentation, I think there's something in the tests that isn't cleaning up. If I run just the #getAllRooms section, then it passes, running the whole file fails.
Attachment #8538494 -
Flags: review?(standard8) → review-
Assignee | ||
Comment 6•9 years ago
|
||
(In reply to Mark Banner (:standard8) from comment #5) > I've done a bit of experimentation, I think there's something in the tests > that isn't cleaning up. Fixed.
Attachment #8538494 -
Attachment is obsolete: true
Attachment #8539246 -
Flags: review?(mdeboer)
Updated•9 years ago
|
Attachment #8539246 -
Flags: review?(mdeboer)
Assignee | ||
Comment 7•9 years ago
|
||
As per discussed on IRC, added checks for added room data properties already provided in event data.
Attachment #8539246 -
Attachment is obsolete: true
Attachment #8539254 -
Flags: review?(mdeboer)
Comment 8•9 years ago
|
||
Comment on attachment 8539254 [details] [diff] [review] Ensure a Loop room entry is highlighted when opened. Review of attachment 8539254 [details] [diff] [review]: ----------------------------------------------------------------- Ship it!
Attachment #8539254 -
Flags: review?(mdeboer) → review+
Assignee | ||
Comment 9•9 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/6b6e49450b85
Iteration: --- → 37.1
Points: --- → 2
Target Milestone: --- → mozilla37
Updated•9 years ago
|
Comment 10•9 years ago
|
||
Comment on attachment 8539254 [details] [diff] [review] Ensure a Loop room entry is highlighted when opened. Approval Request Comment [Feature/regressing bug #]: Rooms [User impact if declined]: Inconsistent notification that someone joined your room (bad) [Describe test coverage new/current, TBPL]: on fxteam; will merge to m-c soon and I'll verify there. [Risks and why]: Low - most of the patch is test changes [String/UUID change made/needed]: none
Attachment #8539254 -
Flags: approval-mozilla-beta?
Attachment #8539254 -
Flags: approval-mozilla-aurora?
https://hg.mozilla.org/mozilla-central/rev/6b6e49450b85
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Updated•9 years ago
|
Updated•9 years ago
|
Attachment #8539254 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Updated•9 years ago
|
Updated•9 years ago
|
Attachment #8539254 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Updated•9 years ago
|
Mentor: anthony.s.hughes
Flags: qe-verify-
Whiteboard: [good first verify]
You need to log in
before you can comment on or make changes to this bug.
Description
•