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

fix: jquery.mobile.theme.css (default)#774

Closed
markusweb wants to merge 3 commits intojquery-archive:masterfrom
markusweb:master
Closed

fix: jquery.mobile.theme.css (default)#774
markusweb wants to merge 3 commits intojquery-archive:masterfrom
markusweb:master

Conversation

@markusweb
Copy link

fixed the repeating of background-image for the search field.
the image was repeated in the input field. so i added "background-postion: no-repeat" to have it only once displayed. also added background-postion.
checked in firefox, safari and on iphone

@scottjehl
Copy link

Can you specify a URL and browser where you see this issue?
The icon looks fine on this page for me: http://jquerymobile.com/test/#docs/forms/forms-search.html

This existing rule should set the repeat to repeat-none for search icons: https://github.com/jquery/jquery-mobile/blob/master/themes/default/jquery.mobile.theme.css#L719

Thanks!

…splay the results with the help of jquery.tmpl.min.js.

to use this: the url must contain "mode=json" as get parameter. in the main/index.html the template has to be inserted.
the json data must contain a "template"-entry in the main section

{"id":"foobar", "template":"templateListIntro", "list":[ ... ]}
@markusweb
Copy link
Author

ok, in my case the ui-icon class is not added the div.
i have fetched the latest development build on 5th january and build the js and css. so i am not using the alpha 2 version.

https://github.com/jquery/jquery-mobile/blob/master/js/jquery.mobile.forms.textinput.js#L35
there is no ui-icon set. so ui-icon is missing.

the URL is:
http://www.wuppertal-live.de/__partnerkalender/mobile/test.html#/intro/show=stage;client=mobile;mode=

@scottjehl
Copy link

ah okay. I'll add that missing class then. Thanks!

This pull request was closed.
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