I found a couple of minor issues in css styling for views exposed filter.
Add styling for reset filter:
/* Blue Button */
input#edit-preview,
input#edit-next,
input#preview-submit,
input#edit-reset,
input[type="submit"][id^="edit-return"] {
cursor: pointer;
border-radius: 0;
border: none;
padding: 6px 10px;
margin-bottom: 1em;
margin-right: 1em;/* LTR */
color: #fff;
text-align: center;
font-weight: normal;
font-size: 1.077em;
font-family: "Lucida Grande",Verdana,sans-serif;
background: #0074BD;
}
input#edit-preview:hover,
input#edit-next:hover,
input#edit-reset:hover,
input#preview-submit:hover,
input[type="submit"][id^="edit-return"]:hover {
background: #004c99;
border-radius: 0;
border: none;
color: #fff;
}
input#edit-preview:active,
input#edit-next:active,
input#edit-reset:active,
input#preview-submit:active,
input[type="submit"][id^="edit-return"]:active {
box-shadow: 0 2px 2px rgba(0,0,0,0.75) inset;
background: #333;
border-radius: 0;
border: none;
color: #fff;
}
Clear both for submit button syntax error
.views-exposed-widget .views-submit-button{
clear: both;
}
should be
.views-exposed-widget.views-submit-button{
clear: both;
}
Also i would sugest this code in mobile.css
.views-exposed-form .views-exposed-widget.views-submit-button,
.views-exposed-form .views-exposed-widget.views-reset-button {
width:auto;
}
Thanks for a great theme!
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | adminimal_theme-fix_exposed_filters_mobile_fix-2386597-1.patch | 367 bytes | hkirsman |
| #2 | adminimal_theme-fix_exposed_filters_button_alignment-2386597-2.patch | 420 bytes | hkirsman |
| #1 | adminimal_theme-2386597-6.jpg | 22.3 KB | hkirsman |
| #1 | adminimal_theme-2386597-5.jpg | 39.83 KB | hkirsman |
| #1 | adminimal_theme-2386597-4.jpg | 19.87 KB | hkirsman |
Comments
Comment #1
hkirsman commentedSo the fix colors reset to blue and moves the buttons below form elements:

Based on what I see on admin/content page:

I would not color reset button to blue. And there allready is a blue button in that section - Update preview.
Now what about the buttons below form elements. In all other core themes they are on the same line. Here's Bartik's example:

I like it. Takes less space for example.
So instead I would fix margin-top:
Result:

And the current problem-> solution by rodpal with mobile is this:

But instead adding:
I would remove:
Results:

If there's room, elements will align on the same line.
I'll add 2 separate patches in next comment.
Comment #2
hkirsman commentedHere are the patches.
Comment #3
ANDiTKO commentedPatch committed.
Thanks @hkirsman and @rodpal!