Skip to content
This repository was archived by the owner on Oct 9, 2018. It is now read-only.

Added enable, disable, show, hide, remove to checkboxes and added show, ...#405

Merged
tneil merged 1 commit intoblackberry:nextfrom
lyricidal:next
Oct 22, 2012
Merged

Added enable, disable, show, hide, remove to checkboxes and added show, ...#405
tneil merged 1 commit intoblackberry:nextfrom
lyricidal:next

Conversation

@lyricidal
Copy link
Contributor

...hide, remove to toggle buttons

Added convenience features to show, hide, and remove checkboxes and toggle buttons.

document.getElementById('theID').show();
document.getElementById('theID').hide();
document.getElementById('theID').remove();

Also added enable/disable for checkboxes only:

document.getElementById('theID').enable();
document.getElementById('theID').disable();

In reference to:
#315
#317

Note: resubmitting with all changes in one (and using the latest next branch as of today) - tried to add this to my previous pull request but was not having any luck (files weren't syncing correctly). Apologies if opening a new request made it any more trouble.

@tneil
Copy link
Collaborator

tneil commented Oct 21, 2012

Cool.. Moved this to the 0.9.5 milestone

tneil pushed a commit that referenced this pull request Oct 22, 2012
Added enable, disable, show, hide, remove to checkboxes and added show, ...
@tneil tneil merged commit c851d9b into blackberry:next Oct 22, 2012
@tneil
Copy link
Collaborator

tneil commented Oct 22, 2012

Looks good Justin, I've merged in the changes

@tneil
Copy link
Collaborator

tneil commented Oct 22, 2012

Oh.. can you also check out the following issue.. I found that there is a slight difference when rendering the disabled state

#410

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants