Skip to content

Conversation

@iacopolea
Copy link
Member

No description provided.

@iacopolea iacopolea marked this pull request as ready for review November 16, 2022 16:48
@iacopolea iacopolea force-pushed the UN-64-progress-widget branch from bd7331b to 8f9aa9f Compare November 21, 2022 09:22
@marcbon marcbon requested a review from cannarocks November 21, 2022 09:35
Copy link
Member

@cannarocks cannarocks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Per il merge mancano:

  • tooltip tradotti e con percentuali
  • tooltip usecase con range al posto del valore esatto EDIT: Ho parlato con design, vogliono l'hover solo sui rettangoli e non sul marker
  • sostituire le traduzioni di default con l'inglese o rimuovere il default
  • Aggiornare ordine widget: prima questo poi quello dei bug unici ed infine la mezza torta

@iacopolea iacopolea requested a review from cannarocks November 21, 2022 16:17
@iacopolea
Copy link
Member Author

Canna vedi un po' che dovrei aver sistemato. Ho anche sistemato le dimensioni del grafico che erano un po' sballate

@d-beezee d-beezee merged commit 2203f59 into UN-93 Nov 22, 2022
@iacopolea iacopolea deleted the UN-64-progress-widget branch November 22, 2022 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants