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

Add default mu-plugin for FacetWP (a11y) #21

Closed
wants to merge 1 commit into from

Conversation

petitphp
Copy link
Member

Ajout d'un default mu-plugin pour FacetWP.

Ce default fait plusieurs choses :

  • forcer l'option a11y de FacetWP,
  • refaire le html de la pagination des facettes,
  • ajout d'un label pour les facettes checkboxes/radio

Pour avis, est-ce que ça reste un mu-plugin ou est-ce que c'est intégré dans un service du thème ?

@ecorica
Copy link
Contributor

ecorica commented Sep 6, 2023

@petitphp on était partie sur l'idée de l'intégrer dans le BFF en tant que service :)

@petitphp
Copy link
Member Author

petitphp commented Sep 6, 2023

@ecorica pas de souci pour moi pour faire ça. C'était pour lancer la conversation. Est-ce qu'il faut créer une issue dans le dépôt du BFF du coup ?

@ecorica
Copy link
Contributor

ecorica commented Sep 25, 2023

@ecorica pas de souci pour moi pour faire ça. C'était pour lancer la conversation. Est-ce qu'il faut créer une issue dans le dépôt du BFF du coup ?

@petitphp yes il y a cette PR déjà dans le BFF : BeAPI/beapi-frontend-framework#372

@petitphp
Copy link
Member Author

Fermé en faveur de la PR BFF

@petitphp petitphp closed this Sep 28, 2023
@petitphp petitphp deleted the feat/add-default-facetwp branch September 28, 2023 07:49
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.

3 participants