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

Raquel #2

Open
wants to merge 15 commits into
base: main
Choose a base branch
from
Open

Raquel #2

wants to merge 15 commits into from

Conversation

raquellenn
Copy link
Collaborator

No description provided.

Tentando fazer o quadrado entrar pro final da linha, vocês tem alguma sugestão?
Copy link
Member

@svallory svallory left a comment

Choose a reason for hiding this comment

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

Muito bom! Não se assuste por ter alguns comentários, isso é comum, mesmo pra desenvolvedores experientes :)

Parabéns pelo avanço! Essa página vai ficar pública, na internet. Então antes da gente fazer o merge, coloca seus dados reais aí ;)

Raquel/index.html Outdated Show resolved Hide resolved
Raquel/index.html Outdated Show resolved Hide resolved
Comment on lines +1 to +14
@font-face {
font-family: aller;
src: url(fontes/Aller_rg.ttf);
}
@font-face {
font-family: aller-lt;
src: url(fontes/Aller_Lt.ttf);
}

@font-face {
font-family: aller-Ltlt;
src: url(fontes/Aller_LtIt.ttf);
}

Copy link
Member

Choose a reason for hiding this comment

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

Tem mais fontes no repositório do que você está usando. Remova os arquivos de fonte que você não usou

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Eu usei as 3! seção body: aller_rg; seção h3 aller-lt e seção h5 aller-ltit

Copy link
Member

Choose a reason for hiding this comment

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

@raquellenn no css tem as 3 que vc usa, mas na pasta fontes tem 7 arquivos de fonte. Remova os arquivos que vc não usou

Raquel/style.css Outdated Show resolved Hide resolved
Raquel/style.css Outdated Show resolved Hide resolved
Falta arrumar os quadrados e algumas linhas
tentando arrumar o quadrado e algumas outras pequenas alterações
Copy link
Member

@svallory svallory left a comment

Choose a reason for hiding this comment

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

Já está bom! Eu estou sendo chato de propósito e me apegando a detalhes pra garantir que vc está entendo os conceitos. Na vida real, provavelmente, não se importariam com como vc usou as tags H, por exemplo.

Comment on lines +1 to +14
@font-face {
font-family: aller;
src: url(fontes/Aller_rg.ttf);
}
@font-face {
font-family: aller-lt;
src: url(fontes/Aller_Lt.ttf);
}

@font-face {
font-family: aller-Ltlt;
src: url(fontes/Aller_LtIt.ttf);
}

Copy link
Member

Choose a reason for hiding this comment

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

@raquellenn no css tem as 3 que vc usa, mas na pasta fontes tem 7 arquivos de fonte. Remova os arquivos que vc não usou

<h5>lugar e data</h5>
<p>Lorem ipsum dolor sit amet consectetur adipisicing elit. Quasi aliquam eius dignissimos asperiores consequatur unde ad eos architecto quam. Sint deserunt harum ullam porro delectus nam at quidem distinctio laborum.</p>
<h4>Assistente de Produção de Objeto</h4>
<h5>As Orfãs da Rainha / mar 2019 - mar 2020</h5>
Copy link
Member

Choose a reason for hiding this comment

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

Pensa nos elementos h1, h2, etc como título, subtítulo, etc. Quando vc escreve um TCC, por exemplo, vc tem
h1 - titulo do trabalho
h2 - capítulo
h3 - título
h4 - subtítulo
As tags <h_> devem ser usadas pra indicar títulos e subtítulos do conteúdo dessa forma.

Você pulou do h2 pro h4 e acho que usou o h5 pra dar destaque a um parágrafo

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Então, mas é que o h3 é o subtítulo do título do currículo. é menor do que os títulos das seções (tipo perfil, habilidades) mas maior que os títulos dos trabalhos. vc acha que deveria usar em ordem independente do tamanho?

Copy link
Member

Choose a reason for hiding this comment

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

A questão aqui é que você pulou do h2 pro h4

Copy link
Member

@svallory svallory left a comment

Choose a reason for hiding this comment

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

Só tem a questão dos headers que vão do 2 pro 4 sem passar pelo 3. Bom trabalho!

<h5>lugar e data</h5>
<p>Lorem ipsum dolor sit amet consectetur adipisicing elit. Quasi aliquam eius dignissimos asperiores consequatur unde ad eos architecto quam. Sint deserunt harum ullam porro delectus nam at quidem distinctio laborum.</p>
<h4>Assistente de Produção de Objeto</h4>
<h5>As Orfãs da Rainha / mar 2019 - mar 2020</h5>
Copy link
Member

Choose a reason for hiding this comment

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

A questão aqui é que você pulou do h2 pro h4

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