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

Server is now http2 + Allow resize based on height #8

Merged
merged 5 commits into from
May 31, 2024

Conversation

Nickinthebox
Copy link
Member

@Nickinthebox Nickinthebox commented May 29, 2024

Nouveau comportement

Les nouveaux serveurs d'image sont http2.

Grace au multiplexage apporté par http2, on n'a plus besoin de séparer les calls sur le serveur (entre img.mapado.net et img1.mapado.net)

Par ailleurs, on permet à présent un redimensionnement basé sur une hauteur (et pas seulement une largeur), ce qui est particulièrement pratique pour un logo affiché sur une ligne dont on connait la hauteur, mais dont la largeur peut varier.

Tache Clickup

🧑‍🔧

@Nickinthebox Nickinthebox requested a review from jdeniau May 30, 2024 10:19

$baseUrl = '//img.mapado.net/2014/1/1/my-image';
$url = $this->manager->buildUrl($baseUrl, 0, 200);
$this->assertSame($baseUrl . '_thumbs/0-200', $url);
Copy link
Member

Choose a reason for hiding this comment

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

Can you add a test with null and null for width and height ? and another one with ['allowwebp' => true] ?

@Nickinthebox Nickinthebox merged commit caa688a into main May 31, 2024
1 check passed
@Nickinthebox Nickinthebox deleted the nr-update_with_new_server_features branch May 31, 2024 10:31
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