Closed Bug 1384281 Opened 7 years ago Closed 7 years ago

Update Screenshots to version 10.9.0

Categories

(Firefox :: Screenshots, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 56
Tracking Status
firefox55 blocking fixed
firefox56 --- fixed

People

(Reporter: ianbicking, Assigned: ianbicking)

Details

Attachments

(1 file)

Only one change in this version:

* Remove Save Full Page and Save Visible https://github.com/mozilla-services/screenshots/pull/3208 - Avoids https://github.com/mozilla-services/screenshots/issues/3182
Attachment #8890084 - Flags: review?(dtownsend) → review?(kmaglione+bmo)
Comment on attachment 8890084 [details]
Bug 1384281 - Export Screenshots 10.9.0 to Firefox;

https://reviewboard.mozilla.org/r/161156/#review166532

::: browser/extensions/screenshots/webextension/selector/ui.js
(Diff revision 1)
> -            overlay.querySelector(".visible").textContent = browser.i18n.getMessage("saveScreenshotVisibleArea");
> -            overlay.querySelector(".full-page").textContent = browser.i18n.getMessage("saveScreenshotFullPage");

Should these locale strings also be removed?
Attachment #8890084 - Flags: review?(kmaglione+bmo) → review+
Comment on attachment 8890084 [details]
Bug 1384281 - Export Screenshots 10.9.0 to Firefox;

https://reviewboard.mozilla.org/r/161156/#review166534

::: browser/extensions/screenshots/webextension/selector/ui.js
(Diff revision 1)
> -            overlay.querySelector(".visible").textContent = browser.i18n.getMessage("saveScreenshotVisibleArea");
> -            overlay.querySelector(".full-page").textContent = browser.i18n.getMessage("saveScreenshotFullPage");

We've only done an extremely minimal removal of the feature (all the code related to the feature is also still in the product) – we did it this way due to the late date, and because we plan to bring the feature back in the next release of Screenshots.
Keywords: checkin-needed
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/7a7e5b833c4b
Export Screenshots 10.9.0 to Firefox; r=kmag
Keywords: checkin-needed
We need to uplift this in 56 before we go live, tracked as blocking.
(In reply to Ritu Kothari (:ritu) from comment #5)
> We need to uplift this in 56 before we go live, tracked as blocking.

I meant 55!
https://hg.mozilla.org/mozilla-central/rev/7a7e5b833c4b
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Comment on attachment 8890084 [details]
Bug 1384281 - Export Screenshots 10.9.0 to Firefox;

Approval Request Comment
[Feature/Bug causing the regression]:
Long-delayed shot creation on Imgur and Giphy: https://github.com/mozilla-services/screenshots/issues/3182

[User impact if declined]:
Users who use the Full Page screenshot feature will see a feature that is very slow.

[Is this code covered by automated tests?]:
No

[Has the fix been verified in Nightly?]:
Yes (by SoftVision)

[Needs manual test from QE? If yes, steps to reproduce]: 
No

[List of other uplifts needed for the feature/fix]:
None

[Is the change risky?]:
No

[Why is the change risky/not risky?]:
It just hides the buttons for taking full page or visible screenshots, it otherwise doesn't change the code.

[String changes made/needed]:
None
Attachment #8890084 - Flags: approval-mozilla-beta?
Comment on attachment 8890084 [details]
Bug 1384281 - Export Screenshots 10.9.0 to Firefox;

disable full page screenshots, beta55+, for 55.0b13
Attachment #8890084 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Target Milestone: --- → mozilla56
Target Milestone: mozilla56 → Firefox 56
Product: Cloud Services → Firefox
You need to log in before you can comment on or make changes to this bug.