Skip to content
This repository has been archived by the owner on Oct 8, 2021. It is now read-only.

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

Closed
wants to merge 1 commit into from
Closed

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

wants to merge 1 commit into from

Conversation

poonkave
Copy link
Contributor

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

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

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

test case : http://jsbin.com/onibuc/338/
@poonkave poonkave mentioned this pull request Mar 13, 2013
@jaspermdegroot
Copy link
Contributor

@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 added a commit to poonkave/jquery-mobile that referenced this pull request May 28, 2013
reference to issue jquery-archive#5708 
and previous pull request. 
jquery-archive#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
@poonkave poonkave mentioned this pull request May 28, 2013
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.

None yet

2 participants