Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug componente Rating #117

Closed
AlessandroVecchi opened this issue May 16, 2023 · 0 comments · Fixed by #122 or #124
Closed

Bug componente Rating #117

AlessandroVecchi opened this issue May 16, 2023 · 0 comments · Fixed by #122 or #124
Assignees
Labels
bug Something isn't working

Comments

@AlessandroVecchi
Copy link

Ciao,
Segnalo 2 errori nel componente CmpRating.

Uno è di natura formale ed è il fatto che un <h3> non può essere annidato dentro in <legend>.

            <legend class="iscrizioni-header w-100">
              <h3 class="step-title d-flex align-items-center justify-content-between drop-shadow">
                <span class="d-block d-lg-inline" data-element="feedback-rating-question">
                  Quali sono stati gli aspetti che hai preferito?
                </span>
                <span class="step">1/2</span>
              </h3>
            </legend>

Peraltro in una versione precedente dello stesso componente il codice era:

              <div class="iscrizioni-header w-100">
                <h3 class="step-title d-flex align-items-center justify-content-between drop-shadow">
                  <legend data-element="feedback-rating-question" class="d-block d-lg-inline">
                    Dove hai incontrato le maggiori difficoltà?
                  </legend>
                  <span class="step">1/2</span>
                </h3>
              </div>

e anche in questo caso mi veniva segnalato che un <legend> non può stare all'interno di un <h3> e ha senso perché uno è un titolo di sezione, l'altro un titolo intendo ad un gruppo di campi. Quindi è sbagliato forzare un <legend> a comportarsi da h3 o viceversa.
Peraltro la formattazione è data dalla classe .step-title quindi l'h3 non serve nemmeno a quello.
Suggerire di eliminare l'h3 oppure il legend a seconda di cosa si vuole ottenere.

Il secondo problema è di impaginazione; da mobile (es. dalla pagina https://italia.github.io/design-comuni-pagine-statiche/sito/homepage.html) si vede così:

Immagine 2023-05-16 162040

Nel codice peraltro ci sono un sacco di classi di cui non capisco bene lo scopo... l'h3 viene impostato come d-flex e poi lo span al suo interno come d-block d-lg-inline.

Io ho risolto aggiungendo la classe "no-wrap" al legend ma è comunque un "taccone"
<legend class="iscrizioni-header w-100 text-wrap">

@zetareticoli zetareticoli added the bug Something isn't working label Jun 7, 2023
@zetareticoli zetareticoli self-assigned this Jun 7, 2023
zetareticoli added a commit that referenced this issue Jun 16, 2023
This was linked to pull requests Jun 16, 2023
zetareticoli added a commit that referenced this issue Jun 19, 2023
* Changed paged subtitle

* Changed heading class

* Better indendation

* Changed card summary classes

* Added new cards list template and content

* Remove obsolete containers

* Renamed template

* Removed custom font size

* Fixed lista card  templates

* Fixed section title

* Updated button at the end of cards list

* Removed dots from menu nav item

* Fixed aria-label

* Fixed card teaser border

* Fixed data-element

* Fixed nav tabs size and spacing

* Fixed wrong brand size

* Fixed text wrap #117

* Added navigation links

* Added url param

* Removed doc link

* Added if
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
2 participants