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

feat: new accessible date widget #607

Draft
wants to merge 1 commit into
base: 11.x.x
Choose a base branch
from

Conversation

deodorhunter
Copy link

@deodorhunter deodorhunter commented Mar 25, 2024

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

@deodorhunter deodorhunter added enhancement New feature or request a11y-accessibility Fixes accessibility issues or adds accessibility features labels Mar 25, 2024
@deodorhunter deodorhunter self-assigned this Mar 25, 2024
@deodorhunter deodorhunter marked this pull request as draft March 25, 2024 08:56
defaultStart: moment().startOf('day'),
defaultEnd: moment().endOf('day'),
isOutsideRange: () => false,
showInputLabels: true,
Copy link
Author

Choose a reason for hiding this comment

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

Suggested change
showInputLabels: true,
showInputLabels: false,
startLabel: intl.formatMessage(messages.scadenza_dal),
endLabel: intl.formatMessage(messages.scadenza_al),

},
...generateFiltersProps(
filterOne,
'filterOne',
Copy link
Author

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]

Copy link
Member

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';

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

Copy link
Author

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

Comment on lines +1 to +3
import { useCalendarCell } from 'react-aria';
import { RangeCalendarState } from 'react-stately';
import { CalendarDate, isSameDay } from '@internationalized/date';
Copy link
Member

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';
Copy link
Member

Choose a reason for hiding this comment

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

Installa

@deodorhunter
Copy link
Author

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.

@pnicolli pnicolli changed the base branch from main to 11.x.x August 9, 2024 22:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y-accessibility Fixes accessibility issues or adds accessibility features enhancement New feature or request has-blockers major-deps-upgrade
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants