Proposed update for sticky footers in compass

_sticky_footer.sass

//  Version: 1.4

// Based on http://ryanfait.com/resources/footer-stick-to-bottom-of-page/
// Mix into the top level.
// Footer element must be outside of root wrapper element.
// Footer must be a fixed height.

// I've run tests on this method in the following browsers. That being said, 
// let me know if you find any bugs (josh.hall@surrealnotions.com).
// Tested browsers: IE6, IE7, IE8, Safar 4, Chrome 3, Firefox 3.6

// Example HAML structure:
// %body.bp
//   #root
//     / if using blueprint or other grid system, set the main grid container here
//     .container
//       / content and/or blueprint columns
//     #root-footer
//   #footer
//     / if using blueprint or other grid system, set the main grid container here
//     .container
//       / footer content and/or blueprint columns


// Required for +clearfix mixin
@import compass/utilities/general/clearfix.sass

// Required for +sticky-footer mixin
@import compass/layout/sticky_footer.sass

// These are the default selector names
!root_selector ||= "#root"
!root_footer_selector ||= "#root-footer"
!footer_selector ||= "#footer"

=grid-sticky-footer(!footer_height = 40px)
  +sticky-footer(!footer_height, !root_selector, !root_footer_selector, !footer_selector)
  // Patches for IE
  #{!root_selector}
    // Necessary to keep the footer out of the content when scrolling in IE8
    :height auto !important
    // Necessary for IE6 to behave and keep the footer at the bottom when not scrolled
    :height 100%
  // Optional grid components... these could easily be handled separately as a single definition for .content
  #{!root_selector} .container
    +container
  #{!footer_selector} .container
    +container