Design: download for Save visible and Save full page #2157
Comments
I think this is a stretch based on the current flows i've seen. Would be nice to resolve, but i don't think it blocks launch. |
I'm having a hard time putting my finger on why, but in this model it feels like "Save full page" is a modifier on the selection action, as opposed to an entirely separate action. Maybe because it looks like a select box and so it's not obvious that each item is actually an action with immediate effects. |
We tried several mockups and decided to go this proposal since it's more consistent to the current regional crop. In short, when users choose to have a full page shot, we offer them the preview of the final result to decide if download or crop. Please find the link below for the prototype and let us know your thoughts or the feasibility, thanks. https://mozilla.invisionapp.com/share/BJB0YYB9W#/225402436_2-1_Shot_page_right_side_menu_-2- |
Should this still be in Milestone 54? |
@ianb is this feasible? would be a good follow on in 56? |
Outside of the UI implementation itself, implementing download-full/visible-page is very simple. The only blocker to this has been the UI to actually ask for it to be downloaded. |
@ianb i guess my question is does the proposal that @ChenMorpheus made in the screenshot above look feasible...if you click the invision link, you can see a stepwise version of the same. password = testpilot (password not intended to be cryptographically secure) |
From Bugzilla: https://bugzilla.mozilla.org/show_bug.cgi?id=1368530
|
Here are the assets and measurement. Assets: https://github.com/mozilla/testpilot-assets/tree/master/Firefox_Screenshots/00_Assets Measurement: https://mozilla.github.io/testpilot-assets/Firefox_Screenshots/Save%20fullpage/ |
Thank you! @fangshih |
Hey @niharikak101 |
Right now we have a noticeable inconsistency where you can download a selection, but not the other options.
The text was updated successfully, but these errors were encountered: