Opened 12 years ago

Closed 11 years ago

#4342 closed defect (fixed)

Tree: keyboard navigation code can be shorter

Reported by: bill Owned by: bill
Priority: high Milestone: 1.1
Component: Accessibility Version: 0.9
Keywords: Cc:
Blocked By: Blocking:

Description

Keyboard navigation code in Tree.js (originally in Controller.js) keeps repeating the lines: 

		if(returnWidget && returnWidget.isTreeNode){
			returnWidget.tree.focusNode(returnWidget);
			return returnWidget;
		}

should be consolidated

Change History (2)

comment:1 Changed 11 years ago by bill

Owner: changed from Becky Gibson to bill
Status: new → assigned

comment:2 Changed 11 years ago by bill

Resolution: → fixed
Status: assigned → closed

(In [11989]) Tree: fix a bunch of lint errors and reduce size of keyboard handling code, which seemed to be returning values that were never used. Fixes #4342. Still a few errors reported in other code, but they appear spurious. !strict

Note: See TracTickets for help on using tickets.