feat: new accessible date widget #607
Conversation
deodorhunter
added
enhancement
deodorhunter
requested review from
Wagner3UB,
giuliaghisini,
pnicolli and
sabrina-bongiovanni
| defaultStart: moment().startOf('day'), | ||
| defaultEnd: moment().endOf('day'), | ||
| isOutsideRange: () => false, | ||
| showInputLabels: true, |
There was a problem hiding this comment.
| showInputLabels: true, | |
| showInputLabels: false, | |
| startLabel: intl.formatMessage(messages.scadenza_dal), | |
| endLabel: intl.formatMessage(messages.scadenza_al), |
| }, | ||
| ...generateFiltersProps( | ||
| filterOne, | ||
| 'filterOne', |
There was a problem hiding this comment.
Nel backlog accessibilita' ci sono degli automated test che si spaccano quando hai BandiSearch ed EventSearch nella stessa pagina perche' ci sono id uguali. Cambiare a filterOne[CTName]
There was a problem hiding this comment.
import { v4 as uuid } from 'uuid';
| @@ -0,0 +1,174 @@ | |||
| import { useDateRangePicker } from 'react-aria'; | |||
There was a problem hiding this comment.
da quelloo che capisco, correggimi se sbaglio, questo componente DateRangeFilter è generico, non è specifico solamente per i filtri dei blocchi di ricerca, giusto?
Se è così, lo sposterei da /Blocks/Common/SearchFilters
e lo metterei a qualche livello superiore, da usare come componente generico
There was a problem hiding this comment.
Non e' stato testato al di fuori dello scope del ticket, per questo l'avevo lasciato in questo contesto, ma possiamo spostarlo
| import { useCalendarCell } from 'react-aria'; | ||
| import { RangeCalendarState } from 'react-stately'; | ||
| import { CalendarDate, isSameDay } from '@internationalized/date'; |
| @@ -0,0 +1,48 @@ | |||
| import { useCalendarGrid, AriaCalendarGridProps } from 'react-aria'; | |||
| import { useLocale } from '@react-aria/i18n'; | |||
|
Update: attualmente in stand-by senza una data di fine, le decisioni implementative nella descrizione sono dipendenti da quando verrà fatto l'update a volto18, o, per chiarezza, alla versione di volto che splitta il codice del core in client, provider, blocks etc. |
Rimane da capire perche' ho avuto problemi alle traduzioni fuori dalle customizzazioni, e come risolvere nel caso in modo diverso. Oltretutto, valutare se importare react-aria-components come dipendenza installata oppure mettere plone/components e usarla dal di li.
Discutere la strada migliore per gestire la possibilità di customizzazione per temi figli
PS: giustamente come visto con @pnicolli alla conf, lazy loading spacca i test, quindi saran da sistemare anche quelli