Hallo labskaus,
if(document.getElementById(f).style.color=='#c50000') {
ich würde hier mal mit alert(document.getElementById(f).style.color) prüfen, was hier überhaupt ankommt. Auch schließe ich mich hier suits Meinung an und würde das a-Tag durch irgendetwas anderes, z.B. einen entsprechend gestylten <button type="button" ... ersetzen.
Gruß, Jürgen