Bug 1679 – Screenshot Tools 5 second delay non-functional

••• Introduction •••
••• Keywords •••

Audacity 3.0.3 development began 19th April 2021

Audacity Bugzilla

Bug 1679 - Screenshot Tools 5 second delay non-functional


Summary:	Screenshot Tools 5 second delay non-functional

Status:	RESOLVED QUICKFIXED

Product:	Audacity
Classification:	Unclassified
Component:	User Interface
Version:	2.2.0
Hardware:	Per OS All

Importance:	P4 RepeatableAll
Assigned To:	Default Assignee for New Bugs

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2017-07-11 11:59 UTC by Steve Daulton
Modified:	2018-08-20 11:46 UTC (History)
CC List:	6 users (show)

See Also:
Steps To Reproduce:	Open Screenshot Tool Check "wait 5 seconds" Click one of the "window" screenshot options. The screenshot is taken almost immediately (the delay is much less than 5 seconds).
Release Note:
First Git SHA:
Group:	---
Workaround:
Closed:	2018-08-20 00:00:00

Flags:	petersampsonaudacity: Test‑OK‑Mac+

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Steve Daulton 2017-07-11 11:59:37 UTC

Selecting the 5 second delay in the Screenshot Tool does not delay the screenshot.

An additional enhancement would be to make the delay selectable - 5 seconds is a bit quick in some cases, such as taking a screenshot of an effect GUI. A configurable delay between say 0 to 10 seconds would be much better.

Comment 1 James Crook 2017-08-14 13:56:17 UTC

DEVEL - FIX MADE
https://github.com/audacity/audacity/commit/e23acb39ebf7b7d04356aca97638758c5d025940

Comment 2 Peter Sampson 2017-08-15 06:09:59 UTC

Testing on macOS Sierra 10.12.6 db97fea 15Aug

There is now an addition checkbox for "Wait 5 seconds ..." before the screenshot is taken.

Tested with full window, full screen and andividual elemenst.

Looks to be ok on Mac

Comment 3 James Crook 2017-08-15 06:27:21 UTC

RESOLVED FIXED
Per comment #2.
RepeatableAll is by default testable on a single OS.

N.B: The checkbox existed before but stopped working due to a mistake I made with even numbers.

Comment 4 James Crook 2017-08-15 06:28:37 UTC

In comment 3, for 'even numbers' read 'event numbers'.

(In more detail I put them in a more logical and tidy order, not realising the reason for the untidy order).