Jump to content

Small screen menu not collapsing after click


nickc

Recommended Posts

Hello

 

I have built a one page website based on this template - http://www.w3schools.com/w3css/tryw3css_templates_website.htm

 

When viewed a smartphone screen the smallscreen navbar does not collapse after clicking a link and I therefore have to toggle the nav button manually.

 

I have created a workaround in order for the site to function as desired which can be found here - http://www.davidclarkandco.co.uk/index.html - but ideally I simply wish to use a single page and have the menu to collapse on click.

 

Thanks in advance for advice/help.

Link to comment
Share on other sites

Have you learned Javascript yet?

 

I have done some in the past and am aware of onclick events. If it can be done that way I'm happy to go with that. Just wasn't sure it was possible with W3.CSS as all the solutions I've found so far relate to bootstrap.

Link to comment
Share on other sites

Unlike Bootstrap, W3.CSS does not use any Javascript. If you look at the example page, what they've done is add Javascript to it. If you look at the bottom of the page, they've defined a function called openNav().

 

What you can do is go through each of the links and have them call openNav when clicked.

// Get a list of links
var links = document.querySelectorAll(".w3-navbar .w3-hide-small > a");

// Loop through the links to add the event listener
for(var i = 0; i < links.length; i++) {
  links[i].addEventListener("click", openNav, false);
}
Link to comment
Share on other sites

 

Unlike Bootstrap, W3.CSS does not use any Javascript. If you look at the example page, what they've done is add Javascript to it. If you look at the bottom of the page, they've defined a function called openNav().

 

What you can do is go through each of the links and have them call openNav when clicked.

// Get a list of links
var links = document.querySelectorAll(".w3-navbar .w3-hide-small > a");

// Loop through the links to add the event listener
for(var i = 0; i < links.length; i++) {
  links[i].addEventListener("click", openNav, false);
}

 

Fabulous. It works. Thanks so much!

Link to comment
Share on other sites

I think the point they're trying to get with these examples is that if you want something done with Javascript you have to write the Javascript yourself, the framework is a purely CSS framework intended just for presentation and not behavior.

Link to comment
Share on other sites

To claim its just css is a stupid statement, as mentioned most of its examples would not work without them adding JavaScript specifically for there w3css examples.

 

Take for instance, there many templates, why include javscript for templates, you have html with w3css classes, and css, just add your own JavaScript so it will work as w3css intended. Going by that statement every website is just a css framework for presentational, with just our own added JavaScript to make it behavioral.

Link to comment
Share on other sites

The difference between W3.CSS and Bootstrap is that Bootstrap comes with a bootstrap.js file while W3.CSS has no associated JS file in the framework. You just include w3.css in your website and everything else has to be added manually on top.

Link to comment
Share on other sites

  • 9 months later...

Hi, I have the same problem as Nickc and I have tried to solve it as Ingolme has offered. Unfortunately I failed. Please tell me where the mistake is. I would be very thankful for help. Below is the script

<div class="w3-padding-0 w3-large w3-text-grey" style="font-weight:bold">
    <a onclick="myAccFunc()" class="w3-button w3-block w3-white w3-left-align"> Наши проекты <i class="fa fa-caret-down"></i></a>
    <div id="demoAcc" href="javascript:void(0)" class="w3-bar-block w3-hide w3-padding-large w3-medium">
      <a href="http://тетрадкадружбы.заботадетям.рф" class="w3-bar-item w3-button w3-light-grey"><i class="fa fa-caret-right w3-margin-right"></i>Тетрадка дружбы</a>
      <a href="http://стритбол.заботадетям.рф" class="w3-bar-item w3-button w3-light-grey"><i class="fa fa-caret-right w3-margin-right"></i>Стритбол</a>
      <a href="http://даридобро.заботадетям.рф" class="w3-bar-item w3-button w3-light-grey"><i class="fa fa-caret-right w3-margin-right"></i>Дари добро</a>
      <a href="http://домребенка.заботадетям.рф" class="w3-bar-item w3-button w3-light-grey"><i class="fa fa-caret-right w3-margin-right"></i>Дом ребенка</a>
      <a href="http://времяцветов.заботадетям.рф" class="w3-bar-item w3-button w3-light-grey"><i class="fa fa-caret-right w3-margin-right"></i>Время цветов</a>
    </div>
    <a href="http://помочь.заботадетям.рф" class="w3-bar-item w3-button">Как помочь</a>
    <div id="navDemo" href="javascript:void(0)">
    <a onclick="openNav()" href="#about" class="w3-bar-item w3-button">О нашем фонде</a>
    <a onclick="openNav()" href="#mission" class="w3-bar-item w3-button">Миссия фонда</a>
    <a onclick="openNav()" href="#docs" class="w3-bar-item w3-button">Учредительные документы</a>   
    <a onclick="openNav()" href="#team" class="w3-bar-item w3-button">Координаторы</a>
    <a onclick="openNav()" href="#rewards" class="w3-bar-item w3-button">Благодарности</a>
    <a onclick="openNav()" href="#contact" class="w3-bar-item w3-button">Контакты</a>
    </div>
  </div>

 

<script>
// Accordion
function myAccFunc() {
    var x = document.getElementById("demoAcc");
    if (x.className.indexOf("w3-show") == -1) {
        x.className += " w3-show";
    } else {
        x.className = x.className.replace(" w3-show", "");
    }
}
// Script to open and close sidebar
function w3_open() {
    document.getElementById("mySidebar").style.display = "block";
    document.getElementById("myOverlay").style.display = "block";
}
 
function w3_close() {
    document.getElementById("mySidebar").style.display = "none";
    document.getElementById("myOverlay").style.display = "none";
}
// Used to toggle the menu on smaller screens when clicking on the menu button
function openNav() {
    var x = document.getElementById("navDemo");
    if (x.className.indexOf("w3-show") == -1) {
        x.className += " w3-show";
    } else {
        x.className = x.className.replace(" w3-show", "");
    }
}
</script>
Link to comment
Share on other sites

(1) Div element cannot use href attribute, they are only for anchor 'a' elements.

(2)  var x = document.getElementById("navDemo");  assign a element with id ref 'navDemo' ie <div id="navDemo" href="javascript:void(0)"> (with invalid href attribute) to variable 'x'

then it checks this specific element classes?

if (x.className.indexOf("w3-show") == -1) {
        x.className += " w3-show";
    } else {
        x.className = x.className.replace(" w3-show", "");
    }

but! can you see the problem here? there are no classes for this element.

Link to comment
Share on other sites

On ‎20‎.‎09‎.‎2016 at 11:38 PM, nickc said:

Thank you, I have corrected the Div element and replaced href attribute to the <a> line above. What about onclick attributes? Shall I also replace script to the one below? :

// Get a list of links

var links = document.querySelectorAll(".w3-navbar .w3-hide-small > a");

// Loop through the links to add the event listener

for(var i = 0; i < links.length; i++) { links[i].addEventListener("click", openNav, false);

}

 

 

Edited by Roman Heidemann
Link to comment
Share on other sites

  • 2 weeks later...

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
×
×
  • Create New...