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

Comments

nod_’s picture

nod_
Primary language French
Location Lille
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
commented
Component: CSS » Bartik theme

This patch modifies Bartik CSS

mherchel’s picture

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

Reviewed and looks good :)

oriol_e9g’s picture

oriol_e9g
Primary language Catalan
Location Barcelona
commented

Go! :)

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
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.