-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: new accessible date widget #607
base: 11.x.x
Are you sure you want to change the base?
Conversation
defaultStart: moment().startOf('day'), | ||
defaultEnd: moment().endOf('day'), | ||
isOutsideRange: () => false, | ||
showInputLabels: true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
import { v4 as uuid } from 'uuid';
@@ -0,0 +1,174 @@ | |||
import { useDateRangePicker } from 'react-aria'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Installa queste tre dipendenze
@@ -0,0 +1,48 @@ | |||
import { useCalendarGrid, AriaCalendarGridProps } from 'react-aria'; | |||
import { useLocale } from '@react-aria/i18n'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Installa
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