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

[FxA] The synced icon wrongly exceeds the shots area if the height of the saved shot is small #4958

Closed
SoftVision-CosminMuntean opened this issue Sep 26, 2018 · 1 comment
Assignees
Labels
[QA]:Major issue Label for QA to mark major issues logged [QA]:Verified fixed Label for QA to mark verified fixed issues

Comments

@SoftVision-CosminMuntean

[Affected versions]:

  • Nightly 64.0a1
  • Screenshots 34.0.0 dev

[Affected Platforms]:

  • All Windows
  • 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 34.0.0 dev version installed from here, on a new clean profile.

[Prerequisites]:

  • You are logged into FxA.
  • Have multiple saved shots synced that have a small height.

[Steps to reproduce]:

  1. Open the Firefox browser with the profile from prerequisites.
  2. Navigate to "My Shots" page.
  3. Observe the Synced icon displayed on the synced shots.

[Expected result]:

  • The synced icon is correctly displayed on each shot.

[Actual results]:

  • The synced icon exceeds the shots area if the image has a small height.

[Notes]:

  • Here is a screenshot of the issue:
    synced icon
@SoftVision-CosminMuntean SoftVision-CosminMuntean added the [QA]:Major issue Label for QA to mark major issues logged label Sep 26, 2018
@chenba chenba self-assigned this Sep 26, 2018
@chenba chenba added this to the Sprint 22 (64-3) milestone Sep 26, 2018
chenba added a commit to chenba/screenshots that referenced this issue Sep 26, 2018
@SoftVision-CosminMuntean
Copy link
Author

I have verified this issue using latest Screenshots 34.0.0 de version and the issue is no longer reproducible. Tested on Windows 7 x64, Mac 10.13 and Arch Linux 4.12.

@SoftVision-CosminMuntean SoftVision-CosminMuntean added the [QA]:Verified fixed Label for QA to mark verified fixed issues label Sep 28, 2018
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
[QA]:Major issue Label for QA to mark major issues logged [QA]:Verified fixed Label for QA to mark verified fixed issues
Projects
None yet
Development

No branches or pull requests

2 participants