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

Rogue Copy to Clipboard Button(?) #4008

Closed
chenba opened this issue Jan 24, 2018 · 1 comment · Fixed by #4038
Closed

Rogue Copy to Clipboard Button(?) #4008

chenba opened this issue Jan 24, 2018 · 1 comment · Fixed by #4038
Assignees
@chenba
Labels
defect firefox Something that has to ship in the add-on in Firefox [QA]:Other issue Valid issues logged by other people than the Softvision QA team. [QA]:Verified fixed Label for QA to mark verified fixed issues
Milestone
Sprint 6 (60-1) 👨🏼‍🚀

Comments

@chenba
Copy link
Collaborator

chenba commented Jan 24, 2018

STR

  1. Resize your browser window so that there's an horizontal scrollbar (imgur.com works)
  2. Move the horizontal scrollbar to right
  3. Start screenshots and drag to select an area
  4. Move the right border of the selection box as close to the right edge of the page as possible

Actual Result

The copy to clipboard button is at the bottom right of the selection box while the rest of the buttons are at the top left. (See first image below.)

Depending on how close the left border of the selection box is to the left edge of the page, the other buttons are partially obscured or not visible. (See second image below)

image

image

Expected

All the buttons are visible...somewhere.
@ianb ianb added defect firefox Something that has to ship in the add-on in Firefox labels Jan 24, 2018
@ghost ghost added this to the Sprint 6 (60-1) 👨🏼‍🚀 milestone Jan 29, 2018
@ghost ghost assigned chenba Jan 29, 2018
chenba added a commit to chenba/screenshots that referenced this issue Jan 29, 2018
@chenba
Remove a button position fix that's no longer needed. (mozilla-servic…
a619bc3 
…es#4008)
chenba added a commit to chenba/screenshots that referenced this issue Jan 29, 2018
@chenba
Remove a button position fix that's no longer needed. (mozilla-servic…
b956bf7 
…es#4008)
@chenba chenba mentioned this issue Jan 29, 2018
Merged
@ianb ianb closed this as completed in #4038 Feb 1, 2018
@SoftVision-CosminMuntean
Copy link

I have retested this issue and is no longer reproducible on the latest Screenshots 29.0.0 version.

@SoftVision-CosminMuntean SoftVision-CosminMuntean added the [QA]:Verified fixed Label for QA to mark verified fixed issues label Feb 9, 2018
@SoftVision-CosminMuntean SoftVision-CosminMuntean added the [QA]:Other issue Valid issues logged by other people than the Softvision QA team. label Aug 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@chenba chenba

Labels
defect firefox Something that has to ship in the add-on in Firefox [QA]:Other issue Valid issues logged by other people than the Softvision QA team. [QA]:Verified fixed Label for QA to mark verified fixed issues
Projects
None yet
Milestone
Sprint 6 (60-1) 👨🏼‍🚀
Development

Successfully merging a pull request may close this issue.

Remove a button position fix that's no longer needed. chenba/screenshots
3 participants
@ianb @chenba @SoftVision-CosminMuntean