Skip to content

Ajout des fonctions de bases pour générer les tailles d'images - #508

Open
cedric07 wants to merge 1 commit into
masterfrom
add/image-size-base
Open

cedric07 wants to merge 1 commit into
masterfrom
add/image-size-base

Conversation

@cedric07

@cedric07 cedric07 commented Sep 18, 2026

Copy link
Copy Markdown
Contributor
  • Introduced new image sizes 'landscape' and 'landscape-2x' for use in Gutenberg.
  • Fix the text domain for translations to 'beapi-frontend-framework'.
  • Added a filter to display custom image sizes in the Gutenberg editor.

Note

Low Risk
Theme bootstrap changes only—new thumbnails and a text-domain fix—with no auth, data, or security impact.

Overview
Registers landscape (600×400) and landscape-2x (1200×800) hard-cropped image sizes during theme setup, and wires them into the block editor by filtering image_size_names_choose so editors can pick Landscape - 600 x 400 in Gutenberg.

Also switches load_theme_textdomain from framework-textdomain to beapi-frontend-framework so the new size label uses the correct theme text domain.

Reviewed by Cursor Bugbot for commit 43535fd. Bugbot is set up for automated code reviews on this repo. Configure here.

- Introduced new image sizes 'landscape' and 'landscape-2x' for use in Gutenberg.
- Updated the text domain for translations to 'beapi-frontend-framework'.
- Added a filter to display custom image sizes in the Gutenberg editor.
@Rahe

Rahe commented Sep 18, 2026

Copy link
Copy Markdown
Member

Ce sont des choses faites par ARI (la déclaration de tailles).
Donc à mettre dans la déclaration de sizes.

Vaut mieux ne pas grouper tes modifs de textodmain et ça, une PR par sujet normalement

@cedric07

cedric07 commented Sep 18, 2026

Copy link
Copy Markdown
Contributor Author

Ici c'est plutôt pour gérer les taille dans gutenberg pour les patterns etc...

ARI n'est pas utilisé pour les images dans gutenberg. Elles utilisent bien ces tailles là.

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.

2 participants