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

General improvements #10

Merged
merged 37 commits into from
Oct 29, 2014
Merged

General improvements #10

merged 37 commits into from
Oct 29, 2014

Conversation

mikaelcarrara
Copy link
Member

Tinha uns estilos do footer meio perdidos ainda e a logo da HE:labs não estava com suporte pra retina.

Fixed.

@carlosbrando
Copy link
Member

👍

@mikaelcarrara mikaelcarrara changed the title Suporte do novo footer para telas retina. General improvements Oct 20, 2014
@mikaelcarrara
Copy link
Member Author

Fiz mais uns improvements de forma geral na home, organizei um pouco mais o CSS pra dar um suporte maior pra tablets e smartphones. Ainda não tá 100% o responsivo mas já ficou mais fácil de dar continuidade. Eliminei também um pouco de syntaxe xhtml antiga que tinha no código, entre outras coisas menores.

@mikaelcarrara
Copy link
Member Author

Ficou certo agora o box-model suportando o responsivo. Agora só preciso ir arrumando os componentes de dentro como listagens, etc.

screen shot 2014-10-21 at 9 37 25 am

@carlosbrando
Copy link
Member

❤️

@mikaelcarrara
Copy link
Member Author

@carlosbrando Vou dar mais uns tapas hoje! Então se puder esperar até o fim do dia pra dar o merge...

@carlosbrando
Copy link
Member

claro

@mikaelcarrara
Copy link
Member Author

Tirei o menu mobile do footer e estilizei ele novamente na topbar.

screen shot 2014-10-21 at 2 43 36 pm

@carlosbrando
Copy link
Member

:shipit:

@mikaelcarrara
Copy link
Member Author

Deixei um markup / style generico pra todas telas de about. Considerei então que profile é igual about.

screen shot 2014-10-21 at 3 27 59 pm

screen shot 2014-10-21 at 3 29 09 pm

@mikaelcarrara
Copy link
Member Author

Boa tarde @carlosbrando!

Ficou faltando o redesign da listagem de episódios responsiva, mas antes eu precisava arrumar os styles em volta que estavam dificultando essa melhoria. Agora todas as páginas estão responsivas, com o código organizado e com esse menu mobile novo.

Tentei criar um style mais bem acabado pro conteúdo do post mas o Jekyll ta gerando o elemento strong em vez de paragraph. Até tentei só colocar o estilo no strong, mas não ficou legal.

Criei uma issue pra isso: #12

screen shot 2014-10-21 at 5 25 59 pm

Fora isso acho que você pode dar merge já.

Qualquer outra alteração eu crio uma nova PR 👍

@mikaelcarrara
Copy link
Member Author

Pronto! 👍

@carlosbrando
Copy link
Member

@rafaelrosafu quando quiser!

@mikaelcarrara
Copy link
Member Author

ping

@rafaelrosafu
Copy link
Contributor

Desculpe a demora. Acabei de testar, mas peguei um problema na página de contatos, as imagens ficaram deformadas, perderam a proporção. A primeira imagem é com o patch aplicado, a segunda é do site como está hoje.

Pode arrumar antes de eu fazer o merge? Não precisa ser hoje, é claro.

screen shot 2014-10-24 at 4 15 16 pm

No site:
screen shot 2014-10-24 at 4 15 26 pm

@mikaelcarrara
Copy link
Member Author

@rafaelrosafu Opa! Beleza, pode deixar!

@mikaelcarrara
Copy link
Member Author

@rafaelrosafu Done!

@mikaelcarrara
Copy link
Member Author

screen shot 2014-10-25 at 8 26 54 pm

screen shot 2014-10-25 at 8 26 39 pm

@mikaelcarrara
Copy link
Member Author

@rafaelrosafu Mais algum detalhe? Ainda não ta 100% pronto, mas essa PR ja é um passo grande, principalmente pro responsivo.

@rafaelrosafu
Copy link
Contributor

O problema dos patrocinadores foi corrigido, agora vi um outro:

screen shot 2014-10-28 at 4 13 39 pm

Com mais 3 episódios fica estranho. Eu posso fazer o merge agora e criamos uma issue só para resolver isso depois, me diga como quer prosseguir.

@mikaelcarrara
Copy link
Member Author

@rafaelrosafu Sim, como disse ainda não está 100%. Já tem lá uma issue pra criar uma nova listagem. Esse problema já é antigo.

@mikaelcarrara
Copy link
Member Author

@rafaelrosafu Criei uma também pra deixar a home 100% responsiva. Nessa PR remodelei todos os boxes e containers pra suportar o responsivo e arrumei boa parte dos componentes dentro dos boxes mas ainda faltou alguns, como por exemplo a listagem de episódios.

@rafaelrosafu rafaelrosafu merged commit 7e977ed into grokpodcast:master Oct 29, 2014
@rafaelrosafu
Copy link
Contributor

Feito :)

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