mvelosop / AspNetCore2CoreUI

ASP.NET MVC Core 2 project using CoreUI.io template
MIT License
88 stars 39 forks source link

Header and sidebar broken on IE #9

Open ASKemp opened 6 years ago

ASKemp commented 6 years ago

This seems a general issue with CoreUI, the header and sidebar do not play nicely with Internet Explorer 10+ at small viewport sizes. The header drops the logo down and the navigation sidebar doesn't close appropriately with respect to the main body content.

Any ideas, has anyone fixed this? Works fine on Chrome.

xidedix commented 6 years ago

Misplaced navbar-brand IE issue has been fixed with @coreui/coreui v2.0.10 Sidebar issue's not fixed yet

mvelosop commented 6 years ago

@ASKemp, thanks for the heads up! I hadn't gotten into this yet.

Thanks for the info @xidedix!

I'm planning to update the repo to the latest CoreUI soon.

In the meantime, I think I found the culprit here:

Couldn't find a centering fix that worked for all browsers, but at least, this seems to fix the vertical placement, which might be good enough, just add this to site.css:

@media (max-width: 991.98px) {
    .app-header .navbar-brand {
        position: static;
    }
}

Hope this helps

ASKemp commented 6 years ago

Thanks for the reply. xidedix solved it with a small amendment, a TOP I think iirc.

Now... onto that closing sidebar/non closing gap issue? ;)

xidedix commented 6 years ago

working on it

xidedix commented 6 years ago

@coreui/coreui v2.0.11 fixes sidebar gap issue at smaller viewport sizes

ASKemp commented 6 years ago

Fantastic, will check it out.