Rendering elements (which can change numbers) using jQuery The 2019 Stack Overflow Developer...

How are presidential pardons supposed to be used?

Format single node in tikzcd

Are my PIs rude or am I just being too sensitive?

What LEGO pieces have "real-world" functionality?

Reference for the teaching of not-self

Is every episode of "Where are my Pants?" identical?

What is special about square numbers here?

Python - Fishing Simulator

Relations between two reciprocal partial derivatives?

Wolves and sheep

Did the UK government pay "millions and millions of dollars" to try to snag Julian Assange?

Why can't devices on different VLANs, but on the same subnet, communicate?

Segmentation fault output is suppressed when piping stdin into a function. Why?

"... to apply for a visa" or "... and applied for a visa"?

Derivation tree not rendering

Why did all the guest students take carriages to the Yule Ball?

How should I replace vector<uint8_t>::const_iterator in an API?

I could not break this equation. Please help me

How is simplicity better than precision and clarity in prose?

In horse breeding, what is the female equivalent of putting a horse out "to stud"?

Is it ok to offer lower paid work as a trial period before negotiating for a full-time job?

Is this wall load bearing? Blueprints and photos attached

Was credit for the black hole image misattributed?

Can smartphones with the same camera sensor have different image quality?



Rendering elements (which can change numbers) using jQuery



The 2019 Stack Overflow Developer Survey Results Are In
Announcing the arrival of Valued Associate #679: Cesar Manara
Planned maintenance scheduled April 17/18, 2019 at 00:00UTC (8:00pm US/Eastern)jQuery highlightNavigation pluginCustom slide show scriptImplementation quality of my afterthought jQuery drop-down menu?Append table cells to select boxes in indexed orderInterupt redirect to catch potential newsletter signupAdd multiple data attributes to elements using jQueryWeb-based test settings manager appGenerate table of contents form HTMLJS micro-view libraryAdd Colour Tags on Click - Version 2





.everyoneloves__top-leaderboard:empty,.everyoneloves__mid-leaderboard:empty,.everyoneloves__bot-mid-leaderboard:empty{ margin-bottom:0;
}







1












$begingroup$


Currently when the user selects some tags in my app I generate them in a function, like this:



// Show tags that are currently used in search
var htmlNodes = '<ul class="tags">';

// If no pinned nodes, reset not found nodes
if (graphFactory.existPinnedNodes() == false) {
graphFactory.updateNotFoundNodes();
}

// Add not found nodes as tags (gray)
for (let x = 0; x < graphFactory.getNotFoundNodes().length; x++) {
htmlNodes = htmlNodes + '<li><a class="notfound" href="#">' + graphFactory.getNotFoundNodes()[x] + '</a></li>';
}

// Add pinned nodes as tags (blue)
for (let q = 0; q < graphFactory.getPinnedNodes().length; q++) {
htmlNodes = htmlNodes + '<li><a href="#">' + graphFactory.getPinnedNodes()[q] + '</a></li>';
}

htmlNodes = htmlNodes + '<div class="tagsend"></div></ul>';
$('#pinnednodeslist').html(htmlNodes);


Basically I create a <ul> of class tags, add some li elements with a specific tag each, and then render it into the #pinnednodeslist element.



As you can see, the number of <li> elements might change, depending on the number of getPinnedNodes or getNotFoundNodes, so I wonder if I have to reconstruct this html every time or if there is a more efficient way to do that...



Maybe a better way is that I don't have to mix so much HTML with my code?



For instance, I was thinking to leave all the elements there and just make them invisible, simply adding a class to show them and the html content to display within... But maybe you know of some other better way to do that?










share|improve this question









New contributor




Dmitry Paranyushkin is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.







$endgroup$



















    1












    $begingroup$


    Currently when the user selects some tags in my app I generate them in a function, like this:



    // Show tags that are currently used in search
    var htmlNodes = '<ul class="tags">';

    // If no pinned nodes, reset not found nodes
    if (graphFactory.existPinnedNodes() == false) {
    graphFactory.updateNotFoundNodes();
    }

    // Add not found nodes as tags (gray)
    for (let x = 0; x < graphFactory.getNotFoundNodes().length; x++) {
    htmlNodes = htmlNodes + '<li><a class="notfound" href="#">' + graphFactory.getNotFoundNodes()[x] + '</a></li>';
    }

    // Add pinned nodes as tags (blue)
    for (let q = 0; q < graphFactory.getPinnedNodes().length; q++) {
    htmlNodes = htmlNodes + '<li><a href="#">' + graphFactory.getPinnedNodes()[q] + '</a></li>';
    }

    htmlNodes = htmlNodes + '<div class="tagsend"></div></ul>';
    $('#pinnednodeslist').html(htmlNodes);


    Basically I create a <ul> of class tags, add some li elements with a specific tag each, and then render it into the #pinnednodeslist element.



    As you can see, the number of <li> elements might change, depending on the number of getPinnedNodes or getNotFoundNodes, so I wonder if I have to reconstruct this html every time or if there is a more efficient way to do that...



    Maybe a better way is that I don't have to mix so much HTML with my code?



    For instance, I was thinking to leave all the elements there and just make them invisible, simply adding a class to show them and the html content to display within... But maybe you know of some other better way to do that?










    share|improve this question









    New contributor




    Dmitry Paranyushkin is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
    Check out our Code of Conduct.







    $endgroup$















      1












      1








      1





      $begingroup$


      Currently when the user selects some tags in my app I generate them in a function, like this:



      // Show tags that are currently used in search
      var htmlNodes = '<ul class="tags">';

      // If no pinned nodes, reset not found nodes
      if (graphFactory.existPinnedNodes() == false) {
      graphFactory.updateNotFoundNodes();
      }

      // Add not found nodes as tags (gray)
      for (let x = 0; x < graphFactory.getNotFoundNodes().length; x++) {
      htmlNodes = htmlNodes + '<li><a class="notfound" href="#">' + graphFactory.getNotFoundNodes()[x] + '</a></li>';
      }

      // Add pinned nodes as tags (blue)
      for (let q = 0; q < graphFactory.getPinnedNodes().length; q++) {
      htmlNodes = htmlNodes + '<li><a href="#">' + graphFactory.getPinnedNodes()[q] + '</a></li>';
      }

      htmlNodes = htmlNodes + '<div class="tagsend"></div></ul>';
      $('#pinnednodeslist').html(htmlNodes);


      Basically I create a <ul> of class tags, add some li elements with a specific tag each, and then render it into the #pinnednodeslist element.



      As you can see, the number of <li> elements might change, depending on the number of getPinnedNodes or getNotFoundNodes, so I wonder if I have to reconstruct this html every time or if there is a more efficient way to do that...



      Maybe a better way is that I don't have to mix so much HTML with my code?



      For instance, I was thinking to leave all the elements there and just make them invisible, simply adding a class to show them and the html content to display within... But maybe you know of some other better way to do that?










      share|improve this question









      New contributor




      Dmitry Paranyushkin is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.







      $endgroup$




      Currently when the user selects some tags in my app I generate them in a function, like this:



      // Show tags that are currently used in search
      var htmlNodes = '<ul class="tags">';

      // If no pinned nodes, reset not found nodes
      if (graphFactory.existPinnedNodes() == false) {
      graphFactory.updateNotFoundNodes();
      }

      // Add not found nodes as tags (gray)
      for (let x = 0; x < graphFactory.getNotFoundNodes().length; x++) {
      htmlNodes = htmlNodes + '<li><a class="notfound" href="#">' + graphFactory.getNotFoundNodes()[x] + '</a></li>';
      }

      // Add pinned nodes as tags (blue)
      for (let q = 0; q < graphFactory.getPinnedNodes().length; q++) {
      htmlNodes = htmlNodes + '<li><a href="#">' + graphFactory.getPinnedNodes()[q] + '</a></li>';
      }

      htmlNodes = htmlNodes + '<div class="tagsend"></div></ul>';
      $('#pinnednodeslist').html(htmlNodes);


      Basically I create a <ul> of class tags, add some li elements with a specific tag each, and then render it into the #pinnednodeslist element.



      As you can see, the number of <li> elements might change, depending on the number of getPinnedNodes or getNotFoundNodes, so I wonder if I have to reconstruct this html every time or if there is a more efficient way to do that...



      Maybe a better way is that I don't have to mix so much HTML with my code?



      For instance, I was thinking to leave all the elements there and just make them invisible, simply adding a class to show them and the html content to display within... But maybe you know of some other better way to do that?







      javascript jquery interface






      share|improve this question









      New contributor




      Dmitry Paranyushkin is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.











      share|improve this question









      New contributor




      Dmitry Paranyushkin is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.









      share|improve this question




      share|improve this question








      edited 6 mins ago









      Jamal

      30.6k11121227




      30.6k11121227






      New contributor




      Dmitry Paranyushkin is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.









      asked 20 hours ago









      Dmitry ParanyushkinDmitry Paranyushkin

      1062




      1062




      New contributor




      Dmitry Paranyushkin is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.





      New contributor





      Dmitry Paranyushkin is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.






      Dmitry Paranyushkin is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.






















          0






          active

          oldest

          votes












          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: "196"
          };
          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: false,
          noModals: true,
          showLowRepImageUploadWarning: true,
          reputationToPostImages: null,
          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
          });


          }
          });






          Dmitry Paranyushkin is a new contributor. Be nice, and check out our Code of Conduct.










          draft saved

          draft discarded


















          StackExchange.ready(
          function () {
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f217377%2frendering-elements-which-can-change-numbers-using-jquery%23new-answer', 'question_page');
          }
          );

          Post as a guest















          Required, but never shown

























          0






          active

          oldest

          votes








          0






          active

          oldest

          votes









          active

          oldest

          votes






          active

          oldest

          votes








          Dmitry Paranyushkin is a new contributor. Be nice, and check out our Code of Conduct.










          draft saved

          draft discarded


















          Dmitry Paranyushkin is a new contributor. Be nice, and check out our Code of Conduct.













          Dmitry Paranyushkin is a new contributor. Be nice, and check out our Code of Conduct.












          Dmitry Paranyushkin is a new contributor. Be nice, and check out our Code of Conduct.
















          Thanks for contributing an answer to Code Review Stack Exchange!


          • 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.


          Use MathJax to format equations. MathJax reference.


          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%2fcodereview.stackexchange.com%2fquestions%2f217377%2frendering-elements-which-can-change-numbers-using-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







          Popular posts from this blog

          Fairchild Swearingen Metro Inhaltsverzeichnis Geschichte | Innenausstattung | Nutzung | Zwischenfälle...

          Pilgersdorf Inhaltsverzeichnis Geografie | Geschichte | Bevölkerungsentwicklung | Politik | Kultur...

          Marineschifffahrtleitung Inhaltsverzeichnis Geschichte | Heutige Organisation der NATO | Nationale und...