Web page title / content information isn't shown in the standalone infobar when two people are in the room

RESOLVED FIXED

Status

P1
normal
Rank:
18
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: standard8, Assigned: mancas)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [btpp-fix-now])

Attachments

(2 attachments)

(Reporter)

Description

3 years ago
The title of the web page / context information isn't show in the standalone infobar when two people are in the room.

You get something until the link generator enters the room, and then it is blank.

Sevaan, can you confirm how this is meant to work please? (I think its update the title of the page being shared as we get the new tile information i.e. on domain name change?)
Flags: needinfo?(sfranks)
Are you referring to the title bar in the conversation window?
(Reporter)

Comment 2

3 years ago
(In reply to Sevaan Franks [:sevaan] from comment #1)
> Are you referring to the title bar in the conversation window?

No, the standalone web page - where the ToS is displayed, but then changes to the context title when you join the room.
Rank: 18
Whiteboard: [btpp-fix-now]
Ah, well it should work however we have the Conversation Window title currently work.

I believe it updated for each domain, unless the user ends the session, and then it updates with the last page the user was on. Also, to prevent flooding, the room title should update after a user has spent X seconds on the page. That way we prevent the title updating if someone is flipping through a lot of tabs.

I feel that we should have a "Welcome to [Room Name]" at the top of the chat though. Why/when did it get removed?

Pau, do you know?
Flags: needinfo?(sfranks) → needinfo?(b.pmm)
(Assignee)

Comment 4

3 years ago
What we have at the top of the chat is "Welcome to Firefox Hello" and as you said, the tile change in the standalone every time the owner spent a few seconds on a different page but when the clicker leaves the room, the ToS is displayed again.
(Assignee)

Updated

3 years ago
Assignee: nobody → b.mcb
Status: NEW → ASSIGNED

Comment 5

3 years ago
You're right Sevaan. I wasn't aware it got removed. I would still go with what we had planned and move forward for future versions.
So, to sum up:

- When users don't set a name for the room = "Welcome to Firefox Hello"
- When users do set a name for the room = "Welcome to [Room Name]"
Flags: needinfo?(b.pmm)
Created attachment 8739368 [details] [review]
[loop] mancas:bug1262455 > mozilla:master
(Assignee)

Updated

3 years ago
Attachment #8739368 - Flags: review?(standard8)
Attachment #8739368 - Flags: review?(crafuse)
(Reporter)

Comment 7

3 years ago
Created attachment 8739413 [details]
Screen shot of issues (link clicker only in room -> both in room)

I probably should have included this to be clear in the beginning this is about the infobar, not the text chat area. 

This never says "Welcome to ...." - it shows the ToS before the room is joined. Then when the link clicker is the only one in the room, it gives the room context information.

Finally, when the link generator joins, the room context information disappears.

That's the weird bit, and I think the intention was to display the latest room context there.

The "Welcome to ..." isn't part of this bug, that's bug 1262441.
(Reporter)

Comment 8

3 years ago
Comment on attachment 8739368 [details] [review]
[loop] mancas:bug1262455 > mozilla:master

r- as this is addressing the wrong issue.
Attachment #8739368 - Flags: review?(standard8)
Attachment #8739368 - Flags: review?(crafuse)
Attachment #8739368 - Flags: review-
Thanks Mark. 

When a guest first joins the room, display TOS.
After clicking the Join button, if the guest is alone display "Welcome to [Room Name]"
If guest is not alone, or the room owner joins, then display page title of whatever is being shared/looked at.
(Assignee)

Updated

3 years ago
Attachment #8739368 - Flags: review?(standard8)
Attachment #8739368 - Flags: review?(fernando.campo)
Attachment #8739368 - Flags: review?(dcritchley)
Attachment #8739368 - Flags: review?(crafuse)
Attachment #8739368 - Flags: review-
(Assignee)

Comment 10

3 years ago
New patch: The info bar display "Welcome to [Room name]" if the room has a name or "Welcome to Firefox Hello" if not.
Comment on attachment 8739368 [details] [review]
[loop] mancas:bug1262455 > mozilla:master

small nit on github, and needs some fixes on the tests.
Otherwise than that code is ok, tested on nightly.

r+ after fixes.
Attachment #8739368 - Flags: review?(standard8)
Attachment #8739368 - Flags: review?(fernando.campo)
Attachment #8739368 - Flags: review?(dcritchley)
Attachment #8739368 - Flags: review?(crafuse)
Attachment #8739368 - Flags: review+
(Reporter)

Comment 12

3 years ago
https://github.com/mozilla/loop/commit/78e4e52bc17daa7a946c0870793f680f87f754b5
Status: ASSIGNED → RESOLVED
Iteration: --- → 48.3 - Apr 25
Last Resolved: 3 years ago
Resolution: --- → FIXED
(Reporter)

Updated

3 years ago
Blocks: 1265865
You need to log in before you can comment on or make changes to this bug.