While mobile device support is not yet the major priority, making sure all the major components of this theme is, so I want to do more testing in IE 7 and 8 especially, as well as FireFox. There are always some weird but annoying bugs that result from my insistence on only working in Chrome/Safari during initial design stages :-)

CommentFileSizeAuthor
#2 theme-compatibility-1354760-2.patch1.04 KBgeerlingguy
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

geerlingguy’s picture

geerlingguy CreditAttribution: geerlingguy commented
Status: Needs review » Needs work

Notes:

  • FireFox: Everything seems to be working fine.
  • Internet Explorer 9: Not tested.
  • Internet Explorer 8: Looks fine after patch in next comment.
  • Internet Explorer 7: Looks fine after patch in next comment.
  • iPad (iOS 5): Not tested.
  • iPhone (iOS 5): Not tested.
  • Android (Gingerbread 2.3.7): Not tested.
geerlingguy’s picture

geerlingguy CreditAttribution: geerlingguy commented
Assigned: Unassigned » geerlingguy
FileSize
theme-compatibility-1354760-2.patch1.04 KB

Attached patch (which I'll commit in a few minutes) fixes a few small issues (mostly overflow/block-level issues caused by HTML5 markup and 'section' or 'article' elements) with Internet Explorer 7 and 8.

[Edit: Committed: http://drupalcode.org/project/parish_theme.git/commit/381aa85]

geerlingguy’s picture

geerlingguy CreditAttribution: geerlingguy as a volunteer commented
Issue summary: View changes
Status: Needs work » Fixed

This was fixed a long while back.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.