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

[5.2] Compress the thumbnails #43282

Merged
merged 4 commits into from
Apr 30, 2024
Merged

[5.2] Compress the thumbnails #43282

merged 4 commits into from
Apr 30, 2024

Conversation

dgrammatiko
Copy link
Contributor

Pull Request for Issue #43272 .

Summary of Changes

Increase the compression rate for the generated thumbnails

Testing Instructions

Follow the issue description

Actual result BEFORE applying this Pull Request

Expected result AFTER applying this Pull Request

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

@brianteeman
Copy link
Contributor

Now we have thumbnails that are fit for purpose. Thanks!!!

Using the images in this archive to test with
test.zip

Filename Filesize kb
Original Before PR After PR
image1.png 44.05 105.85 6.58
image2.png 452.83 75.3 8.41
image3.png 30.47 93.39 9.51
image4.jpg 250.94 27.79 6.6
image5.jpg 114,55 26.0 6.87
image6.jpg 260.38 34.84 8.97

@brianteeman
Copy link
Contributor

I have tested this item ✅ successfully on 08c6404


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43282.

1 similar comment
@alikon
Copy link
Contributor

alikon commented Apr 15, 2024

I have tested this item ✅ successfully on 08c6404


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43282.

@alikon
Copy link
Contributor

alikon commented Apr 15, 2024

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43282.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Apr 15, 2024
@HLeithner HLeithner changed the base branch from 5.1-dev to 5.2-dev April 24, 2024 09:06
@HLeithner
Copy link
Member

This pull request has been automatically rebased to 5.2-dev.

@HLeithner HLeithner changed the title [5.1] Compress the thumbnails [5.2] Compress the thumbnails Apr 24, 2024
@pe7er pe7er self-assigned this Apr 30, 2024
@pe7er pe7er enabled auto-merge (squash) April 30, 2024 08:48
@pe7er pe7er merged commit c5d627c into joomla:5.2-dev Apr 30, 2024
3 checks passed
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Apr 30, 2024
@pe7er
Copy link
Contributor

pe7er commented Apr 30, 2024

Thanks @dgrammatiko !

@Quy Quy added this to the Joomla! 5.2.0 milestone Apr 30, 2024
@Quy Quy removed the PR-5.1-dev label Apr 30, 2024
@dgrammatiko dgrammatiko deleted the patch-1 branch July 8, 2024 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants