wp-media / imagify-plugin

Speed up your website with lighter images without losing quality.
https://imagify.io
73 stars 26 forks source link

Inconsistent status for next-gen when enable/disable display next-gen in specific scenario #884

Open Mai-Saad opened 7 months ago

Mai-Saad commented 7 months ago

Before submitting an issue please check that you’ve completed the following steps:

Describe the bug When enabling display next-gen while the next-gen filter is used, then disabling the filter, the UI is set to webP even though it was avif

To Reproduce Steps to reproduce the behavior:

  1. install and activate Imagify 2.2.2
  2. enable avif
  3. enable webp using filter => options are dimmed
  4. disable the filter => next-gen options active with avif enabled while
  5. install and activate Imagify 2.2.2
  6. enable avif
  7. enable webp using filter => options are dimmed
  8. enable display next-gen
  9. disable the filter => next-gen options active with webP enabled

Expected behavior Both scenarios shall be consistent either in keeping the previous value of the filter or setting it to default (shouldn't be affected by enabling Display next-gen option) => please refer to @wp-media/productimagify to confirm expectation

Screenshots If applicable, add screenshots to help explain your problem.

Additional context Add any other context about the problem here.

Acceptance Criteria (for WP Media team use only) Clear instructions for developers, to be added before the grooming