Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Melhorando a reaponsividade #43

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

schirrel
Copy link

@schirrel schirrel commented Oct 1, 2019

O botões estavam ficando um por cima dos outros ao redimensionar a pagina, e sua altura também não era ajustada.
O botão permanecia de acordo com a pagina inicialmente renderizada. Agora ao redimensionar os span aparecem ou somem

@coveralls
Copy link

coveralls commented Oct 1, 2019

Coverage Status

Coverage decreased (-2.5%) to 94.388% when pulling 2bd78d3 on CoderSquirrel:master into 64c7d16 on globocom:master.

@rodrigolourenco
Copy link
Contributor

Opa, tudo bem ? Fiz a análise aqui do seu MR e tem umas considerações que eu preciso levar pra você.
Primeiro obrigado por contribuir !!! <3
Agora a análise :

  • Por conta da alteração nos spans tivemos dois efeitos colaterais, o primeiro é que quando definimos uma largura mínima o span continua ali disposto na página e isso faz com que ele não respeite a largura setada. Para efeito de comparação segue um print
    image

  • O outro ponto também é relacionado ao span mas para o tema minimal, ele está exibindo o span sempre
    image

Se puder dar uma olhadinha eu agradeço.

@schirrel
Copy link
Author

brigado @odrigolourenco ultimamente eu estou com uns problemas pessoais entao nao vou conseguir, mas pode deixar o pr aberto que assim que as coisas melhorarem eu corrijo tudinho <3

…nto da regra de tamanho do componente. Adicionado recreate para quando for inicializada a o componente em telas pequenas e redimensionada para telas grandes, ateriormente ao redimensionar continuava com a quantidade minima para o tamanho inicial
@schirrel
Copy link
Author

@rodrigolourenco oi rodrigo, percebi que a minha abordagem pra corrigir a responsividade meio que ia contra a regra de tamanho do componente. Alterei para que ela so seja aplicada ao componente quando o mesmo possuir a classe .share-full.
A minha proposta de melhorar o redimensionamento foi mais para o caso de um usuário carregar a tela em um tamanho e por alguma razão redimensiona-lá.
Para tentear contornar isso respeitando a regra do tamanho adicionei um metódo de recreate ao share.js.

Obrigado :P

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.

3 participants