✔ border color ändern und wieder zurück zum anfangswert

[johnjonnyjonson]

Hello ,
ich habe folgendes Problem
Ich will beim Klick auf ein Bild die Rahmen Farbe ändern und beim nächsten Klick die Farbe wieder zurücksetzten.
Ich habe es nach einem anderen Artikel so versucht:

<script type="text/javascript" language="JavaScript">

function farbTausch(id)
{
document.getElementById(id).style.borderColor = (document.getElementById(id).style.borderColor == '#FFFFFF')?'#F69C00':'FFFFFF';
}
</script>

sobald ich das doppelte = entferne setzt er zwar die Farbe beim ersten Klick´, beim zweiten Klick tut sich aber nichts.

Bin über jede Antwort Dankbar !

 

[sheel]

Hi

dir fehlt beim letzten FFFFFF ein #

 

[johnjonnyjonson]

achso ja das war nur ein kopier fehler! auch wenn # drin ist tut sich nichts

 

[johnjonnyjonson]

noch zum Verständnis, das ganze soll aus diese div funktionieren

<div onClick="farbTausch('id')"><img id="id" src="...." alt="" style="border: thick solid #FFFFFF"></div>

 

[SpiceLab]

Offensichtlich versagt hier die if/else-Bedingung mit hexadezimalen Farbcodes

Mit Farbnamen, in der korrekten Reihenfolge benannt, und mit einem gültigen ID-Bezeichner für <img> anstelle von "id" funktioniert das Script.

function farbTausch(id) {
  document.getElementById(id).style.borderColor = (document.getElementById(id).style.borderColor == 'orange') ? 'white' : 'orange';
}

<div onclick="farbTausch('imgID')"><img id="imgID" .../></div>

https://jsfiddle.net/spicelab/ojzqafjq/

 

[johnjonnyjonson]

danke dir, das ding ist das ich einen bestimmten orange ton brauche und die id hab ich aus privaten rechten hier so gesetzt.

 

[SpiceLab]

danke dir, das ding ist das ich einen bestimmten orange ton brauche und die id hab ich aus privaten rechten hier so gesetzt.

Dieser bestimmte Hexcode #F69C00 wird leider nicht in der Farbpalette mit 140 Farbnamen geführt.

Und was so ein ID-Bezeichner mit privaten Rechten zu tun hat, bleibt Dein Geheimnis

 

[johnjonnyjonson]

also ich weiß nicht ob den Code ohne hex mal ausprobiert hast, bei mir funktioniert er zumindest nicht

 

[SpiceLab]

also ich weiß nicht ob den Code ohne hex mal ausprobiert hast, bei mir funktioniert er zumindest nicht

Glaubst Du ernsthaft, dass ich so eine Behauptung in den Raum stelle, sowie ein nicht funktionstüchtiges Demo bei https://jsfiddle.net erstelle und hier verlinke?

Tja, da kennst Du mich leider sehr schlecht

Mit Farbnamen, in der korrekten Reihenfolge benannt, und mit einem gültigen ID-Bezeichner für <img> anstelle von "id" funktioniert das Script.

function farbTausch(id) {
  document.getElementById(id).style.borderColor = (document.getElementById(id).style.borderColor == 'orange') ? 'white' : 'orange';
}

<div onclick="farbTausch('imgID')"><img id="imgID" .../></div>

https://jsfiddle.net/spicelab/ojzqafjq/

Mit welchem Browser hast Du das denn erfolglos überprüft?

 

[johnjonnyjonson]

mit Internet Explorer und Chrome,
da hast du mich falsch verstanden das sollte kein angriff sein, mich hat es nur gewundert dass es bei mir nicht klappt.
und ich habe es noch nochmal ausprobiert -> geht immer noch nicht obwohl es gehen sollte.
das ganze bewegt sich in einer Panel Umgebung als klasse, kann es sein, dass es dadurch aus irgeneinem grund geblockt wird.
bzw. ich verwende davor noch ein anderes script.

<div class="panel panel-default">
<a name="down1"></a>
<h3 class="panel-body1">
<a id="an" href="javascript:toggle('down1')" style="font-size:26px">
<div onClick="farbtausch('bild')">
<img id="bild" src="images/ImgResponsive_Placeholder.png" alt="" style="border:  thick solid #FFFFFF"></div>
<p align="left" id="an">Bla</p>
</a>
</h3>
<div id="pflege" style="display: none">bla
</div>
</div>

und das wär das script zu toggle:

<script type="text/javascript">
function toggle(control){
    var elem = document.getElementById(control);
    if(elem.style.display == "none"){
        elem.style.display = "block";
    }else{
        elem.style.display = "none";

    }
}</script>