Das sollte aber auch mit einem Code passen....nur ich kann das nicht :oops:
Nee, also eigentlich nur dann, wenn die Anzahl der Elemente "menuitem" und "menu" gleich wären.
Dann könnte man das ganze in einer Schleife abhandeln. Aber da dies unwahrscheinlich ist, muss man das Spiel halt zweimal durchlaufen lassen. 
Danke für die Erklärung :klasse:
Wenn du es etwas eleganter haben willst, kannst du das auch in eine Funktion packen...
var removeAtrr = function(eTag) {
var ele = document.getElementsByTagName(eTag);
for (var i = 0; i < ele.length; i++) {
var status = ele[i].removeAttribute("accesskey");
}
};
removeAtrr("menu");
removeAtrr("menuitem");Wie wäre es mit document.querySelectorAll('menu,menuitem')? 
etwas eleganter haben willst
Genau so meinte ich das 
Sören, ich habe davon leider 0 Ahnung wie und wo ich deinen Text einfügen müsste :oops:
Stimmt, Sören...
Da kann man mal sehen, wie jQuery-verwöhnt ich doch bin.
Also, andreas, dann so:
var matches = document.querySelectorAll('menu , menuitem');
for (var i = 0; i < matches.length; i++) {
var status = matches[i].removeAttribute("accesskey");
}dann so:
Ok, dann nehme ich den...der sieht gut aus :wink:
Ich danke euch beiden dafür :klasse:
andreas: keine Sorge, du musstest meinen Hinweis nicht verstehen. Ich hatte darauf gesetzt, dass EffPeh damit was anfangen kann. 
Aber zur Klarstellung: es ist egal*, welche Variante du nimmst, die Variante mit document.querySelectorAll() ist einfach kürzer, weil du damit beides zusammenfassen kannst. Beides tut das Gleiche**.
*) es gibt einen Performance-Unterschied. Aber der ist in dieser Dimension vernachlässigbar.
**) stimmt technisch auch nicht ganz. getElementsByTagName() liefert eine HTMLCollection zurück und querySelectorAll() eine NodeList. Das muss uns aber nicht kümmern, weil es im Ergebnis für uns auf das Gleiche herauskommt.
... den Code packe ich in meine userChrome.css...?
Danke!
getElementsByTagName() liefert eine HTMLCollection zurück und querySelectorAll() eine NodeList.
Was bedeutet, das andreas das auch so lösen könnte...
document.querySelectorAll('menu , menuitem').forEach(
function(currentValue, currentIndex, listObj) {
var status = currentValue.removeAttribute("accesskey");
}
);Die vielen Wege nach Rom… Aber wenn wir hier schon dabei sind, unterschiedliche Wege zu diskutieren, ein paar allgemeine Tipps zur Verbesserung:
1. Im Beispiel oben heißt es for (var i = 0; i < matches.length; i++) { }
Daran ist schlecht, dass die Länge von matches in jedem Schleifen-Durchlauf neu berechnet wird. matches.length sollte außerhalb der Schleife einer Variablen zugewiesen werden und diese Variable sollte dann in der Schleife verwendet werden.
2. Im letzten Beispiel, ist es da wirklich notwendig, das Ergebnis einer Variablen zuzuweisen?
3. Grundsätzlich für jeden Code, insbesondere da wir hier exklusiv von Firefox sprechen und uns nicht um Kompatibilität mit anderen und älteren Browsern kümmern müssen: anstelle von var sollte eigentlich immer const oder let verwendet werden, sofern möglich. Was das Passende ist, hängt natürlich vom konkreten Code ab.
"`var` is now the weakest signal available when you define a variable in JavaScript."
Literatur dazu: https://medium.com/javascript-sce…st-ba58b8dcde75
@Sören:
Ja, das ist sicher alles richtig, was du da anmerkst.
Ich bin einfach zu oldskool und auch nicht mehr wirklich im Thema drin.
Aber ich werde mich bemühen, die nächsten Codes entsprechend anzupassen.
Dann habe ich ja mein Ziel erreicht. Die ganzen Basteleien am Browser interessieren mich gar nicht. Aber wenn ich zur Qualitäts-Steigerung des geschriebenen Codes beitragen und die Bastler zu besseren Entwicklern machen kann, freue ich mich.
Und ich freue mich immer, wenn ich noch etwas dazu lernen kann. Auch wenn ich es gar nicht mehr beruflich mache, so interessiert mich das alles immer noch nach wie vor.
Und ich freue mich darüber, dass es hier so tolle User gibt die anderen helfen :klasse:
An alle dafür mal ein dickes: Danke
Dito... besonders auch mal an Dich...
BTW: Wird der untere Code nun nochmal optimiert?
Optimiert...
Entweder so:
let matches = document.querySelectorAll('menu , menuitem');
let ml = matches.length;
for (var i = 0; i < ml; i++) {
matches[i].removeAttribute("accesskey");
}
oder so:
document.querySelectorAll('menu , menuitem').forEach(
function(currentValue, currentIndex, listObj) {
currentValue.removeAttribute("accesskey");
}
);Bei der for-Schleife kommt man auch ohne matches.length aus:
for (let element of matches) {
element.removeAttribute("accesskey");
};In jedem Fall wird das Attribut allerdings nur bei Menüeinträgen entfernt, die nach dem Öffnen des Fensters gleich vorhanden sind. Beim Webentwickler-Menü ist das nicht bei allen der Fall. Bei der Sidebar bleiben die Accesskeys auch. Da müsste man noch etwas machen. Wie Andreas will.