Closed Bug 1227106 Opened 9 years ago Closed 9 years ago

Add 'Open Last Tab' dynamic Quick Action to home screen

Categories

(Firefox for iOS :: General, defect)

Other
iOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Tracking Status
fxios 2.0+ ---

People

(Reporter: fluffyemily, Assigned: fluffyemily, Mentored)

References

(Depends on 1 open bug)

Details

Attachments

(1 file)

48 bytes, text/x-github-pull-request
sleroux
: review+
tecgirl
: ui-review-
Details | Review
If the apps tab queue is not empty, add a 'Open Last Sent Tab' (wording to be confirmed) Quick Action to the quick actions menu, otherwise do not display

https://mozilla.invisionapp.com/share/SN5082W3J#/screens/116857630
Mentor: randersen
Summary: Add 'Open Last Sent Tab' dynamic Quick Action to home screen → Add 'Open Last Tab' dynamic Quick Action to home screen
Wording to be 'Open Last Tab'. Confirm with :tecgirl before implementing.
Assignee: nobody → etoop
Status: NEW → ASSIGNED
This is not really possible as originally invisioned as we can only display actions that are defined when the app last went into the background/was terminated. This means that tabs sent to the device from an extension cannot be added to the dynamic actions until _after_ the app has been opened.

:tecgirl - what we can do here is add the action when we open the sent tabs in the app, so that next time you hit quick actions you can see the last tab that was sent to you (similar to the bookmarks). Is this enough for a first pass while you rethink?
Flags: needinfo?(randersen)
:fluffyemily - yes, that would be good enough for now. thank you!
Flags: needinfo?(randersen)
Attached file Pull request
Use the View Later or Send Tab action extensions to add an Open Last Tab quick action to the home screen menu.
Attachment #8695896 - Flags: ui-review?(randersen)
Attachment #8695896 - Flags: review?(sleroux)
Comment on attachment 8695896 [details] [review]
Pull request

Looks good - same comments as the other related PR. Also unable to test the force touch interaction.
Attachment #8695896 - Flags: review?(sleroux) → review+
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Comment on attachment 8695896 [details] [review]
Pull request

I could not get this to work using View Later, but I did get it to work after using Send Tab. However, the URL isn't show below the title, see http://c.tecgirl.com/eA8x
Attachment #8695896 - Flags: ui-review?(randersen) → ui-review-
(In reply to Robin Andersen [:tecgirl] from comment #6)
> Comment on attachment 8695896 [details] [review]
> Pull request
> 
> I could not get this to work using View Later, but I did get it to work
> after using Send Tab. However, the URL isn't show below the title, see
> http://c.tecgirl.com/eA8x

I'm raising these as a 2 separate issues:

View Later not setting Last Tab quick action: https://bugzilla.mozilla.org/show_bug.cgi?id=1236567
URL not showing below title of Last Tab quick action: https://bugzilla.mozilla.org/show_bug.cgi?id=1236565
Blocks: 1236567
Blocks: 1236565
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: