search

teamdigitale / dati-semantic-frontend

Frontend for the NDC semantic repository
GNU Affero General Public License v3.0
5 stars 10 forks source link

[FE] Ricerca sul catalogo: stati di hover mancanti #504

Closed steps88 closed 6 months ago

steps88 commented 7 months ago

Sul dropdown di ordinamento, la card, e le chips di ricerca mancano gli stati all'hover

Dropdown > https://designers.italia.it/design-system/componenti/dropdown/ Card (speciale) > https://designers.italia.it/design-system/componenti/card/ Chips con x > https://italia.github.io/bootstrap-italia/docs/componenti/chips/#varianti-standard-e-grandi

image

SCHEMA hover card SCHEMA hover chips

FrankMaverick commented 7 months ago

Di seguito alcune domande.

Dropdown ordinamento Su Figma è stata prototipata una select, per la versione desktop, non una dropdown. Quindi è stato implementato come da prototipo.

  1. La dropdown è presente nella versione mobile, quindi si deve utilizzare quest'ultima versione?

Card (speciale) Si suggerisce l'utilizzo della card speciale, ma questa card diventa tutta cliccabile, per cui non sarebbe più possibile cliccare su URI.

  1. Lo stato di hover sulla card deve essere inserito su tutta la card (come card speciale) o solo sul titolo dell'Asset?

Chips con x L'hover può essere inserito, ma come da comportamento bootstrap, l'hover sulla chips non prevede l'icona di chiusura di colore blu (che invece è grigia) e lo sfondo della chips di colore bianco (che invece è trasparente).

  1. Inseriamo l'hover con icona di chiusura grigia e sfondo trasparente come da bootstrap?

@Clou-dia @AgostinoPurificato @VincenzoDeSisto

steps88 commented 7 months ago

Dropdown ordinamento Perdonami, ho sbagliato il riferimento: sono i filtri per asset semantico e per categoria a non avere l'hover sulle voci nella tendina. Quello di ordinamento è corretto.

Card (speciale) Solo sul titolo dell'asset, valutate voi quale variante sia la migliore per ottenere il risultato

Chips con x Non importa la personalizzazione del colore, usiamo il componente bootstrap italia

VincenzoDeSisto commented 6 months ago

La issue è stata rilasciata in ambiente di test. Attendiamo un vostro riscontro. Grazie

@Clou-dia @steps88 @AgostinoPurificato

Clou-dia commented 6 months ago

Testato sui i browser: Safari, Chrome, Edge, Firefox. Tutti ok al netto di Safari

Comportamento attuale L'hover deve essere applicato solo agli elementi cliccabili (titolo, url), dal test anche gli elementi chips (tipologia risorsa e status stabile bozza) hanno stato hover, succede solo su Safari.

Comportamento atteso > Visto che le chips non sono cliccabili (ad oggi) non devono avere lo stato di hover

Schermata 2024-05-02 alle 12 29 14 Schermata 2024-05-02 alle 17 34 10
VincenzoDeSisto commented 6 months ago

@Clou-dia Non riusciamo a replicare il bug (screen in allegato). Potreste gentilmente descrivere passo dopo passo i passaggi per replicare la issue? Grazie in anticipo.

image (1)
Clou-dia commented 6 months ago

Provo con un video. Quando passo il mouse sugli elementi chips, vengono sottolineate. Solo da Safari (provato da SO Big Sure e Monterey) https://github.com/teamdigitale/dati-semantic-frontend/assets/98462345/7b9e20f0-f572-443b-9b2d-182030968759

Clou-dia commented 6 months ago

non ancora verificata

VincenzoDeSisto commented 6 months ago

E' stata rilasciata la fix in ambiente di test. Attendiamo un vostro riscontro. Grazie

@Clou-dia

Clou-dia commented 6 months ago

RIfacendo il test da Safari lo vedo risolto in dev ma non in test https://github.com/teamdigitale/dati-semantic-frontend/assets/98462345/cbca6e9f-dd5a-4fdd-b205-7deeda1092ed

VincenzoDeSisto commented 6 months ago

A causa di alcuni problemi nel flusso di rilascio, la modifica (presente in dev) non era stata propagata correttamente su test. La risoluzione adesso è presente anche sull'ambiente di test. Attendiamo vostro riscontro. Grazie

@Clou-dia

Clou-dia commented 6 months ago

verificato, grazie