search

StartBootstrap / startbootstrap-clean-blog

A clean Bootstrap blog theme created by Start Bootstrap
https://startbootstrap.com/theme/clean-blog/
MIT License
1.78k stars 1.85k forks source link

Show header on rollup does not work with Opera #22

Closed donchambers closed 3 years ago

donchambers commented 8 years ago

Opera Test platform:

Version: 12.18 Build: 1872 Platform: Win32 System: Windows 7

davidtmiller commented 8 years ago

I'll have to look into this, I've not tested this theme in Opera...

yarwelp commented 8 years ago

I have the same problem using Firefox 48.0 on Fedora 23 Linux.

pirtleshell commented 7 years ago

Are you both sure that <script> tag is properly included? I got a Hexo version of this theme and it was missing the theme's script. After adding it in, my once only-on-the-top navbar was there as I scrolled up. If you take a look at the source of the example, you'll see the last script tag is

<script src="js/clean-blog.min.js"></script>
yarwelp commented 7 years ago

Testing https://blackrockdigital.github.io/startbootstrap-clean-blog/ using FreeBSD 11.0 with a cleanly installed Firefox 49.0 and no extensions, it works. You might be right, @PirtleShell. When I tested using Firefox 48.0 on Fedora 23 Linux, I did not check to ensure that scripts were loaded. Will test again when I'm back at my desktop computer.

yarwelp commented 7 years ago

Worked fine in Firefox 49.0 on Fedora 23 Linux when I tested now. Don't know what is different now from the last time I tested, aside from that I have updated Firefox from 48 to 49 in the meantime.

davidtmiller commented 7 years ago

Still keeping an eye on this. With #49 I need to do some cross-browser testing to see what's going on here.

curtismcginity commented 6 years ago

@donchambers I think I found the issue, or at least a fix/workaround. In nav, I changed <div class="container"> back to

<div class="container-fluid">,

like it was in the bs3 version.

@davidtmiller forsee issues with this down the road?

davidtmiller commented 4 years ago

I guess I don't like the idea of forcing the .container-fluid for formatting reasons. I will keep this open for now and see if the upcoming Bootstrap 4.5 changes, and eventually Bootstrap 5, addresses this issue.