Skip to content

Conversation

mateusfccp
Copy link
Contributor

<meta name="viewport" content="width=device-width, initial-scale=1" />

Com esse ajuste, temos duas melhorias:

  1. Evitamos 300ms de delay de input;
  2. Melhoramos a experiência em dispositivos mobile.
Antes Depois
image image

Mudança no estilo (CSS) da logo

Os estilos height, background-color, border-radius e border foram removidos.

Com esse ajuste, temos duas melhorias:

  1. A logo antes estava mostrando uma borda cinza apenas de um lado, porque o estilo foi projetado para logos quadradas, enquanto a do nosso site é 64x57;
  2. O tamanho 35x35 forçado na logo fazia com que ela ficasse distorcida, já que ela não é quadrada.
Antes Depois
image image

Servindo a logo como parte de nossos arquivos estáticos

Com esse PR, passamos a servir nossa logo diretamente dos nossos arquivos estáticos.

Com esse ajuste, temos duas melhorias:

  1. Pude otimizar o tamanho da logo, reduzindo seu tamanho por quase metade;
  2. Fazemos com que o site seja 100% servido em HTTPS, resolvendo o warning abaixo.
image

<link rel="icon" href="data:,">

Adicionando essa tag no cabeçalho do HTML, evitamos a requisição do favicon.ico. Considerando que nosso site não possui favicon, é melhor não requisitá-lo, economizando 250B de banda e uma requisição desnecessária.

Signed-off-by: Mateus Felipe C. C. Pinto <mateusfccp@gmail.com>
@mateusfccp mateusfccp requested a review from ryukinix April 22, 2024 19:01
Copy link
Member

@ryukinix ryukinix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LVGTM (Looks Very Good to Me).

@ryukinix ryukinix merged commit 116fd6d into master Apr 22, 2024
@ryukinix ryukinix deleted the small_improvements branch April 22, 2024 19:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants