search

m0r13 / mclogalyzer

Minecraft Server Log Analyzer
GNU General Public License v3.0
44 stars 20 forks source link

Upgrade to Bootstrap 3 #11

Closed johnwoltman closed 9 years ago

johnwoltman commented 9 years ago

I'd like the mclogalyzer template to be upgraded to Bootstrap 3. With BS3 it's easier to make mobile sites. I upgraded my fork of this repo, and will make a PR based on it. But a problem is that the the navigation that uses javascript doesn't work well on mobile devices (particularly iPhones/iPads). In my fork I removed the javascript navigation and use old-style anchor tags with links based on IDs.

Is that acceptable?

m0r13 commented 9 years ago

I already upgraded to Bootstrap 3 yesterday. If the navigation works the same way with that old-style thing, that should be okay.

johnwoltman commented 9 years ago

I hadn't noticed because I was working on my own branch. I merged your changes back into my fork. Your navigation stuff works really well.