Closed
Bug 1044188
Opened 10 years ago
Closed 10 years ago
Standalone -- change unsupported browser notice to be more accurate
Categories
(Hello (Loop) :: Client, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
34 Sprint 1- 8/4
People
(Reporter: mreavy, Assigned: mreavy)
References
Details
(Whiteboard: [prioritize uplifting])
Attachments
(1 file, 2 obsolete files)
2.67 KB,
patch
|
dmosedale
:
review+
|
Details | Diff | Splinter Review |
When a link-clicker clicks on a URL from a browser that doesn't suport WebRTC, he/she sees: "To use Loop, please use the latest version of Firefox." Instead the link clicker should se "Please try this link in a WebRTC-enabled browser."
Assignee | ||
Comment 1•10 years ago
|
||
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → mreavy
Status: NEW → ASSIGNED
Assignee | ||
Updated•10 years ago
|
Attachment #8462777 -
Flags: review?(dmose)
Assignee | ||
Comment 2•10 years ago
|
||
Assignee | ||
Comment 3•10 years ago
|
||
Assignee | ||
Updated•10 years ago
|
Attachment #8462854 -
Flags: review?(dmose)
Assignee | ||
Updated•10 years ago
|
Attachment #8462853 -
Attachment is obsolete: true
Assignee | ||
Updated•10 years ago
|
Attachment #8462777 -
Attachment is obsolete: true
Attachment #8462777 -
Flags: review?(dmose)
Comment 4•10 years ago
|
||
Comment on attachment 8462854 [details] [diff] [review]
Update message for link clickers using browsers without WebRTC support
Review of attachment 8462854 [details] [diff] [review]:
-----------------------------------------------------------------
Since there's not yet a process in place for anyone to translate our standalone strings, and this one's going to have to change before long anyway, I don't think it's a problem leave the string id as is.
ux-r not currently required.
r=dmose
Attachment #8462854 -
Flags: review?(dmose) → review+
Assignee | ||
Updated•10 years ago
|
Target Milestone: --- → 34 Sprint 1- 8/4
Comment 5•10 years ago
|
||
Assignee | ||
Updated•10 years ago
|
Whiteboard: [prioritize uplifting]
Comment 6•10 years ago
|
||
Is this bug only about a non supported browser on a supported platform?
If yes then we need a bug for non supported platforms (iOS) - we had https://bugzilla.mozilla.org/show_bug.cgi?id=974895 although it's now been closed - should we just re-open it to implement the new UX?
We need some UX here as it's not part of the current link clicker design.
Flags: needinfo?(dhenein)
Comment 7•10 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 8•10 years ago
|
||
(In reply to Romain Testard [:RT] from comment #6)
> Is this bug only about a non supported browser on a supported platform?
The filed definition was unclear.
> If yes then we need a bug for non supported platforms (iOS) - we had
> https://bugzilla.mozilla.org/show_bug.cgi?id=974895 although it's now been
> closed - should we just re-open it to implement the new UX?
If the device isn't supported, then we currently show:
===
Sorry, Loop does not currently support your device.
Please open this page using the latest Firefox on Windows, Android, Mac or Linux.
===
If we want to change that as well, please file a new bugs. Re-opening old bugs generally doesn't work and messes up tracking if backports are needed.
Comment 9•10 years ago
|
||
Thanks. I created a new bug to track the implementation of a new UX for non supported platforms: https://bugzilla.mozilla.org/show_bug.cgi?id=1045495
We need a new UX for this bug (current NI to Darrin) and I also NI Arcadio who was interested in this scenario in past conversations around promotion of Firefox to non current Firefox users. Arcadio please let us know quickly if you require a particular wording there.
Blocks: 994274
Flags: needinfo?(alainez)
Comment 10•10 years ago
|
||
(In reply to Romain Testard [:RT] from comment #9)
> We need a new UX for this bug (current NI to Darrin) and I also NI Arcadio
> who was interested in this scenario in past conversations around promotion
> of Firefox to non current Firefox users. Arcadio please let us know quickly
> if you require a particular wording there.
We've already landed the patch here, and Maire has requested that we push this text live asap. I don't know where it came from, but please can we handle further changes (which I assume will be non-urgent) in another bug.
Comment 11•10 years ago
|
||
OK - new bug created to track new UX implementation: https://bugzilla.mozilla.org/show_bug.cgi?id=1045498
Flags: needinfo?(dhenein)
Flags: needinfo?(alainez)
Comment 12•10 years ago
|
||
I believe this bug should be covered well enough with smoketesting via https://moztrap.mozilla.org/manage/case/14509/. Please needinfo me to request specific testing.
Whiteboard: [prioritize uplifting] → [prioritize uplifting][qa-]
Flags: qe-verify-
Whiteboard: [prioritize uplifting][qa-] → [prioritize uplifting]
You need to log in
before you can comment on or make changes to this bug.
Description
•