GoogleChromeLabs/squoosh

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

Open

#1.032 aberto em 27 de mai. de 2021

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)TypeScript (20.335 stars) (1.431 forks)batch import
UIhelp wanted

Description

  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)

Guia do colaborador

Pilha de tecnologia
typescriptweb components
Domain
frontend
Tipo Issue
bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
under 1 hour
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
understanding of web componentsstate management basics
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70
Direção de pesquisa
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.