Closed
Bug 1250708
(saved-screenshots)
Opened 9 years ago
Closed 4 years ago
[meta] Let "snapshotted pages"/saved screenshots ride the trains
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
INCOMPLETE
People
(Reporter: mcomella, Unassigned)
References
Details
(Keywords: meta)
Initial implementation behind a nightly flag will occur in bug 1243558. Requirements:
* Smart folder
* Url association
To let it ride the trains (this bug):
* Probes
* Time/date timestamp
* Change the favicon
* Update the snackbar confirmation text (i.e. indicate offline and button to show bookmarks list)
* Display screenshot w/ intent
v2 (if we decide to move forward with this):
* Associate screenshots w/ full-page screenshots (e.g. PDFs)
* Syncing
* Clean up our references to the screenshots when they change on disk
Bugs to implement to follow! :)
Reporter | ||
Comment 1•9 years ago
|
||
Also:
* Using pages w/ screenshots to increase rating in frecency algo
Updated•9 years ago
|
Alias: bookmark-screenshots
Reporter | ||
Comment 2•9 years ago
|
||
(In reply to Michael Comella (:mcomella) from comment #1)
> Also:
> * Using pages w/ screenshots to increase rating in frecency algo
To be clear, v2.
Some items from bug 1243558 comment 11 need to ride trains:
- How to disable this functionality? (e.g. invasion of privacy!)
- Does removing an item from this smart folder remove the screenshot from disk or just FF's representation of it?
* initial antlam thoughts: just FF representation
Comment 3•9 years ago
|
||
(In reply to Michael Comella (:mcomella) from comment #2)
> (In reply to Michael Comella (:mcomella) from comment #1)
> > Also:
> > * Using pages w/ screenshots to increase rating in frecency algo
>
> To be clear, v2.
Why is this v2? This is more important than actually displaying things in a smart folder.
Reporter | ||
Comment 4•9 years ago
|
||
I'm of the opinion the folder is important in order to 1) avoid dumping things the user doesn't care about into their bookmarks (though you were talking about "hidden" bookmarks before – what did you mean by that?) and 2) to easily remove this feature if we decide we don't like it.
Perhaps Anthony has a different opinion though?
Flags: needinfo?(alam)
Comment 5•9 years ago
|
||
(In reply to Michael Comella (:mcomella) from comment #4)
> I'm of the opinion the folder is important in order to 1) avoid dumping
> things the user doesn't care about into their bookmarks (though you were
> talking about "hidden" bookmarks before – what did you mean by that?) and 2)
> to easily remove this feature if we decide we don't like it.
My point about hidden bookmarks is now moot since these screenshots will be saved in a URL metadata table, not in bookmarks. My "hidden bookmark" comment was about the fact that these bookmarks would never really be visible outside the magic "Screenshots" folder. Using bookmarks was merely a way to get frecency weighting.
Now that we are using URL metadata, it's not relevant. No screenshots would ever be saved into the user's bookmarks. We need to _add_ code to make them visible in a magic folder and _add_ code to make them weight frecency.
Updated•9 years ago
|
Alias: bookmark-screenshots → saved-screenshots
Summary: [meta] Let bookmarks-linked-screenshots ride the trains → [meta] Let "snapshotted pages"/saved screenshots ride the trains
Comment 6•9 years ago
|
||
(In reply to Mark Finkle (:mfinkle) from comment #3)
> (In reply to Michael Comella (:mcomella) from comment #2)
> > (In reply to Michael Comella (:mcomella) from comment #1)
> > > Also:
> > > * Using pages w/ screenshots to increase rating in frecency algo
> >
> > To be clear, v2.
>
> Why is this v2? This is more important than actually displaying things in a
> smart folder.
(In reply to Michael Comella (:mcomella) from comment #4)
> I'm of the opinion the folder is important in order to 1) avoid dumping
> things the user doesn't care about into their bookmarks (though you were
> talking about "hidden" bookmarks before – what did you mean by that?) and 2)
> to easily remove this feature if we decide we don't like it.
>
> Perhaps Anthony has a different opinion though?
Using the "has been screenshotted" signal to weight the URL differently is likely going to be important. So, it's probably not a "v2" right now.
And yes, I'd also like to have a smart folder of these screenshots rather than have them scattered amongst the Mobile Bookmarks root.
Flags: needinfo?(alam)
Comment 8•4 years ago
|
||
We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly](https://play.google.com/store/apps/details?id=org.mozilla.fenix) an issue can be reported at the [Fenix GitHub project](https://github.com/mozilla-mobile/fenix/). If you want to discuss your report please use [Mozilla's chat](https://wiki.mozilla.org/Matrix#Connect_to_Matrix) server https://chat.mozilla.org and join the [#fenix](https://chat.mozilla.org/#/room/#fenix:mozilla.org) channel.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → INCOMPLETE
Assignee | ||
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•