Nav bar doesn't work on mobile

Question

Nav bar doesn't work on mobile

KirstieJane opened this issue 8 years ago · 14 comments

KirstieJane commented 8 years ago

So saaaad!

I can see the nav-bar but when I click on it it doesn't show me anything.

If anyone knows why please comment!

Answer 1 · 2016-10-13T20:36:15.000Z

The JavaScript isn't loaded.

Open the developer tools of your web browser and look at the Network pane. For Firefox, it is explained on MDN.

Answer 2 · 2016-10-13T20:36:38.000Z

I checked with Web IDE as well (you need to download an ISO and then hit the browser app on the left side to get access to Developer Tools there).

The image is too big. Related to #98?

Answer 3 · 2016-10-13T20:38:22.000Z

Concerning WebIDE.

My battery runs dry in a few minutes.

Answer 4 · 2016-10-14T01:10:39.000Z

i am taking a look at this as well. more soon.

Answer 5 · 2016-10-17T15:55:23.000Z

@KirstieJane Did you import the jQuery before importing bootstrap, out of order it won't work!
For ex:
`<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.3/jquery.min.js"></script>

Answer 6 · 2016-10-18T08:01:18.000Z

Hi @geminiimatt - thank you for taking this on - I'm going to assign you to this issue! [EDIT: wait - no I'm not because you aren't a contributor. Ok, you're virtually assigned 😉] Very likely @amitkumarj441 and @Ryuno-Ki's comments will be helpful!

@Ryuno-Ki - the fact that the image is too big is a different issue!! It needs to be fixed too. If you have time and feel like it you could open a new issue to describe the problem!

Thank you all for your help 😄

Answer 7 · 2016-10-18T14:11:51.000Z

Hey @KirstieJane ,

I hope you checked my comment on this issue, well I'll soon fix this issue by adding the navbar switching part in js file. Let me assign this issue, I'm going to fix this bug.

Cheers,
Amit Kumar Jaiswal

Answer 8 · 2016-10-18T21:26:07.000Z

Hey, @KirstieJane,

sadly I'm overloaded with work at the moment. But I'm still watching this repo and chime in, whenever I can help.

Answer 9 · 2016-10-20T11:32:34.000Z

Hey @KirstieJane,

It looks like none of the pages source bootstrap js, and only the search_database page has jquery. I've added a cdn link to all the pages, and the dropdown now works, although the first couple of links sit next to the logo. Would it be worth having this as a navbar-brand class and moving it out of the collapse, so that it is always visible?

E I also did a bit of html-linting while in the files, there were a few instances of improperly closed tags, mis-matching tags

Answer 10 · 2016-10-20T17:31:36.000Z

CDNs are fine. Until they got down (b/c of a DDoS, for example).

We should look after adding some decent styles in a <noscript> to keep the site usable.

Answer 11 · 2016-10-24T17:07:30.000Z

Thanks @stain88! I've merged your changes! And now the navbar works! Great!

BUT - I agree that it shouldn't have the logo on the inside of the collapse. Any pull requests on that front are very welcome 😄

Answer 12 · 2016-10-24T17:26:53.000Z

Cool, I'll work on that tomorrow

Answer 13 · 2016-10-24T17:47:06.000Z

@Ryuno-Ki I don't think DDoS happen often enough for a no script version to be of high priority. But if you or anyone else wanted to work on it then that's fine.

Answer 14 · 2016-10-24T17:53:15.000Z

Problem is, I was silently laughing at that too, but then that whole DynDNS DDOS happened and took down Twitter, Github, Reddit, and others, so I'll put it in locally with the nav bar changes.