Don't try to activate disabled items in dropdown menus
ClosedPublic
Actions

Authored by epriestley on Oct 14 2013, 12:26 AM.

Tags

None

Referenced Files

	F15523268: D7293.id16423.diff
	Mon, Apr 21, 12:18 AM

	F15520242: D7293.diff
	Sun, Apr 20, 4:46 AM

	F15507989: D7293.id16448.diff
	Wed, Apr 16, 12:51 AM

	F15478980: D7293.diff
	Tue, Apr 8, 4:33 AM

	F15425854: D7293.id16423.diff
	Mar 23 2025, 6:27 AM

	F15422450: D7293.id.diff
	Mar 22 2025, 8:28 AM

	F15343928: D7293.id16448.diff
	Mar 10 2025, 1:59 AM

	F15296020: D7293.diff
	Mar 5 2025, 11:20 AM

View All Files

Subscribers

aran

Details

Reviewers

btrahan
chad
va.multimoney

Commits

Restricted Diffusion Commit
rP994823524b57: Don't try to activate disabled items in dropdown menus

Summary

Fixes an issue where the user could click a disabled dropdown menu item and get an exception or some other nonsense. Instead, just don't activate anything.

Test Plan

Clicked a disabled header, like "Members of project" in the policy dropdown.

Diff Detail

Branch

papply2

Lint

Lint Passed

Unit

No Test Coverage

Event Timeline

fhmgh

Although I'm sure your review was exceptionally through, I'll wait for @btrahan to take a look too.

chad accepted this revision.Oct 14 2013, 3:56 PM

Closed by commit rP994823524b57 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

__celerity_resource_map__.php

84 lines

webroot/

rsrc/

js/

core/

DropdownMenu.js

6 lines

Diff 16423

View Options

src/__celerity_resource_map__.php