Closed Bug 1078220 Opened 10 years ago Closed 10 years ago

Missed Incoming call from Desktop to mobile (via sharing URL) does not show the specific link in the call log

Categories

(Firefox OS Graveyard :: Gaia::Loop, defect)

ARM
Gonk (Firefox OS)
defect
Not set
critical

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 1078225

People

(Reporter: oteo, Assigned: arnau)

References

Details

(Whiteboard: [mobile app][blocking][tef-triage])

Attachments

(3 files)

Attached image no-link.png
STR:
1. Make a Loop call to a no registered contact in Loop via the address book (e.g noloopid@gmail.com)
2. As the contact is not available, Share URL screen is shown. Send that URL via  e-mail to your Desktop.
3.Initiate a Loop call from your Desktop, clicking on that link.
4.Do not answer the call from the device.

CURRENT RESULT:
In the call log, the specific link does not appear, only {link} string is shown, what it's not correct. See no-link.png attachment

Also there are two entries in the Call log, what it's wrong but it's alreday reported in bug 1077408 
EXPECTED RESULT:
A incoming call from a clicked URL should show the specific link
Attached image link-text-overlaps.png
The fault seems to be due to a missing "$" in line 323 in index.html file.
After adding it, the link is shown but there is an overlapping problem as we need more lines to show this info (as we do in the Share URL tab)

Arnau, could you help us to resolve both issues (the missing $ and the overlapping) in this bug?
Flags: needinfo?(rnowmrch)
Blocks: 1036490
Whiteboard: [mobile app][blocking][tef-triage]
Assignee: nobody → rnowmrch
Status: NEW → ASSIGNED
Attached file patch in github
Attachment #8500963 - Flags: review?(borja.bugzilla)
Flags: needinfo?(rnowmrch)
Comment on attachment 8500963 [details] [review]
patch in github

This needs some fixes in CSS added as requirement. Could you take a look? Thanks!
Attachment #8500963 - Flags: review?(borja.bugzilla)
Comment on attachment 8500963 [details] [review]
patch in github

Text goes in one line, so r? again ;)
Attachment #8500963 - Flags: review?(borja.bugzilla)
Comment on attachment 8500963 [details] [review]
patch in github

Please rebase & merge!
Attachment #8500963 - Flags: review?(borja.bugzilla) → review+
This was added in bug 1078225, so closing as duplicate
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: