QA/Hello context
Contents
Hello with screen sharing
Ownership
Developer contacts: Mike de Boer, :mikedeboer on IRC; Mark Banner, :standard8 on IRC
QA Lead: Bogdan Maris mail , :bogdan_maris on IRC
Status
Overview
[LANDED] Nightly 41: first landed
[LANDED] Aurora 40: landed
[LANDED] Beta 40: landed
[LANDED] Release 40: landed
Risk analysis
Event | Probability | Impact | Current situation | Mitigation steps |
---|---|---|---|---|
Layout issues caused by Context | probable | high | Hello panel, Conversation menu and StandaloneUI will suffer some layout issues due to Context | Some issues were found that introduced some layout issues: bug query |
Failed to show Context | improbable | high | Context might be missing from Hello panel, Conversation menu and StandaloneUI | Periodical regression testing covers this area |
Unable to edit Context | improbable | medium | Context might not be able to be edited using the edit button | Periodical regression testing covers this area |
Summary
Context
This is a draft
As a desktop client user creating a conversation, I want to attach the URL and page title of the active tab to the conversation as contextual information so that users joining conversations are more likely to be engaged.
Acceptance criteria
- A new panel tickbox allows adding context about the currently active tab when creating a new conversation
- A thumbnail of the tab, the favicon of the active tab, the tab page title and tab URL are displayed besides the tickbox
- Creating a new conversation when having this new panel tickbox selected creates a conversation with metadata allowing to recover the thumbnail, favicon, page title and URL of the page when accessing the conversation from the desktop client or the standalone client
- The tickbox is disabled by default
- A telemetry event is sent when conversations are created with enabled tickbox
https://bug1115341.bugzilla.mozilla.org/attachment.cgi?id=8563657
As a desktop client user in a conversation, I want to see contextual information so that I am better prepared and more engaged when entering the conversation.
Acceptance criteria:
- When in a conversation (alone or not), the following details about the attached URL get displayed to the desktop client user:
- Thumbnail and Favicon of the webpage the URL points to - Content of the <title> tag of the webpage the URL points to - URL
- The contextual information in the conversation panel can be closed through the close button (next time the conversation is opened these details will re-appear)
- The contextual information in the conversation panel can be edited through the edit button:
- Clicking the edit button opens a new panel (slides up from the bottom of the conversation window) - The user can edit the URL and the description of the context - Selecting the tickbox allows changing the context to the context of the currently active tab (gets the URL and title page as a description of the conversation)
- Clicking the URL opens the URL in a new tab
- Accessing the URL from the conversation window will trigger a Telemetry event
Conversation window context on call Conversation window context on edit
As a link clicker in a conversation, I want to see any contextual information associated with the conversation so that I am better prepared and more engaged when I join.
Acceptance criteria:
- When in a conversation or out of a conversation, the following details about the attached URL get displayed to the link clicker on the right hand side of the screen:
- Thumbnail and favicon of the webpage the URL points to - Content of the <title> tag of the webpage the URL points to - URL
- Clicking the URL opens the URL in a new tab
- Accessing the URL from the standalone client will trigger a GA event
UX: Name me
As a desktop client user, I want my Hello panel to include indications about attached context so that I know the context of conversations before joining them.
Acceptance criteria:
- The favicon of the URL context added to conversations appears next to the conversation name in the panel
- Hovering over the Favicon displays a tooltip with the site title
- Conversations where no URL context was added get displayed without changes
- The URLs are not displayed in the panel anymore
UX: Name me
Enabling the feature
- Enabled by default in Nightly:
- Build from 2015-05-10 bug
References
- The planned UX for integrating Screen Sharing in Hello is available here.
- Hello context architecture can be found in this Wiki page
Test cases
- [ON TRACK] Draft test suite: gdoc.
Bug work
Tracking bug
meta: Bug 1115340 - [meta] Implement ability to add context to conversations (v1)
- blocks: Bug 971986 - (loop_mvp) [meta] Loop MVP desktop Client
- dependency tree: Bugzilla query
Bug triage +/- for verification
Bug 1172847 - 'Let’s Talk About' hard to see in conversation window after someone joins the conversation due to its color
- ↳ fixed in FF40
Bug 1170627 - Save Context button doesn't exit screen
- ↳ fixed in FF40
Bug 1154775 - Upgrade OpenTok library to v2.5.1
- ↳ fixed in FF41
Bug 1168851 - Text chat should be disabled when the other participant leaves the room
- ↳ fixed in FF41
Bug 1110937 - When link clickers visit an expired URL they should see an Expired Link page, rather than the Link Clicker UI
- ↳ fixed in FF42
Bug fix verification
[VERIFIED] Bug 1165287 - Glitches when opening hello panel
- ↳ 2015-05-26
[VERIFIED] Bug 1142522 - Implement transition from roomName to encrypted context
- ↳ 2015-06-10
[VERIFIED] Bug 1142522 - Implement transition from roomName to encrypted context
- ↳ 2015-06-10
[VERIFIED] Bug 1164510 - Context needs a default favicon for the panel room list
- ↳ 2015-06-15
[VERIFIED] Bug 1162909 - Implement layout changes for context view in the conversation window
- ↳ 2015-06-15
[VERIFIED] Bug 1162905 - Implement layout changes for context view in the rooms list
- ↳ 2015-06-09
[VERIFIED] Bug 1162903 - Switch to using favicons as the context URL preview image
- ↳ 2015-06-09
[VERIFIED] Bug 1162570 - Enable context in nightly builds
- ↳ 2015-05-18
[VERIFIED] Bug 1162444 - Update conversation window context view data after editing context
- ↳ 2015-06-09
[VERIFIED] Bug 1162442 - Add tooltips to the buttons inside the conversation window context area
- ↳ 2015-05-18
[VERIFIED] Bug 1158725 - Share link button has stopped working in latest nightly
- ↳ 2015-05-18
[VERIFIED] Bug 1142588 - Implement context in conversations display for standalone UI
- ↳ 2015-06-11
[VERIFIED] Bug 1142587 - Implement indicators for context in conversations in the panel alongside room names
- ↳ 2015-06-05
[VERIFIED] Bug 1142532 - Update the invitation email that is sent if context is included in conversations
- ↳ 2015-06-11
[VERIFIED] Bug 1142525 - Implement adding context to conversations when a new conversation is started
- ↳ 2015-06-09
[VERIFIED] Bug 1171940 - Move the context views to the chat area inside conversations
- ↳ 2015-07-09
[VERIFIED] Bug 1170627 - Save Context button doesn't exit screen
- ↳ 2015-06-15
Logged bugs
[NEW] Bug 1169632 - Conversation name vanishes if the window is detached by dragging
- ↳ 2015-05-29:
[RESOLVED INVALID] Bug 1169607 - Unable to activate Context on 39 branch
- ↳ 2015-05-29:
[FIXED] Bug 1165287 - Glitches when opening hello panel
- ↳ 2015-05-15:
[RESOLVED WORKSFORME] Bug 1165289 - Extra white space in hello panel
- ↳ 2015-05-15:
[RESOLVED DUPLICATE] Bug 1165846 - Context remains active even though 'Talk about...' is not checked
- ↳ 2015-05-18:
[RESOLVED DUPLICATE] Bug 1165882 - Missing conversation name in hello panel
- ↳ 2015-05-18:
[REOPENED] Bug 1165927 - Unaligned checkbox with `Let`s talk about' string
- ↳ 2015-05-18:
[NEW] Bug 1171925 - Name and favicon from conversation window are redirecting to shared webpage in context
- ↳ 2015-06-05:
[NEW] Bug 1174132 - Incomplete name shown in context if website name contains vertical bar
- ↳ 2015-06-12:
[NEW] Bug 1174702 - Unnecessary scroll while editing context
- ↳ 2015-06-15:
[RESOLVED WORKSFORME] Bug 1175833 - Buttons from conversation window are gone after adding/editing context
- ↳ 2015-06-18:
[NEW] Bug 1175841 - [Linux] Inconsistent font size in context edit screen
- ↳ 2015-06-18:
[NEW] Bug 1180655 - Context shown for some about: pages
- ↳ 2015-07-06:
[NEW] Bug 1180700 - Favicon missing from StandaloneUI context, when comments field contains long word, more then 20 characters
- ↳ 2015-07-06:
[NEW] Bug 1182055 - [Linux] Context not displayed via Chromium
- ↳ 2015-07-09:
Sign off
Criteria
- All the test cases were executed.
- All the blocker, critical, major bugs have been fixed.
Results
Aurora Sign-off - (2015-07-02)
- [DONE] Aurora 40
Beta Sign-off - (2015-07-02)
- [DONE] Beta 40
Release Sign-off (date)
- [DONE] RC 40.0 (date).