Piccola richiesta di funzionalità per core: aggiungere 3 classi

Ciao, sto lavorando allo stile della casella ‘citazione’… è possibile che il core di Discourse acquisisca delle classi sulla freccia in giù (vedi citazione nel contesto) e sulla freccia in su (salta al messaggio citato)?

E, cosa più importante, potrebbe esserci anche una classe sull’aside principale .quote (div genitore) per indicare se l’utente citato è bloccato?

Non sto chiedendo modifiche visive, non sto chiedendo cambiamenti al modo in cui funziona l’“ignorare” :wink: voglio solo alcune classi per avere a disposizione delle opzioni per me e per altri sviluppatori di temi futuri.

Sono anche felice di provarci io stesso e inviare una PR, se qualcuno può indicarmi grossolanamente la direzione giusta su dove nel codebase dovrei tentare di farlo.

Sei sicuro di doverlo fare? Non puoi indirizzarlo in altri modi? Puoi persino usare l’attributo title per indirizzarlo, se proprio necessario.

Forse riesco a selezionare quei due elementi (le frecce) via CSS, sì. Saranno dei selettori piuttosto complessi e, ovviamente, più fragili rispetto all’aggiunta diretta di classi su quegli elementi, ma probabilmente ce la faccio.

La parte più difficile è capire se quell’utente sia ignorato o meno. Per quanto riesco a vedere, nessuno dei contenitori genitore aggiunge una classe per indicare lo stato “ignorato” nella citazione. Senza quella classe, non posso fare nulla di diverso in CSS. Cosa ne pensi? Fammi sapere se sbaglio :slight_smile:

L’utente ignorato è stato citato, probabilmente non è presente; questo richiederebbe effettivamente una modifica al core.

Non sono sicuro di come mi senta nel fare lavoro qui per aggiungere la classe; immagino che se apri una PR e risulta abbastanza semplice, sono felice di accettarla.

Vediamo se riesco a capirlo :slight_smile:

È stato sorprendentemente facile! Mi ci è voluto più tempo per trovare dove aggiungere il codice che per scrivere quella singola riga. Non è sempre così? :wink:

EHI RAGAZZI, È IL MIO PRIMO PULL REQUEST PER DISCOURSE CORE! :partying_face:

PR in corso: FEATURE: add class to <aside> quote block when quoting an ignored user by pnoeric · Pull Request #9511 · discourse/discourse · GitHub

Le frecce su e giù sono facili

.quote-controls .svg-icon-title
.quote-controls .back

Non sono per niente sicuro riguardo ai post bloccati, potresti avere ragione sul fatto che la tua PR sia la strada migliore.