Change section opacity onclick?












0














I'm trying to change the opacity of a with javascript when a div is clicked.



Here is my html:



<section id="section2">
<img onclick="one()" class=img src="1.png" />
</section>
<section id="6-1">
Hello World
</section>


And my javascript:



function one() {
document.getElementById('6-1').style.opacity = '1';
document.getElementById('section2').style.opacity = '0';
}


For some reason, its not working



Thanks!










share|improve this question


















  • 1




    What browser are you testing in?
    – Likwid_T
    Dec 8 '12 at 22:33






  • 1




    And what does your css look like?
    – Likwid_T
    Dec 8 '12 at 22:34






  • 3




    ID's starting with numbers are invalid, other than that it works just fine -> FIDDLE
    – adeneo
    Dec 8 '12 at 22:46












  • im using safari
    – Thomas Lai
    Dec 8 '12 at 22:52










  • When using opacity, you should try to look for cross-browser compatibility. So, IE 9+ and most modern browsers use opacity=1, IE8- uses filter='alpha(opacity=100)', others use -moz-opacity=1 and -khtml-opacity=1.
    – Marvin
    Dec 8 '12 at 23:38
















0














I'm trying to change the opacity of a with javascript when a div is clicked.



Here is my html:



<section id="section2">
<img onclick="one()" class=img src="1.png" />
</section>
<section id="6-1">
Hello World
</section>


And my javascript:



function one() {
document.getElementById('6-1').style.opacity = '1';
document.getElementById('section2').style.opacity = '0';
}


For some reason, its not working



Thanks!










share|improve this question


















  • 1




    What browser are you testing in?
    – Likwid_T
    Dec 8 '12 at 22:33






  • 1




    And what does your css look like?
    – Likwid_T
    Dec 8 '12 at 22:34






  • 3




    ID's starting with numbers are invalid, other than that it works just fine -> FIDDLE
    – adeneo
    Dec 8 '12 at 22:46












  • im using safari
    – Thomas Lai
    Dec 8 '12 at 22:52










  • When using opacity, you should try to look for cross-browser compatibility. So, IE 9+ and most modern browsers use opacity=1, IE8- uses filter='alpha(opacity=100)', others use -moz-opacity=1 and -khtml-opacity=1.
    – Marvin
    Dec 8 '12 at 23:38














0












0








0







I'm trying to change the opacity of a with javascript when a div is clicked.



Here is my html:



<section id="section2">
<img onclick="one()" class=img src="1.png" />
</section>
<section id="6-1">
Hello World
</section>


And my javascript:



function one() {
document.getElementById('6-1').style.opacity = '1';
document.getElementById('section2').style.opacity = '0';
}


For some reason, its not working



Thanks!










share|improve this question













I'm trying to change the opacity of a with javascript when a div is clicked.



Here is my html:



<section id="section2">
<img onclick="one()" class=img src="1.png" />
</section>
<section id="6-1">
Hello World
</section>


And my javascript:



function one() {
document.getElementById('6-1').style.opacity = '1';
document.getElementById('section2').style.opacity = '0';
}


For some reason, its not working



Thanks!







javascript html css onclick opacity






share|improve this question













share|improve this question











share|improve this question




share|improve this question










asked Dec 8 '12 at 22:32









Thomas Lai

102313




102313








  • 1




    What browser are you testing in?
    – Likwid_T
    Dec 8 '12 at 22:33






  • 1




    And what does your css look like?
    – Likwid_T
    Dec 8 '12 at 22:34






  • 3




    ID's starting with numbers are invalid, other than that it works just fine -> FIDDLE
    – adeneo
    Dec 8 '12 at 22:46












  • im using safari
    – Thomas Lai
    Dec 8 '12 at 22:52










  • When using opacity, you should try to look for cross-browser compatibility. So, IE 9+ and most modern browsers use opacity=1, IE8- uses filter='alpha(opacity=100)', others use -moz-opacity=1 and -khtml-opacity=1.
    – Marvin
    Dec 8 '12 at 23:38














  • 1




    What browser are you testing in?
    – Likwid_T
    Dec 8 '12 at 22:33






  • 1




    And what does your css look like?
    – Likwid_T
    Dec 8 '12 at 22:34






  • 3




    ID's starting with numbers are invalid, other than that it works just fine -> FIDDLE
    – adeneo
    Dec 8 '12 at 22:46












  • im using safari
    – Thomas Lai
    Dec 8 '12 at 22:52










  • When using opacity, you should try to look for cross-browser compatibility. So, IE 9+ and most modern browsers use opacity=1, IE8- uses filter='alpha(opacity=100)', others use -moz-opacity=1 and -khtml-opacity=1.
    – Marvin
    Dec 8 '12 at 23:38








1




1




What browser are you testing in?
– Likwid_T
Dec 8 '12 at 22:33




What browser are you testing in?
– Likwid_T
Dec 8 '12 at 22:33




1




1




And what does your css look like?
– Likwid_T
Dec 8 '12 at 22:34




And what does your css look like?
– Likwid_T
Dec 8 '12 at 22:34




3




3




ID's starting with numbers are invalid, other than that it works just fine -> FIDDLE
– adeneo
Dec 8 '12 at 22:46






ID's starting with numbers are invalid, other than that it works just fine -> FIDDLE
– adeneo
Dec 8 '12 at 22:46














im using safari
– Thomas Lai
Dec 8 '12 at 22:52




im using safari
– Thomas Lai
Dec 8 '12 at 22:52












When using opacity, you should try to look for cross-browser compatibility. So, IE 9+ and most modern browsers use opacity=1, IE8- uses filter='alpha(opacity=100)', others use -moz-opacity=1 and -khtml-opacity=1.
– Marvin
Dec 8 '12 at 23:38




When using opacity, you should try to look for cross-browser compatibility. So, IE 9+ and most modern browsers use opacity=1, IE8- uses filter='alpha(opacity=100)', others use -moz-opacity=1 and -khtml-opacity=1.
– Marvin
Dec 8 '12 at 23:38












2 Answers
2






active

oldest

votes


















1














Like ModernDesigner said, enquote your attributes and change your ids to not start with numbers, but other than that, I still couldn't figure out your problem. But for now you can use jQuery to make it work.



Here's a Fiddle that toggles it every time you click it, so hope this helps



It kept saying the function wasn't defined, so maybe you had some incorrect syntax or something. For now though i used jQuery's .click() function instead of onClick= because it wasn't working.






share|improve this answer





























    0














    try to set the value without the ' ', like this:



    document.getElementById("section2").style.opacity = 1;



    it worked for me just fine.






    share|improve this answer





















      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%2f13782606%2fchange-section-opacity-onclick%23new-answer', 'question_page');
      }
      );

      Post as a guest















      Required, but never shown

























      2 Answers
      2






      active

      oldest

      votes








      2 Answers
      2






      active

      oldest

      votes









      active

      oldest

      votes






      active

      oldest

      votes









      1














      Like ModernDesigner said, enquote your attributes and change your ids to not start with numbers, but other than that, I still couldn't figure out your problem. But for now you can use jQuery to make it work.



      Here's a Fiddle that toggles it every time you click it, so hope this helps



      It kept saying the function wasn't defined, so maybe you had some incorrect syntax or something. For now though i used jQuery's .click() function instead of onClick= because it wasn't working.






      share|improve this answer


























        1














        Like ModernDesigner said, enquote your attributes and change your ids to not start with numbers, but other than that, I still couldn't figure out your problem. But for now you can use jQuery to make it work.



        Here's a Fiddle that toggles it every time you click it, so hope this helps



        It kept saying the function wasn't defined, so maybe you had some incorrect syntax or something. For now though i used jQuery's .click() function instead of onClick= because it wasn't working.






        share|improve this answer
























          1












          1








          1






          Like ModernDesigner said, enquote your attributes and change your ids to not start with numbers, but other than that, I still couldn't figure out your problem. But for now you can use jQuery to make it work.



          Here's a Fiddle that toggles it every time you click it, so hope this helps



          It kept saying the function wasn't defined, so maybe you had some incorrect syntax or something. For now though i used jQuery's .click() function instead of onClick= because it wasn't working.






          share|improve this answer












          Like ModernDesigner said, enquote your attributes and change your ids to not start with numbers, but other than that, I still couldn't figure out your problem. But for now you can use jQuery to make it work.



          Here's a Fiddle that toggles it every time you click it, so hope this helps



          It kept saying the function wasn't defined, so maybe you had some incorrect syntax or something. For now though i used jQuery's .click() function instead of onClick= because it wasn't working.







          share|improve this answer












          share|improve this answer



          share|improve this answer










          answered Dec 9 '12 at 2:40









          jackcogdill

          3,12431842




          3,12431842

























              0














              try to set the value without the ' ', like this:



              document.getElementById("section2").style.opacity = 1;



              it worked for me just fine.






              share|improve this answer


























                0














                try to set the value without the ' ', like this:



                document.getElementById("section2").style.opacity = 1;



                it worked for me just fine.






                share|improve this answer
























                  0












                  0








                  0






                  try to set the value without the ' ', like this:



                  document.getElementById("section2").style.opacity = 1;



                  it worked for me just fine.






                  share|improve this answer












                  try to set the value without the ' ', like this:



                  document.getElementById("section2").style.opacity = 1;



                  it worked for me just fine.







                  share|improve this answer












                  share|improve this answer



                  share|improve this answer










                  answered Nov 23 at 1:57









                  Patrícia Silva

                  1




                  1






























                      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.





                      Some of your past answers have not been well-received, and you're in danger of being blocked from answering.


                      Please pay close attention to the following guidance:


                      • 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%2f13782606%2fchange-section-opacity-onclick%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

                      Lallio

                      Unable to find Lightning Node

                      Futebolista