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

[Annotations] "Error saving edited shot" dialog is displayed when you try to save a very large edited shot #3943

Closed
SoftVision-CosminMuntean opened this issue Jan 5, 2018 · 0 comments
Assignees
Labels
[QA]:Normal issue Label for QA to mark normal issues logged [QA]:Verified fixed Label for QA to mark verified fixed issues

Comments

@SoftVision-CosminMuntean

[Notes]:

  • Also, after you dismiss the dialog, you get another one: "Connection error".

[Affected versions]:

  • Nightly 59.0a1
  • Screenshots 28.0.0 dev

[Affected Platforms]:

  • All Windows
  • All Mac
  • All Linux

[Prerequisites]:

  • The "xpinstall.signatures.required" boolean pref is set to "false".
  • The "extensions.legacy.enabled" boolean pref is set tot "true".
  • Have the Screenshots 28.0.0 dev version installed from here, on a new clean profile.

[Steps to reproduce]:

  1. Open latest Nightly build with the profile from prerequisites and navigate to a large website (eg. imgur.com).
  2. Scroll the page to load more content and from the "Page Actions" menu choose the "Take a Screenshot" option.
  3. Click the "Save full page" button from the top right corner of the page.
  4. Click the "Edit this shot" button.
  5. Draw something on the shot and click the "Save" button.
  6. Observe the behavior.

[Expected result]:

  • The changes are successfully saved.

[Actual result]:

  • "Error saving edited shot" dialog is displayed and the changes are not saved.

[Additional Notes]:

  • Here is a screen recording with the issue:
    error saving edited shot
@ghost ghost added this to the Sprint 5 (60-0) 🔘 milestone Jan 9, 2018
@chenba chenba assigned chenba and unassigned nihakhanna Jan 22, 2018
chenba added a commit to chenba/screenshots that referenced this issue Jan 22, 2018
@SoftVision-CosminMuntean SoftVision-CosminMuntean added [QA]:Verified fixed Label for QA to mark verified fixed issues [QA]:Normal issue Label for QA to mark normal issues logged labels May 7, 2018
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
[QA]:Normal issue Label for QA to mark normal issues logged [QA]:Verified fixed Label for QA to mark verified fixed issues
Projects
None yet
Development

No branches or pull requests

5 participants