Bug 662 – Plot Spectrum: Selection and Focus trapped in "Grids"

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

Audacity 3.0.3 development began 19th April 2021

Audacity Bugzilla

Bug 662 - Plot Spectrum: Selection and Focus trapped in "Grids"


Summary:	Plot Spectrum: Selection and Focus trapped in "Grids"

Status:	RESOLVED FIXED

Product:	Audacity
Classification:	Unclassified
Component:	User Interface
Version:	2.0.5
Hardware:	Other Linux

Importance:	P3 Repeatable
Assigned To:	Default Assignee for New Bugs

URL:
Keywords:	patch_ready

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2013-10-08 15:14 UTC by Gale Andrews
Modified:	2018-08-20 11:46 UTC (History)
CC List:	6 users (show)

See Also:
Steps To Reproduce:	1 Open Plot Spectrum 2 Press TAB to move into "Grids" and then TAB or SHIFT + TAB are trapped.
Release Note:	GROUP:Accessibility *(Linux) '''If you TAB into "Grids" in Plot Spectrum, selection and focus become trapped.''' A mouse click elsewhere will be required to free the focus. '''Workaround:''' Access the required part of Plot Spectrum with TAB or SHIFT + TAB so as to avoid "Grids". Grids can be set on or off by exiting Audacity, show hidden files and folders then in ~/.audacity-data/audacity.cfg, ensure you have a line [FreqWindow] followed by DrawGrid=1 if you want grids or DrawGrid=0 if you don't want grids.
First Git SHA:
Group:	---
Workaround:
Closed:	2018-08-20 00:00:00

Flags:	gale: Accessibility+

Attachments
Fix for spectrum and equalizer "Grids" controls (1.17 KB, patch) 2013-10-08 18:11 UTC, Leland Lucius	Details \| Diff
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 Gale Andrews 2013-10-08 15:14:34 UTC

Presumably can be fixed by left-aligning the checkbox on Linux only, as in the solution for bug 356.

Comment 1 Leland Lucius 2013-10-08 18:11:45 UTC

Created attachment 411 [details]
Fix for spectrum and equalizer "Grids" controls

Comment 2 Gale Andrews 2013-10-08 22:23:30 UTC

Marked "patch_ready", Leland, thanks.

Comment 3 Steve Daulton 2013-10-08 22:53:17 UTC

Works here too (Debian Stable)

Comment 4 Vaughan Johnson 2013-10-09 01:34:44 UTC

(In reply to comment #3)

Looks good. Leland, the honors are yours, please commit.

Comment 5 Leland Lucius 2013-10-11 11:55:52 UTC

Can I still commit this?

Comment 6 Gale Andrews 2013-10-11 13:22:13 UTC

(In reply to comment #5)
Leland wrote:
> Can I still commit this?
It doesn't affect strings. It does affect visible GUI on Linux, but the Manual does not show Linux images of the affected dialogues, so would not need to change if this was committed.  Up to Vaughan to decide, but I'm +1 given we agreed to commit the patch attached to bug 57.

Comment 7 Vaughan Johnson 2013-10-11 15:47:24 UTC

(In reply to comment #6)

>> Can I still commit this?
> It doesn't affect strings. It does affect visible GUI on Linux, but the Manual
> does not show Linux images of the affected dialogues, so would not need to
> change if this was committed.  Up to Vaughan to decide, but I'm +1 given we
> agreed to commit the patch attached to bug 57.
> 

Yes, please commit it.

Comment 8 Vaughan Johnson 2013-10-11 15:51:34 UTC

(In reply to comment #7)

SVN is back up for commits, so please proceed. :-)

Comment 9 Leland Lucius 2013-10-11 16:13:54 UTC

Committed