cfpb / regulations-site

(DEPRECATED) Web interface for viewing U.S. federal regulations and other regulatory information
Other
28 stars 43 forks source link

IE9 crash be gone #690

Closed ascott1 closed 9 years ago

ascott1 commented 9 years ago

IE9 was continually crashing after a few seconds. It took several days to diagnose that the problem was two concurrent elements styled with display:none. Using visibility:hidden changes the design slightly, so is only applied to IE9.

This also removes the compiled CSS from the repo since we're no longer checking in the JavaScript.

KimberlyMunoz commented 9 years ago

I can't actually replicate the issue this is supposed to fix. :fearful:

ascott1 commented 9 years ago

That's a good thing! (Aside from the wasted time... ). Maybe it's something weird with the IE9 VM I have.

ascott1 commented 9 years ago

Closing this. The couple of worthwhile things in here will be in a new pull request.