Ok, ecco alcune modifiche minori per iniziare:
- Se posso accettare/non accettare, è lo stesso pulsante che abbiamo ora (ho spostato il testo all’interno del pulsante)…
- Se non posso accettare/non accettare, invece di mostrare un pulsante, visualizza solo il testo “risolto” con un segno di spunta. Questo lo rende meno simile a un pulsante, che era la richiesta originale dell’OP.
Ho anche aggiunto un attributo title a questo testo che dice “Questa è la soluzione accettata per questo argomento”. Il nostro metodo attuale di avere un pulsante che non fa nulla è probabilmente molto confuso per chiunque utilizzi uno screen reader. Questo lo chiarirebbe in modo significativo.
Ecco una PR per queste modifiche:
Nelle modifiche sopra non ho apportato aggiornamenti aggiuntivi per i dispositivi mobili… il nostro layout lì tende ad essere un po’ rotto quando si espandono tutti i controlli perché non c’è abbastanza spazio per tutto (ad esempio, se sei un amministratore).
Come prossimo passo, vorrei aggiungere una classe quando i controlli del post vengono espansi/ripiegati in modo da poter mostrare/nascondere il testo “risolto”. Quindi nella maggior parte dei casi tutto starà bene così:
