Skip to content
This repository has been archived by the owner on Jan 17, 2023. It is now read-only.

Add option for a fullpage screenshot #1613

Closed
myBestSoftAndPref opened this issue Sep 29, 2016 · 12 comments
Closed

Add option for a fullpage screenshot #1613

myBestSoftAndPref opened this issue Sep 29, 2016 · 12 comments
Labels
Milestone

Comments

@myBestSoftAndPref
Copy link

myBestSoftAndPref commented Sep 29, 2016

There is not "choose all page" option like in Screengrab (fix version)
or even in Developer Tools instruments.

It's really weird situation for so big corporation with good programmers like Mozilla!

@myBestSoftAndPref myBestSoftAndPref changed the title There is not "choose all page option" There is not "choose all page" option Sep 29, 2016
@fzzzy
Copy link
Contributor

fzzzy commented Sep 29, 2016

We actually did have this feature in earlier versions of Page Shot but we really simplified the product down to the bare minimum for launch. I do think this would be a very useful feature. Hopefully we will be able to add it back in soon.

@ianb
Copy link
Contributor

ianb commented Sep 29, 2016

@myBestSoftAndPref : out of curiosity, when you say "choose all page" do you mean the entire visible area, or the entire page including lower parts of the page that may not be visible?

Also, can you share what you want to use the whole-page capture for? This might help us figure out where to put the controls and how to describe them in a later version.

@jnachtigall
Copy link

@myBestSoftAndPref @ianb I'm not the OP but did an upvote.

For me this would be entire page, i.e. exactly what screenshot --fullpage does in the devtools. To me, this is the one USP that pageshot has over other screenshot software where this is technically not possible. (Before screenshot --fullpage me and other people usually did several screenshots and then post-edited them in Gimp/Photoshop to assemble them back together)

I use this very often to share screenshots with desigerns / devlopers where I only have a desktop which is y pixels high (e.g. 768), but the area that is of interest is bigger than y (e.g. a longish page).

@jnachtigall
Copy link

FWIW, if the dragging (area selection) would scroll the page down when it hits the bottom so that the actual screenshot area can be bigger than the screen size, then this might be also ok instead of a fullpage option.

Maybe also make the selection snappy towards the edges of the browser window so that's it's easy to select the full width/height.

@ianb
Copy link
Contributor

ianb commented Sep 30, 2016

#193 describes the scroll-near-edge-of-page feature. I opened #1658 about making sure the snapping that does exist includes the edges of the screen.

If you are patient you can currently make a selection larger than the size of the screen by resizing, scrolling, resizing, until you've got it to the size you want.

@ianb ianb closed this as completed Sep 30, 2016
@ianb ianb reopened this Sep 30, 2016
@ianb ianb modified the milestone: Altair Oct 4, 2016
@ianb ianb added the needs:UX label Oct 5, 2016
@johngruen johngruen changed the title There is not "choose all page" option Add option for a fullpage screenshot Oct 5, 2016
@ianb ianb self-assigned this Oct 7, 2016
ianb added a commit that referenced this issue Oct 7, 2016
…full length) screen capture

Still requires UX review
@ianb
Copy link
Contributor

ianb commented Oct 7, 2016

This will land in a simple state in 40088fe (pending review) looking like:

image

I just added two vertically-stacked buttons for the two options. Obviously we'll want to review the UX. Both buttons capture the respective regions and immediate save/upload and open a new tab.

@ianb ianb removed their assignment Oct 10, 2016
@johngruen
Copy link
Contributor

@ianb could you make all three buttons the same width?

@ianb ianb self-assigned this Oct 10, 2016
ianb added a commit that referenced this issue Oct 11, 2016
@ianb ianb removed their assignment Oct 11, 2016
@fzzzy
Copy link
Contributor

fzzzy commented Oct 13, 2016

I think this is fixed now in master, right?

@ChenMorpheus
Copy link

Hi all,
We came up with the visual style with Windows shot and Fullpage shot. Please find the link below for more details, and let me know if we need to file another ticket, thanks.

https://mozilla.invisionapp.com/share/ER934Y4ND#/197768052_1-1_Select_Modes

@johngruen
Copy link
Contributor

@ChenMorpheus we may roll this out iteratively, can you provide the icon assets from the spec as SVGs?

@fangshih
Copy link

Here are the icons assets, Thanks!
#1613 Fullpage-icon-Assets.zip

@ianb
Copy link
Contributor

ianb commented Nov 1, 2016

This feature was released in the v1 release which went out on October 26th.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

7 participants