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

Correções na página inicial #24

Merged
merged 3 commits into from
Jul 20, 2015
Merged

Correções na página inicial #24

merged 3 commits into from
Jul 20, 2015

Conversation

alexandre-mbm
Copy link
Contributor

Para a resolução da issue #9.

captura de tela de 2015-07-17 18-40-56

Simplificação de expressões e correções em CSS (alinhamentos + coisa relevante a telas pequenas).


Tenho uma ideia para resolver #4. Apenas usa a imagem que já temos. É simples e resolveria muito da "ausência de cores" por todo o site, e sem exagero. Devo adicionar os commits neste mesmo pull request ou criar outro?

@alexandre-mbm
Copy link
Contributor Author

Devo adicionar os commits neste mesmo pull request ou criar outro?

Vou assumir que devo criar outro pull request. Será melhor para organizar eventuais discussões.

@alexandre-mbm
Copy link
Contributor Author

A nova expressão "o mapa do mundo livre", na minha opinião, faz um trocadilho melhor do que o trocadilho que era feito pela expressão "o mapa livre do mundo".

</h2>
<p>
O OpenStreetMap é um projeto de produção colaborativa de dados geo-espaciais abertos. </br>
Qualquer pessoa pode editar o <em><a href="http://www.openstreetmap.org/?locale=pt-BR#map=4/-15.19/-51.31" target="_blank">mapa</a></em> e os dados são redistribuídos sob uma <em><a href="http://www.openstreetmap.org/copyright?locale=pt-BR" target="_blank">licença aberta</a></em>.
O OpenStreetMap é um projeto de produção colaborativa de dados geo-espaciais abertos e reutilizáveis.</br>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Essa adição não é auto-explicativa. Acho melhor remover ela. Abaixo já é comentado sobre a licença de qualquer forma.

@jgpacker
Copy link
Collaborator

Adicionei umas observações no texto. Fora isto, está bacana :-)

@alexandre-mbm
Copy link
Contributor Author

Adicionei umas observações no texto.

Tenho respondido e vou tentar acatar a grande maioria das sugestões. Novos commits em breve. Não vou reescrever o histórico por que senão ele perde o sentido.

@alexandre-mbm
Copy link
Contributor Author

pr24-tentativa2

@jgpacker
Copy link
Collaborator

Na minha opinião ficou legal 👍

@alexandre-mbm
Copy link
Contributor Author

Depois que este pull request for aceito, com ou mais commits, eu farei a sincronia do #25.

@alexandre-mbm
Copy link
Contributor Author

Estou ciente das imperfeições de posicionamento e espaçamento que são denunciadas através da imagem de tela abaixo. Elas não foram geradas por este pull request. Pelo contrário, eram piores.

imperfeicoes


Descobri como corrigir. Não façam merge. Acrescentarei commits.

@alexandre-mbm
Copy link
Contributor Author

Pronto, agora sim! Podem aceitar/fazer o merge, se quiserem.

openstreetmap_brasil_-_2015-07-18_17 56 27

@alexandre-mbm
Copy link
Contributor Author

Se preferem, na tela pequena, conteúdo de rodapé alinhado à esquerda, posso fazer isso.

@nighto
Copy link
Member

nighto commented Jul 20, 2015

7df22dd

Acho que este commit resolve a quebra de linha em "dados geo-espaciais abertos".

naoliv added a commit that referenced this pull request Jul 20, 2015
Correções na página inicial
@naoliv naoliv merged commit 185c457 into master Jul 20, 2015
@alexandre-mbm
Copy link
Contributor Author

@nighto, 7df22dd foi empurrado para dentro do master através da mesclagem manual 13308af.

@alexandre-mbm
Copy link
Contributor Author

osmbrasil-padding-left

Alguém discorda da estratégia que é mostrada na imagem acima? Eu achei melhor centralizar as outras duas colunas para obter um visual geral mais equilibrado. Sigo em frente com o padding-left?

O problema, como todos podem ver, é a criação inevitável de uma terceira linha. Na tela menor, o padding-left voltaria ao valor de antes.

@nighto
Copy link
Member

nighto commented Jul 21, 2015

Faz sentido pra mim.

@alexandre-mbm
Copy link
Contributor Author

Atualizei o branch homepage pensando que este pull request seria "reaberto". Não aconteceu. Não vou empurrá-lo para o master por conta própria. Preferi colocá-lo como footer na discussão #28.

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.

4 participants