Replacing GitHub usernames with links The Next CEO of Stack OverflowSimplifying up/down vote codeIs there a “code less” way to construct this JS loop?Is there a better way of writing this JavaScript, or is it idiomatic?Is there anything can be improved in this JavaScript module?Is there a more succinct way of writing this simple JavaScript loop?Interupt redirect to catch potential newsletter signupSlim down Javascript MomentJS FunctionSimple jQuery plugin that excerpt textIs there a better way to sort this items?Is there a better way to create constants in javascript?

That's an odd coin - I wonder why

How do I secure a TV wall mount?

Can this transistor (2n2222) take 6V on emitter-base? Am I reading datasheet incorrectly?

Incomplete cube

Another proof that dividing by 0 does not exist -- is it right?

Ising model simulation

How seriously should I take size and weight limits of hand luggage?

The sum of any ten consecutive numbers from a fibonacci sequence is divisible by 11

What day is it again?

Can I cast Thunderwave and be at the center of its bottom face, but not be affected by it?

Early programmable calculators with RS-232

Create custom note boxes

logical reads on global temp table, but not on session-level temp table

My ex-girlfriend uses my Apple ID to login to her iPad, do I have to give her my Apple ID password to reset it?

How to show a landlord what we have in savings?

Is this a new Fibonacci Identity?

How to implement Comparable so it is consistent with identity-equality

Shortening a title without changing its meaning

How can a day be of 24 hours?

How to unfasten electrical subpanel attached with ramset

How can I replace x-axis labels with pre-determined symbols?

Prodigo = pro + ago?

Could you use a laser beam as a modulated carrier wave for radio signal?

Does the Idaho Potato Commission associate potato skins with healthy eating?



Replacing GitHub usernames with links



The Next CEO of Stack OverflowSimplifying up/down vote codeIs there a “code less” way to construct this JS loop?Is there a better way of writing this JavaScript, or is it idiomatic?Is there anything can be improved in this JavaScript module?Is there a more succinct way of writing this simple JavaScript loop?Interupt redirect to catch potential newsletter signupSlim down Javascript MomentJS FunctionSimple jQuery plugin that excerpt textIs there a better way to sort this items?Is there a better way to create constants in javascript?










2












$begingroup$


A user wrote an issue asking if inside the bio (or any arbitrary tag), any tagged accounts can be automatically linked like on Github.



I gave them this function:



 <script>
let bio = "The coolest company to work for is @github! There is also @aws and @microsoft.";
let linkedBio = "";
let found = false;
let link = "";
for (let i = 0; i < bio.length; i++)
let currentChar = bio[i];

if (currentChar.startsWith("@"))
link += currentChar;
found = true;
continue // we don't need to look at other chars until next iterations


if (found)
if (/[^A-Za-z0-9]/.test(currentChar)) // we found the end of the link (you can add more special chars to this)
let tag = "<a href=https://github.com/" + link.substr(1) + ">" + link + "</a>"
linkedBio += tag + currentChar // we add the special char at the end of the tag so that it actually appears, otherwise it does not
link = "";
found = false;
else
link += currentChar;

else
linkedBio += currentChar;



if (link.length > 0) // means we need to add a link that is the last possible thing without anything after it
let tag = "<a href=https://github.com/" + link.substr(1) + ">" + link + "</a>"
linkedBio += tag


document.getElementById("bio").innerHTML = linkedBio
</script>


I was going to use string.split(" "), but then I realized that if I did that I would destroy the specific formatting of the text if I used array.join(" ") after all of the formatting changes.



Ex:
Original: "I work for the coolest companies: @aws, @microsoft, and @apple"



Joined: "I work for the coolest companies: @aws , @microsoft , and @apple"



The commas are messed up.



Is there a way to slim it down and/or simplify it? JS isn't my strongest language.










share|improve this question









New contributor




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







$endgroup$
















    2












    $begingroup$


    A user wrote an issue asking if inside the bio (or any arbitrary tag), any tagged accounts can be automatically linked like on Github.



    I gave them this function:



     <script>
    let bio = "The coolest company to work for is @github! There is also @aws and @microsoft.";
    let linkedBio = "";
    let found = false;
    let link = "";
    for (let i = 0; i < bio.length; i++)
    let currentChar = bio[i];

    if (currentChar.startsWith("@"))
    link += currentChar;
    found = true;
    continue // we don't need to look at other chars until next iterations


    if (found)
    if (/[^A-Za-z0-9]/.test(currentChar)) // we found the end of the link (you can add more special chars to this)
    let tag = "<a href=https://github.com/" + link.substr(1) + ">" + link + "</a>"
    linkedBio += tag + currentChar // we add the special char at the end of the tag so that it actually appears, otherwise it does not
    link = "";
    found = false;
    else
    link += currentChar;

    else
    linkedBio += currentChar;



    if (link.length > 0) // means we need to add a link that is the last possible thing without anything after it
    let tag = "<a href=https://github.com/" + link.substr(1) + ">" + link + "</a>"
    linkedBio += tag


    document.getElementById("bio").innerHTML = linkedBio
    </script>


    I was going to use string.split(" "), but then I realized that if I did that I would destroy the specific formatting of the text if I used array.join(" ") after all of the formatting changes.



    Ex:
    Original: "I work for the coolest companies: @aws, @microsoft, and @apple"



    Joined: "I work for the coolest companies: @aws , @microsoft , and @apple"



    The commas are messed up.



    Is there a way to slim it down and/or simplify it? JS isn't my strongest language.










    share|improve this question









    New contributor




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







    $endgroup$














      2












      2








      2





      $begingroup$


      A user wrote an issue asking if inside the bio (or any arbitrary tag), any tagged accounts can be automatically linked like on Github.



      I gave them this function:



       <script>
      let bio = "The coolest company to work for is @github! There is also @aws and @microsoft.";
      let linkedBio = "";
      let found = false;
      let link = "";
      for (let i = 0; i < bio.length; i++)
      let currentChar = bio[i];

      if (currentChar.startsWith("@"))
      link += currentChar;
      found = true;
      continue // we don't need to look at other chars until next iterations


      if (found)
      if (/[^A-Za-z0-9]/.test(currentChar)) // we found the end of the link (you can add more special chars to this)
      let tag = "<a href=https://github.com/" + link.substr(1) + ">" + link + "</a>"
      linkedBio += tag + currentChar // we add the special char at the end of the tag so that it actually appears, otherwise it does not
      link = "";
      found = false;
      else
      link += currentChar;

      else
      linkedBio += currentChar;



      if (link.length > 0) // means we need to add a link that is the last possible thing without anything after it
      let tag = "<a href=https://github.com/" + link.substr(1) + ">" + link + "</a>"
      linkedBio += tag


      document.getElementById("bio").innerHTML = linkedBio
      </script>


      I was going to use string.split(" "), but then I realized that if I did that I would destroy the specific formatting of the text if I used array.join(" ") after all of the formatting changes.



      Ex:
      Original: "I work for the coolest companies: @aws, @microsoft, and @apple"



      Joined: "I work for the coolest companies: @aws , @microsoft , and @apple"



      The commas are messed up.



      Is there a way to slim it down and/or simplify it? JS isn't my strongest language.










      share|improve this question









      New contributor




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







      $endgroup$




      A user wrote an issue asking if inside the bio (or any arbitrary tag), any tagged accounts can be automatically linked like on Github.



      I gave them this function:



       <script>
      let bio = "The coolest company to work for is @github! There is also @aws and @microsoft.";
      let linkedBio = "";
      let found = false;
      let link = "";
      for (let i = 0; i < bio.length; i++)
      let currentChar = bio[i];

      if (currentChar.startsWith("@"))
      link += currentChar;
      found = true;
      continue // we don't need to look at other chars until next iterations


      if (found)
      if (/[^A-Za-z0-9]/.test(currentChar)) // we found the end of the link (you can add more special chars to this)
      let tag = "<a href=https://github.com/" + link.substr(1) + ">" + link + "</a>"
      linkedBio += tag + currentChar // we add the special char at the end of the tag so that it actually appears, otherwise it does not
      link = "";
      found = false;
      else
      link += currentChar;

      else
      linkedBio += currentChar;



      if (link.length > 0) // means we need to add a link that is the last possible thing without anything after it
      let tag = "<a href=https://github.com/" + link.substr(1) + ">" + link + "</a>"
      linkedBio += tag


      document.getElementById("bio").innerHTML = linkedBio
      </script>


      I was going to use string.split(" "), but then I realized that if I did that I would destroy the specific formatting of the text if I used array.join(" ") after all of the formatting changes.



      Ex:
      Original: "I work for the coolest companies: @aws, @microsoft, and @apple"



      Joined: "I work for the coolest companies: @aws , @microsoft , and @apple"



      The commas are messed up.



      Is there a way to slim it down and/or simplify it? JS isn't my strongest language.







      javascript dom






      share|improve this question









      New contributor




      Landon 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




      Landon 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









      200_success

      131k17156420




      131k17156420






      New contributor




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









      asked 4 hours ago









      LandonLandon

      133




      133




      New contributor




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





      New contributor





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






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




















          1 Answer
          1






          active

          oldest

          votes


















          3












          $begingroup$

          The function you want is replace:



          function link_ats(text) 
          return text.replace( /@(w+)/g, '<a href=https://github.com/$1>@$1</a>' )


          document.getElementById("bio").innerHTML = link_ats("The coolest company to work for is @github! There is also @aws and @microsoft.", "bio");





          share|improve this answer









          $endgroup$













            Your Answer





            StackExchange.ifUsing("editor", function ()
            return StackExchange.using("mathjaxEditing", function ()
            StackExchange.MarkdownEditor.creationCallbacks.add(function (editor, postfix)
            StackExchange.mathjaxEditing.prepareWmdForMathJax(editor, postfix, [["\$", "\$"]]);
            );
            );
            , "mathjax-editing");

            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
            );



            );






            Landon 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%2f216685%2freplacing-github-usernames-with-links%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









            3












            $begingroup$

            The function you want is replace:



            function link_ats(text) 
            return text.replace( /@(w+)/g, '<a href=https://github.com/$1>@$1</a>' )


            document.getElementById("bio").innerHTML = link_ats("The coolest company to work for is @github! There is also @aws and @microsoft.", "bio");





            share|improve this answer









            $endgroup$

















              3












              $begingroup$

              The function you want is replace:



              function link_ats(text) 
              return text.replace( /@(w+)/g, '<a href=https://github.com/$1>@$1</a>' )


              document.getElementById("bio").innerHTML = link_ats("The coolest company to work for is @github! There is also @aws and @microsoft.", "bio");





              share|improve this answer









              $endgroup$















                3












                3








                3





                $begingroup$

                The function you want is replace:



                function link_ats(text) 
                return text.replace( /@(w+)/g, '<a href=https://github.com/$1>@$1</a>' )


                document.getElementById("bio").innerHTML = link_ats("The coolest company to work for is @github! There is also @aws and @microsoft.", "bio");





                share|improve this answer









                $endgroup$



                The function you want is replace:



                function link_ats(text) 
                return text.replace( /@(w+)/g, '<a href=https://github.com/$1>@$1</a>' )


                document.getElementById("bio").innerHTML = link_ats("The coolest company to work for is @github! There is also @aws and @microsoft.", "bio");






                share|improve this answer












                share|improve this answer



                share|improve this answer










                answered 3 hours ago









                Oh My GoodnessOh My Goodness

                2,069315




                2,069315




















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









                    draft saved

                    draft discarded


















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












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











                    Landon 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%2f216685%2freplacing-github-usernames-with-links%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

                    कुँवर स्रोत दिक्चालन सूची"कुँवर""राणा कुँवरके वंशावली"

                    Why is a white electrical wire connected to 2 black wires?How to wire a light fixture with 3 white wires in box?How should I wire a ceiling fan when there's only three wires in the box?Two white, two black, two ground, and red wire in ceiling box connected to switchWhy is there a white wire connected to multiple black wires in my light box?How to wire a light with two white wires and one black wireReplace light switch connected to a power outlet with dimmer - two black wires to one black and redHow to wire a light with multiple black/white/green wires from the ceiling?Ceiling box has 2 black and white wires but fan/ light only has 1 of eachWhy neutral wire connected to load wire?Switch with 2 black, 2 white, 2 ground and 1 red wire connected to ceiling light and a receptacle?

                    चैत्य भूमि चित्र दीर्घा सन्दर्भ बाहरी कडियाँ दिक्चालन सूची"Chaitya Bhoomi""Chaitya Bhoomi: Statue of Equality in India""Dadar Chaitya Bhoomi: Statue of Equality in India""Ambedkar memorial: Centre okays transfer of Indu Mill land"चैत्यभमि