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.

Sign up for GitHub

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

Jump to bottom

Correções na página inicial #24

Merged
merged 3 commits into from
Jul 20, 2015
Merged

Correções na página inicial #24

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
af3c511
Correções na página inicial
alexandre-mbm Jul 17, 2015
56c1df5
Tentativa 2 para o pull request #24
alexandre-mbm Jul 18, 2015
23d2429
Melhoramento para telas menores
alexandre-mbm Jul 18, 2015
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion _config.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
# Site settings
title: OpenStreetMap Brasil
email:
description: "Informações sobre como mapear e participar da comunidade do OpenStreetMap."
description: "Associação de mapeadores"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Vejo o texto anterior como preferível.

baseurl: ""
url: "http://www.openstreetmap.com.br"
twitter_username:
Expand Down
37 changes: 20 additions & 17 deletions css/main.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -120,27 +120,30 @@ a:visited { color: #205caa; }
.site-footer .column { float: left; margin-bottom: 15px; }

.footer-col-1 {
width: 270px; /*fallback*/
width: -webkit-calc(35% - 10px);
width: -moz-calc(35% - 10px);
width: -o-calc(35% - 10px);
width: calc(35% - 10px);
margin-right: 10px
width: 302px; /*fallback*/
width: -webkit-calc(38% - 10px);
width: -moz-calc(38% - 10px);
width: -o-calc(38% - 10px);
width: calc(38% - 10px);
margin-right: 10px;
text-align: center;
}
.footer-col-2 {
width: 175px; /*fallback*/
width: -webkit-calc(23.125% - 10px);
width: -moz-calc(23.125% - 10px);
width: -o-calc(23.125% - 10px);
width: calc(23.125% - 10px);
margin-right: 10px
width: -webkit-calc(24% - 10px);
width: -moz-calc(24% - 10px);
width: -o-calc(24% - 10px);
width: calc(24% - 10px);
margin-right: 10px;
text-align: center;
}
.footer-col-3 {
width: 335px; /*fallback*/
width: -webkit-calc(41.875%);
width: -moz-calc(41.875%);
width: -o-calc(41.875%);
width: calc(41.875%);
width: 302px; /*fallback*/
width: -webkit-calc(38%);
width: -moz-calc(38%);
width: -o-calc(38%);
width: calc(38%);
text-align: center;
}

.site-footer ul { list-style: none; }
Expand Down Expand Up @@ -196,7 +199,7 @@ a:visited { color: #205caa; }
/* ----------------------------------------------------------*/

.hero {
height: 200px;
min-height: 200px;
text-align: center;
margin-top: 50px;
}
Expand Down
12 changes: 6 additions & 6 deletions index.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,11 @@

<section class="hero">
<h2>
Conheça o OpenStreetMap, o mapa livre do mundo
Conheça o OpenStreetMap, o mapa do mundo livre
Copy link
Collaborator

Choose a reason for hiding this comment

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

Pessoalmente achei as duas formas insatisfatórias.
Que tal "o mapa mundial editável e livre" ?

</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.

Qualquer pessoa pode editar o <a href="http://www.openstreetmap.org/?locale=pt-BR#map=4/-15.19/-51.31" target="_blank">mapa</a>. Os dados são redistribuídos sob licença <a href="http://www.openstreetmap.org/copyright?locale=pt-BR" target="_blank">ODbl</a>.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Colocar "[..]sobre a licença[..]" e acho melhor manter o "[..] e os dados [..]" em vez de separar em duas frases

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK, verei como fazer. Mas o termo correto é "sob".

Copy link
Collaborator

Choose a reason for hiding this comment

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

Ah sim, foi erro meu. Pode deixar "sob"

</p>
</section>

Expand All @@ -20,17 +20,17 @@ <h3>
</h3>
<p>
<a href="/aprenda/">
Criar uma conta e comece a explorar o universo do OpenStreetMap →
Crie uma conta e comece a mapear →
Copy link
Collaborator

Choose a reason for hiding this comment

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

Me parece que "explorar o universo do OpenStreetMap" fica mais interessante.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Semanticamente eu concordo em parte, apesar de não gostar de repetir a palavra OpenStreetMap mais uma vez. O negócio é que eu tentei deixar a expressão mais curta... para a arrumação na tela grande, assim como com aquela expressão "Associação de mapeadores".

Mas eu vou tentar alterar essas coisas no sentido que você propôs.

</a>
</p>
</div>
<div class="teaser col-1-2">
<h3>
Participe da comunidade
Participe da comunidade!
Copy link
Collaborator

Choose a reason for hiding this comment

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

Fica meio inconsistente com os outros. Acho melhor retirar.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Do que você fala? O sinal de interrogação? Eu não acho. Você faz questão?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Sim, o sinal de exclamação, mas não faço questão não.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Então esse eu vou deixar. Não vejo como inconsistente. Faz par com o título do teaser ao lado.

</h3>
<p>
<a href="/comunidade/">
Conheça os canais de conversa da comunidade brasileira →
Conheça os canais de conversa brasileiros →
Copy link
Collaborator

Choose a reason for hiding this comment

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

Estava melhor antes, por mencionar a comunidade.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

É, eu concordo. Estou reunindo todos os apontamento em novo(s) commit(s).

</a>
</p>
</div>
Expand Down