GoogleChromeLabs/squoosh

“Copy settings to other side” ignores “advanced” expand/collapsed state

Open

Aperta il 27 mag 2021

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)TypeScript (20.335 star) (1431 fork)batch import
UIhelp wanted

Descrizione

  1. Open image
  2. Select AVIF (or any other codec with “advanced options”
  3. Expand advanced options and change some of them
  4. Click “copy settings to other side”
  5. Advanced section on other side is not expanded (but settings are indeed copied)

Guida contributor

Tech stack
typescriptweb components
Dominio
frontend
Tipo issue
bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
under 1 hour
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
stale
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
clear
Prerequisiti
understanding of web componentsstate management basics
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70
Direzione di ricerca
Investigate the 'copy settings to other side' functionality in the codebase. This is likely in a settings panel component. The advanced options' expand/collapse state is not being copied, but the settings values are. Look for where the expanded state is stored and ensure it is included when copying. Check the component that handles the copy action, possibly in the settings panel or the main image editor. Files such as 'settings panel.ts' or related may be relevant.