Closed Bug 1228089 Opened 10 years ago Closed 8 years ago

Detect if the user has a URL in their clipboard when re-entering the app

Categories

(Firefox for iOS :: General, defect)

Other
iOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: sleroux, Assigned: bmunar, Mentored)

References

Details

Attachments

(2 files)

48 bytes, text/x-github-pull-request
Details | Review
48 bytes, text/x-github-pull-request
fluffyemily
: review+
fluffyemily
: feedback+
Details | Review
Whenever the app enters the foreground, we need to check the UIPasteboard to see if there a link in the user's clipboard. If there is a URL, we will show the open url toast that will be built as part of https://bugzilla.mozilla.org/show_bug.cgi?id=1228087. For now, let's just print to the console whenever a URL is detected.
Assignee: nobody → bmunar
Attachment #8712871 - Flags: review?(sleroux)
Comment on attachment 8712871 [details] [review] PR - some questions in the PR Left some answers on the PR!
Left a couple of comments on the PR. I think it will make more sense to combine this patch with the snack bar view patch since this patch doesn't actually add any functionality - just debug logging.
Comment on attachment 8712871 [details] [review] PR - some questions in the PR Clearing r? until patch is revised.
Attachment #8712871 - Flags: review?(sleroux)
Attached file Pull request
Attachment #8760327 - Flags: review?(rnewman)
Attachment #8760327 - Flags: review?(rnewman) → review?(etoop)
Comment on attachment 8760327 [details] [review] Pull request Hey. You've deleted some files from another package as part of this patch, which I think must have been a mistake. Could you please update the patch to leave the file deletions out? Thanks
Attachment #8760327 - Flags: feedback+
Comment on attachment 8760327 [details] [review] Pull request LGTM. Thanks :rawisa Sorry it took me so long to get around to it. It got lost in my review queue.
Attachment #8760327 - Flags: review?(etoop) → review+
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: