Closed Bug 1187392 Opened 9 years ago Closed 7 years ago

Remove pin-the-web setting, and "add to homescreen" feature

Categories

(Firefox OS Graveyard :: Gaia::System::Browser Chrome, defect, P2)

ARM
Gonk (Firefox OS)
defect

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: benfrancis, Unassigned)

References

Details

(Whiteboard: [systemsfe])

Attachments

(1 obsolete file)

Add to homescreen will be replaced with pinning in 2.5
Blocks: 1185772
I believe this is already fixed as part of earlier pin-the-web work. I don't see it in the context menu anymore so I'm closing for now. Feel free to re-open if I'm wrong.
Status: NEW → RESOLVED
blocking-b2g: --- → 2.5+
Closed: 9 years ago
Priority: -- → P2
Resolution: --- → WORKSFORME
Discussed in IRC, let's use this bug to track the work to remove "Add to homescreen" entirely once Pin the Web is on by default. Currently you switch between add to homescreen and pin with a pref.
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
We also want to use this bug for removing the setting behind which pin-the-web is hidden. This is a non-user facing feature, and so it doesn't block 2.5.
blocking-b2g: 2.5+ → ---
Summary: Remove "add to homescreen" feature → Remove pin-the-web setting, and "add to homescreen" feature
Let's disable the bookmark marionette js tests for the new homescreen for the time being, since they are flaky and they test a code path we won't support after 2.5. When we land this bug, let's just remove the tests completely.
Comment on attachment 8684983 [details] [gaia] mikehenrty:bug-1187392-disable-bookmark-tests > mozilla-b2g:master Chris, can you give the rubber stamp here?
Attachment #8684983 - Flags: review?(chrislord.net)
Comment on attachment 8684983 [details] [gaia] mikehenrty:bug-1187392-disable-bookmark-tests > mozilla-b2g:master Unfortunately not, sorry :( bookmark_test can be disabled because (as you pointed out on IRC) it's covered by the pinning_the_web test... But the rest of them are testing functionality that no other marionette test hits. We'd need to replace the bit in these tests that opens and saves a bookmark with a bit that opens and pins a site instead (which ought to be trivial, I think?) Personally, I'd prefer to keep bookmark_test and do the same search/replace there too, I'd like to keep the tests as (for the most part) one-test-per-file (then the pinning-the-web test could be reduced to a pin-page test, I would guess?)
Attachment #8684983 - Flags: review?(chrislord.net) → review-
Yeah, you're probably right, we should manually convert each test. My hope was that we had enough functionality coverage in our existing pin the web tests such that we could just remove the bookmark tests. Better to convert, and indeed that was my idea in bug 1215524.
Depends on: 1215524
Attachment #8684983 - Attachment is obsolete: true
Firefox OS is not being worked on
Status: REOPENED → RESOLVED
Closed: 9 years ago7 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: