Skip to:
Content

BuddyPress.org

Ticket #7314: 7314-css-fix-commented.patch

File 7314-css-fix-commented.patch, 1.9 KB (added by hnla, 4 years ago)

Update patch with inline comments for clarity.

  • src/bp-templates/bp-legacy/css/twentysixteen.css

     
    5858* @section 1.0 - Theme - Structural Elements. overrides, Globals
    5959*-------------------------------------------------------------------------------
    6060*/
     61/**
     62* Corrective measure to deal with 2016 / BP handling of no-js
     63* classes - hide menu if BP still showing 'no-js'.
     64*/
     65@media screen and (max-width: 905px) {
     66        html.js body.no-js .site-header-menu {
     67                display: none;
     68        }
     69}
     70
    6171.buddypress div.clear {
    6272        display: none;
    6373}
  • src/bp-templates/bp-legacy/css/twentysixteen.scss

     
    235235*-------------------------------------------------------------------------------
    236236*/
    237237
     238// A conflict between BP's body.no-js being removed only
     239// when the DOM is fully loaded & Twentysixteens hardcoded
     240// class on the html el & scripting in head at start of DOM
     241// causes mobile menu to stay opened too long.
     242// This ruleset hides menu if html.js (2016 JS has kicked in) but BP still
     243// rendering body.no-js before it's own JS removal kicks in.
     244// See trac ticket https://buddypress.trac.wordpress.org/ticket/7314
    238245
     246/**
     247* Corrective measure to deal with 2016 / BP handling of no-js
     248* classes - hide menu if BP still showing 'no-js'.
     249*/
    239250
     251@media screen and (max-width: 905px) {
     252
     253        html.js {
     254
     255                body.no-js {
     256
     257                        .site-header-menu {
     258                                display: none;
     259                        }
     260                }
     261        }
     262}
     263
    240264.buddypress {
    241265        // Remove any empty clearing elements, CSS provides clearing.
    242266        div.clear {display: none;}