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

Listview filter is not themable #1790

Closed
swanson opened this issue Jun 6, 2011 · 6 comments
Closed

Listview filter is not themable #1790

swanson opened this issue Jun 6, 2011 · 6 comments

Comments

@swanson
Copy link

swanson commented Jun 6, 2011

The search filter for listviews has a hard-coded theme set and cannot be changed, ideally it should inherit the theme for the listview if possible; if not at least make this an option like data-filter-theme="a"

@StevenBlack
Copy link
Contributor

@swanson where are you seeing the hard coded list filter theme? Are you using the latest nightlies? See http://jquerymobile.com/blog/2011/04/16/latest/

@swanson
Copy link
Author

swanson commented Jun 6, 2011

jQuery Mobile v1.0a4.1

https://github.com/jquery/jquery-mobile/blob/master/js/jquery.mobile.listview.filter.js#L20

The <form> always gets set to ui-bar-c regardless of how the listview or page is themed.

It appears to be there in the 20110605 nightly as well. Just search for "ui-listview-filter ui-bar-c"

@StevenBlack
Copy link
Contributor

Ah! Right you are. I swear I searched for that.

In my opinion this is a bug with a good chance of eventual remediation.

@toddparker
Copy link
Contributor

Yeah, we'll take a look at adding this at some point. It should be themable.

@swanson
Copy link
Author

swanson commented Jun 7, 2011

@StevenBlack @toddparker thanks guys, I've just patched my local version to use the desired theme for the moment. If I get around to it, I'll try my hand at fixing it and send a pull request.

@adamvaughan
Copy link

I added pull request 1837 to correct this issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
5 participants