Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

UI: prototype -> jquery - replace calls to checked, enable and disable #1148

Closed
bmclaughlin opened this issue Dec 1, 2014 · 1 comment
Closed

UI: prototype -> jquery - replace calls to checked, enable and disable #1148

bmclaughlin opened this issue Dec 1, 2014 · 1 comment
Assignees
@h-kataria
Labels
technical debt

Comments

@bmclaughlin
Copy link
Contributor

Replace calls to prototype's .checked(), .enable() and .disable() functions with jQuery's .prop() function.
.prop() documentation link

Parent issue: #214
@bmclaughlin bmclaughlin mentioned this issue Dec 2, 2014
Closed
@dclarizio dclarizio assigned h-kataria and unassigned bmclaughlin Jan 7, 2015
bmclaughlin added a commit to bmclaughlin/manageiq that referenced this issue Jan 7, 2015
@bmclaughlin
Updated js calls to .enable and .disable to use jQuery
ca365a7 
ManageIQ#1148
@bmclaughlin bmclaughlin mentioned this issue Jan 7, 2015
Closed
bmclaughlin added a commit to bmclaughlin/manageiq that referenced this issue Jan 7, 2015
@bmclaughlin
Updated javascript_disable_field method signature to element id and bool
58a1251 
ManageIQ#1148
h-kataria pushed a commit to h-kataria/manageiq that referenced this issue Jan 13, 2015
@bmclaughlin @h-kataria
Updated javascript_disable_field method signature to element id and bool
3f2e81f 
ManageIQ#1148
h-kataria added a commit to h-kataria/manageiq that referenced this issue Jan 14, 2015
@h-kataria
Converted prototype -> jquery
64d91da 
- replace calls to checked
- split 'javascript_disable_field' method into 2 methods: javascript_disable_field & javascript_enable_field for more clarity.

ManageIQ#1148
@h-kataria
Copy link
Contributor

Fixed by #1353

@Fryguy Fryguy added this to the Sprint 18 Ending Jan 26, 2015 milestone Feb 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@h-kataria h-kataria

Labels
technical debt
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Fryguy @bmclaughlin @h-kataria