-
Notifications
You must be signed in to change notification settings - Fork 49
Folha de estilo com seletores muito abrangentes #28
Comments
👍 |
Olá @pguedesbr Realmente, a abrangência está maior que a necessária. Vamos providenciar as alterações assim que possível. Se você puder, por favor, mencionar exemplos de temas, telas e/ou disponibilizar alguma evidência (print), que você tenha nos ajudaria no processo de validação das alterações. Obrigado pela contribuição! (: |
@gabriellucius As consequências mais visíveis em meu modulo em desenvolvimento foram em labels (fonte branca) e tabelas (com bordas). A partir dessas evidencias constatei que a folha de estilo do pagseguro estava muito abrangentes. |
@pguedesbr você usa algum tema específico na sua loja? Você customizou os estilos da sua loja? Pode disponibilizar algum print do comportamento? (embora haja alguns seletores mais abrangentes no módulo, nas lojas de testes que temos não está comprometendo o layout, então caso você disponibilize mais informações ficará mais fácil atuar no problema). |
@gabriellucius estamos usando o tema Porto E-Commerce mas na real, não é muito relevante. Exemplo abaixo, um seletor ".label" mudando a cor dos atributos na aba de Mais Informações do produto: Reitero que não devem tratar caso a caso. O que vocês devem fazer é prefixar todas as regras css com seletores que as limitem apenas a blocos do próprio módulo do Pagseguro. Ou mais, evitem usar regras css de reset como parte do módulo. |
Algum update nesse bug? Esta folha do pagseguro está alterando diversas sessões de qualquer tema, até Blank e Luma. Não podem amarrar esses ajustes dentro de uma classe css do módulo? |
@willfrombrazil ainda estamos validando. |
O arquivo style.css desse módulo possui uma série de seletores extremamente abrangentes, com button, table, input, select.... etc sem nenhuma limitação de escopo.
Como resultado, as lojas ficam com seus temas deformados pelas regras de estilo do módulo do pagseguro. Isso precisa ser corrigido o quanto antes!
The text was updated successfully, but these errors were encountered: