retepasw / joomla-italiapa

Implementazione delle Linee guida di design per i servizi web della PA basato su it Web Toolkit.
https://www.eshiol.it/it/b013
GNU General Public License v3.0
13 stars 9 forks source link

Contatti - Icona telefono #415

Closed eshiol closed 3 years ago

eshiol commented 3 years ago

Pull Request for Issue #414 .

Summary of Changes

Corretta visualizzazione icona telefono contatti.

Testing Instructions

Creare un contatto e compilare il campo telefono Creare una voce di menu di tipo Contatti - Singolo contatto con opzione Link contatto su Mostra image

Expected result

image

Actual result

image

Documentation Changes Required

Knock-Out commented 3 years ago

Applicata Commit SHA: 94c0ac8f68 ✅✅✅

### System information pkg_italiapa-3.9.0.20-dev- a8b72be19a88c94725a08a7ecdf5177eac203710

NioBee commented 3 years ago

Applicata Commit SHA: 94c0ac8f68 ✅

image

Secondo me la visualizzazione è corretta, tuttavia non capisco se l'icona debba risultare cliccabile.. perchè le altre hanno comunque un collegamento..

System information

Template: 3.9.0.20-dev Joomla! 3.9.24 Stable Versione PHP 7.3.23

Knock-Out commented 3 years ago

@NioBee

Secondo me la visualizzazione è corretta, tuttavia non capisco se l'icona debba risultare cliccabile.. perchè le altre hanno comunque un collegamento..

L'icon dovrebbe essere di tipo:

NioBee commented 3 years ago

@NioBee

Secondo me la visualizzazione è corretta, tuttavia non capisco se l'icona debba risultare cliccabile.. perchè le altre hanno comunque un collegamento..

L'icon dovrebbe essere di tipo:

  • Icon-earth: Link
  • Icon-envelope e phone: Tooltip

e allora mancano i tooltip 😅 E comunque nell'icona bustina al momento è attivo il link mailto

atodaro commented 3 years ago

Applicata Commit SHA: 94c0ac8f68 con successo. La funzionalità richiesta da @NioBee possiamo inserirla come nuova issue.

Knock-Out commented 3 years ago

Informazione "Telefono" duplicata.

Actual result

Contatti

Expected result

Contatti-(1)

atodaro commented 3 years ago

Io forse lascerei così le informazioni. Tuttavia, sarebbe necessario effettuare un test con uno screen reader per "sentire" come vengono lette le informazioni associate alle icone e verificare se si rende necessario avere entrambi i dati

NioBee commented 3 years ago

Quello che facevo notare io in merito all'icona telefono, è che l'icona in qualche senso è "muta", entrando nel codice della pagina il numero di telefono si vede, ma per un normale utente che visita il sito, si trova un'icona ma ad essa non è associata alcuna informazione, il numero non compare (e per questo facevo riferimento al tooltip). Tra l'altro la visualizzazione del contatto lasciando le impostazioni di default non riporta la schermata di @Knock-Out ma qualcosa di più ridotto e semplice: image Invece andando nell'elenco contatti per categoria compare così: image

Knock-Out commented 3 years ago

Riguardo il mio commento

In configurazione globale basterebbe nascondere la visualizzazione del contatto telefonico e mantenere l' opzione link contatto in mostra.

@NioBee

Tra l'altro la visualizzazione del contatto lasciando le impostazioni di default non riporta la schermata di @Knock-Out ma qualcosa di più ridotto e semplice:

In configurazione globale del componente Contatti attivare le informazioni contatto desiderate come:

Nel singolo contatto compilare i campi scelti.

eshiol commented 3 years ago

@NioBee

Secondo me la visualizzazione è corretta, tuttavia non capisco se l'icona debba risultare cliccabile.. perchè le altre hanno comunque un collegamento..

L'icon dovrebbe essere di tipo:

  • Icon-earth: Link
  • Icon-envelope e phone: Tooltip

e allora mancano i tooltip 😅 E comunque nell'icona bustina al momento è attivo il link mailto

Hai provato con la PR#416 attiva?

NioBee commented 3 years ago

Hai provato con la PR#416 attiva?

tooltip

😏😝