I'm trying to collapse and close the nav element with jQuery










0















I'm trying to collapse and close the nav element with jQuery. I'm using rails 5 with gem coffee-rails and 4.2.2 gem jquery-rails, 4.3.1 but the animate option doesn't respond.



$(document).ready(function() 
$('nav a.mobile_menu').on('click', function()
var currentNavHeight = $('nav').height();
if (currentNavHeight < 6)
var newNavHeight = $('nav > ul').height() + 15;
$('nav').animate(
'height': newNavHeight + 'px'
, 750);
else
$('nav').animate(
'height': '0px'
, 750);

);
);


nav 
padding: 50px 15px 20px 15px;
background-color: #4b0a0c;
height: 0px;
overflow: hidden;










share|improve this question




























    0















    I'm trying to collapse and close the nav element with jQuery. I'm using rails 5 with gem coffee-rails and 4.2.2 gem jquery-rails, 4.3.1 but the animate option doesn't respond.



    $(document).ready(function() 
    $('nav a.mobile_menu').on('click', function()
    var currentNavHeight = $('nav').height();
    if (currentNavHeight < 6)
    var newNavHeight = $('nav > ul').height() + 15;
    $('nav').animate(
    'height': newNavHeight + 'px'
    , 750);
    else
    $('nav').animate(
    'height': '0px'
    , 750);

    );
    );


    nav 
    padding: 50px 15px 20px 15px;
    background-color: #4b0a0c;
    height: 0px;
    overflow: hidden;










    share|improve this question


























      0












      0








      0








      I'm trying to collapse and close the nav element with jQuery. I'm using rails 5 with gem coffee-rails and 4.2.2 gem jquery-rails, 4.3.1 but the animate option doesn't respond.



      $(document).ready(function() 
      $('nav a.mobile_menu').on('click', function()
      var currentNavHeight = $('nav').height();
      if (currentNavHeight < 6)
      var newNavHeight = $('nav > ul').height() + 15;
      $('nav').animate(
      'height': newNavHeight + 'px'
      , 750);
      else
      $('nav').animate(
      'height': '0px'
      , 750);

      );
      );


      nav 
      padding: 50px 15px 20px 15px;
      background-color: #4b0a0c;
      height: 0px;
      overflow: hidden;










      share|improve this question
















      I'm trying to collapse and close the nav element with jQuery. I'm using rails 5 with gem coffee-rails and 4.2.2 gem jquery-rails, 4.3.1 but the animate option doesn't respond.



      $(document).ready(function() 
      $('nav a.mobile_menu').on('click', function()
      var currentNavHeight = $('nav').height();
      if (currentNavHeight < 6)
      var newNavHeight = $('nav > ul').height() + 15;
      $('nav').animate(
      'height': newNavHeight + 'px'
      , 750);
      else
      $('nav').animate(
      'height': '0px'
      , 750);

      );
      );


      nav 
      padding: 50px 15px 20px 15px;
      background-color: #4b0a0c;
      height: 0px;
      overflow: hidden;







      jquery css ruby-on-rails






      share|improve this question















      share|improve this question













      share|improve this question




      share|improve this question








      edited Nov 13 '18 at 16:08









      Rory McCrossan

      243k29210246




      243k29210246










      asked Nov 13 '18 at 16:06









      Zekarias ZicoZekarias Zico

      62




      62






















          1 Answer
          1






          active

          oldest

          votes


















          0














          Any reason you're not using jQuery's toggle() method?



          https://www.w3schools.com/jquery/tryit.asp?filename=tryjquery_eff_toggle



          https://api.jquery.com/toggle/






          share|improve this answer




















          • 1





            Please don't use W3Schools as a reference. Their guides are often outdated and sometimes just plain wrong. When dealing with jQuery, use their reference: api.jquery.com. Also the OP probably isn't using toggle() because it doesn't animate the element as their logic attempts. slideToggle() would be more appropriate

            – Rory McCrossan
            Nov 13 '18 at 16:11











          Your Answer






          StackExchange.ifUsing("editor", function ()
          StackExchange.using("externalEditor", function ()
          StackExchange.using("snippets", function ()
          StackExchange.snippets.init();
          );
          );
          , "code-snippets");

          StackExchange.ready(function()
          var channelOptions =
          tags: "".split(" "),
          id: "1"
          ;
          initTagRenderer("".split(" "), "".split(" "), channelOptions);

          StackExchange.using("externalEditor", function()
          // Have to fire editor after snippets, if snippets enabled
          if (StackExchange.settings.snippets.snippetsEnabled)
          StackExchange.using("snippets", function()
          createEditor();
          );

          else
          createEditor();

          );

          function createEditor()
          StackExchange.prepareEditor(
          heartbeatType: 'answer',
          autoActivateHeartbeat: false,
          convertImagesToLinks: true,
          noModals: true,
          showLowRepImageUploadWarning: true,
          reputationToPostImages: 10,
          bindNavPrevention: true,
          postfix: "",
          imageUploader:
          brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
          contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
          allowUrls: true
          ,
          onDemand: true,
          discardSelector: ".discard-answer"
          ,immediatelyShowMarkdownHelp:true
          );



          );













          draft saved

          draft discarded


















          StackExchange.ready(
          function ()
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53285000%2fim-trying-to-collapse-and-close-the-nav-element-with-jquery%23new-answer', 'question_page');

          );

          Post as a guest















          Required, but never shown

























          1 Answer
          1






          active

          oldest

          votes








          1 Answer
          1






          active

          oldest

          votes









          active

          oldest

          votes






          active

          oldest

          votes









          0














          Any reason you're not using jQuery's toggle() method?



          https://www.w3schools.com/jquery/tryit.asp?filename=tryjquery_eff_toggle



          https://api.jquery.com/toggle/






          share|improve this answer




















          • 1





            Please don't use W3Schools as a reference. Their guides are often outdated and sometimes just plain wrong. When dealing with jQuery, use their reference: api.jquery.com. Also the OP probably isn't using toggle() because it doesn't animate the element as their logic attempts. slideToggle() would be more appropriate

            – Rory McCrossan
            Nov 13 '18 at 16:11
















          0














          Any reason you're not using jQuery's toggle() method?



          https://www.w3schools.com/jquery/tryit.asp?filename=tryjquery_eff_toggle



          https://api.jquery.com/toggle/






          share|improve this answer




















          • 1





            Please don't use W3Schools as a reference. Their guides are often outdated and sometimes just plain wrong. When dealing with jQuery, use their reference: api.jquery.com. Also the OP probably isn't using toggle() because it doesn't animate the element as their logic attempts. slideToggle() would be more appropriate

            – Rory McCrossan
            Nov 13 '18 at 16:11














          0












          0








          0







          Any reason you're not using jQuery's toggle() method?



          https://www.w3schools.com/jquery/tryit.asp?filename=tryjquery_eff_toggle



          https://api.jquery.com/toggle/






          share|improve this answer















          Any reason you're not using jQuery's toggle() method?



          https://www.w3schools.com/jquery/tryit.asp?filename=tryjquery_eff_toggle



          https://api.jquery.com/toggle/







          share|improve this answer














          share|improve this answer



          share|improve this answer








          edited Nov 13 '18 at 16:16

























          answered Nov 13 '18 at 16:10









          brandongatlinbrandongatlin

          64




          64







          • 1





            Please don't use W3Schools as a reference. Their guides are often outdated and sometimes just plain wrong. When dealing with jQuery, use their reference: api.jquery.com. Also the OP probably isn't using toggle() because it doesn't animate the element as their logic attempts. slideToggle() would be more appropriate

            – Rory McCrossan
            Nov 13 '18 at 16:11













          • 1





            Please don't use W3Schools as a reference. Their guides are often outdated and sometimes just plain wrong. When dealing with jQuery, use their reference: api.jquery.com. Also the OP probably isn't using toggle() because it doesn't animate the element as their logic attempts. slideToggle() would be more appropriate

            – Rory McCrossan
            Nov 13 '18 at 16:11








          1




          1





          Please don't use W3Schools as a reference. Their guides are often outdated and sometimes just plain wrong. When dealing with jQuery, use their reference: api.jquery.com. Also the OP probably isn't using toggle() because it doesn't animate the element as their logic attempts. slideToggle() would be more appropriate

          – Rory McCrossan
          Nov 13 '18 at 16:11






          Please don't use W3Schools as a reference. Their guides are often outdated and sometimes just plain wrong. When dealing with jQuery, use their reference: api.jquery.com. Also the OP probably isn't using toggle() because it doesn't animate the element as their logic attempts. slideToggle() would be more appropriate

          – Rory McCrossan
          Nov 13 '18 at 16:11


















          draft saved

          draft discarded
















































          Thanks for contributing an answer to Stack Overflow!


          • Please be sure to answer the question. Provide details and share your research!

          But avoid


          • Asking for help, clarification, or responding to other answers.

          • Making statements based on opinion; back them up with references or personal experience.

          To learn more, see our tips on writing great answers.




          draft saved


          draft discarded














          StackExchange.ready(
          function ()
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53285000%2fim-trying-to-collapse-and-close-the-nav-element-with-jquery%23new-answer', 'question_page');

          );

          Post as a guest















          Required, but never shown





















































          Required, but never shown














          Required, but never shown












          Required, but never shown







          Required, but never shown

































          Required, but never shown














          Required, but never shown












          Required, but never shown







          Required, but never shown







          這個網誌中的熱門文章

          Barbados

          How to read a connectionString WITH PROVIDER in .NET Core?

          Node.js Script on GitHub Pages or Amazon S3