if (notesValue) { document.querySelector(`.card-${domain.id} .add-notes-button`).classList.add('notes-filled'); } else { document.querySelector(`.card-${domain.id} .add-notes-button`).classList.remove('notes-filled'); }La refactorización se realiza simplemente utilizando el segundo parámetro de fuerza del Método de toggle :
Element.classList.toggle("className", forceBoolean)
MDN Docs DOMTokenList.toggle()
const EL_button = document.querySelector(`.card-${domain.id} .add-notes-button`); EL_button.classList.toggle('notes-filled', notesValue);document .querySelector(`.card-${domain.id} .add-notes-button`) .classList .toggle('notes-filled', notesValue);Usar alternar es una buena idea, pero para un caso más general en el que no existe un método como ese, puede elegir qué método en el objeto apuntar, por ejemplo:
document.querySelector(`.card-${domain.id} .add-notes-button`) .classList[notesValue ? 'add' : 'remove']('notes-filled');