866066 - Add a screenshot button to the desktop-helper.

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia, defect)

Description

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Attachment: Patch

I found that useful when it is times to takes screenshots for UX.

Comment 1

[Kevin Grandon :kgrandon]

Hmm, we have to think of where we draw the line - though this is a pretty light patch, we don't want the panel getting too bloated.

Generally I would advocate letting this be it's own standalone plugin, though admittedly plugins are difficult to keep with our current profile situation. It also appears that most of the screenshot plugins are not seriously maintained, see: https://addons.mozilla.org/en-US/firefox/addon/screengrab/

I am just on the fence about this right now - but perhaps we can land it as we can always back out later should the UI become too bloated.

Comment 2

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

(In reply to Kevin Grandon :kgrandon from comment #1)
> Hmm, we have to think of where we draw the line - though this is a pretty
> light patch, we don't want the panel getting too bloated.
> 
> Generally I would advocate letting this be it's own standalone plugin,
> though admittedly plugins are difficult to keep with our current profile
> situation. It also appears that most of the screenshot plugins are not
> seriously maintained, see:
> https://addons.mozilla.org/en-US/firefox/addon/screengrab/
> 
> I am just on the fence about this right now - but perhaps we can land it as
> we can always back out later should the UI become too bloated.

As I said I found it convenient. Let's remove it later it the panel is too bloated but let makes it easy and handful for now (this is my 2 cents).

Comment 3

[Kevin Grandon :kgrandon]

Actually this totally makes sense as it's hardware buttons that the phone supports. If possible I'd prefer to add a hardware button that's something like home + sleep button presses and have the system app handle the screenshot. Let me experiment with your patch.

Comment 4

[Kevin Grandon :kgrandon]

Ah, I see that the current screenshot logic is in shell.js. Yet another argument to see if we can embed or share code with it somehow. In this case, I think the approach is good.

Comment 5

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

https://github.com/vingtetun/gaia/commit/142dda6dfe500479c8cfba2bdcd3244b70434ea9