Make WordPress Core

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#23997 closed defect (bug) (fixed)

Twenty Thirteen: Search Box Alignment IE9

Reported by: rdall's profile RDall Owned by: lancewillett's profile lancewillett
Milestone: 3.6 Priority: normal
Severity: normal Version: 3.6
Component: Bundled Theme Keywords: has-patch
Focuses: Cc:

Description

On Windows 7 IE 9 The search box is not vertically centered in the nav bar. Also there is no text "search..." in the input box and no search icon as seen in FF on the mac.

See Screenshot:
http://cl.ly/image/1m2X202D0w3O

Attachments (1)

23997.diff (1.0 KB) - added by obenland 11 years ago.

Download all attachments as: .zip

Change History (14)

#1 @SergeyBiryukov
11 years ago

  • Summary changed from Search Box Alignment IE9 to Twenty Thirteen: Search Box Alignment IE9

#2 @lancewillett
11 years ago

  • Milestone changed from Awaiting Review to 3.6

#3 @obenland
11 years ago

IE9 does not support placeholders. Not sure how much we can do about the alignment issue either.

#4 @RDall
11 years ago

Doesn't Twenty-Thirteen have it's own IE style sheet?

#5 @mjbanks
11 years ago

Could always add an IE9 conditional to the <html> element in addition to IE7 and IE8 to tweak the IE9 specific styles.

As for placeholders, what about using jquery.placeholder - https://github.com/mathiasbynens/jquery-placeholder - if the browser doesn't support HTML5 placeholders, it will drop them into place, but do nothing if the browser supports it.

@obenland
11 years ago

#6 @obenland
11 years ago

  • Keywords has-patch needs-testing added; needs-patch removed

IE9 doesn't know what a search input type is. Patch adds a fallback for outdated browsers.

#7 @lancewillett
11 years ago

@obenland Should the older selectors just go into IE file instead?

#8 @obenland
11 years ago

ie.css doesn't get loaded for IE9. Also there may be other browsers out there that might not support search type inputs.

#9 @lancewillett
11 years ago

  • Owner set to lancewillett
  • Resolution set to fixed
  • Status changed from new to closed

In 23961:

Twenty Thirteen: add fallback selectors for older browsers that don't understand search input elements. Fixes #23997, props obenland.

#10 @lancewillett
11 years ago

  • Keywords needs-testing removed
  • Resolution fixed deleted
  • Status changed from closed to reopened

Looking at IE7 today, saw the search field wasn't usable.

#11 follow-up: @RDall
11 years ago

I didn't think WP was still supporting IE7 ? My Bad… 

#12 @lancewillett
11 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

In 24035:

Twenty Thirteen: search form fixes for IE7, and add fallback search type selector to RTL stylesheet, see r23961. Closes #23997.

#13 in reply to: ↑ 11 @SergeyBiryukov
11 years ago

Replying to RDall:

I didn't think WP was still supporting IE7 ? My Bad… 

See #18199.

Note: See TracTickets for help on using tickets.