He have dropped IE 6/7/8 support, so we don't need anymore zoom: 1 property in our CSS to force the hasLayout: true property in IE 6/7

CommentFileSizeAuthor
no-zoom1.patch416 bytesoriol_e9g
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

nod_’s picture

nod_
Primary language French
Location Lille
CreditAttribution: nod_ commented
Issue tags: -drop ie7 +ie7, +ie8

just putting IE8 to have it show up in the list I'm watching, and it seems to be used up to IE10 on some CSS+JS Bugs.

LewisNyman’s picture

LewisNyman
he/him
CreditAttribution: LewisNyman commented
Component: CSS » Bartik theme

This patch modifies Bartik CSS

mherchel’s picture

mherchel
he/him
Primary language English
Location Gainesville, FL, US
CreditAttribution: mherchel commented
Status: Needs review » Reviewed & tested by the community

Reviewed and looks good :)

oriol_e9g’s picture

oriol_e9g
Primary language Catalan
Location Barcelona
CreditAttribution: oriol_e9g commented

Go! :)

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Yay, less code!

Committed and pushed to 8.0.x. Thanks!

  • webchick committed 88c988a on 8.0.x 
    Issue #2400883 by oriol_e9g: Remove zoom: 1 CSS property

Status: Fixed » Closed (fixed)

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