Update textinput.js - added 'refresh' method #5766

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@poonkave
Contributor

reference to issue #5708
added 'refresh' method to textinput  

test case : http://jsbin.com/onibuc/346/

@poonkave poonkave Update textinput.js
reference to issue #5708
added 'refresh' method to textinput  

test case : http://jsbin.com/onibuc/338/
fa0e026
@poonkave poonkave referenced this pull request Mar 13, 2013
Closed

.textinput('refresh') #5708

@jaspermdegroot
Member

@poonkave

Thanks a lot for the PR!
Although this resolves the issue as mentioned in #5708 it doesn't provide a complete refresh method. Refresh should allow to change all options on-the-fly (theme, mini, clear button text, etc.). What we probably need to do here is move code from the create method to a new refresh method and have the former call the latter.
Do you feel like working on a PR for that?

Thanks!

@poonkave poonkave added a commit to poonkave/jquery-mobile that referenced this pull request May 28, 2013
@poonkave poonkave Update textinput.js
reference to issue #5708 
and previous pull request. 
#5766 
added 'refresh' method to textinput. 
added a _private method '_refresh' and called same from create and refresh. 

demo test page :http://jsbin.com/edidil/7/ [Got various combinations of input fields and button to refresh input.] 

demo test page : http://johnchacko.net/samples/textinputs.html
c2606a3
@poonkave poonkave referenced this pull request May 28, 2013
Closed

Update textinput.js #6049

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