Skip to content
This repository has been archived by the owner on May 28, 2023. It is now read-only.

Activity component #27

Merged
merged 4 commits into from
Oct 18, 2022
Merged

Conversation

tarsojabbes
Copy link
Member

Issue

#21

Qual o objetivo dessa Pull Request?

Essa PR inclui a implementação do componente Activity que recebe as props titulo e info. A implementação utiliza da biblioteca PropTypes e possui defaultProps para "titulo", além de possuir uma pequena documentação.

Que problema está resolvendo?

#21

Como pode ser manualmente testado?

O componente pode ser importado em alguma página e testado dentro de um layout grid ou flex, como esperado segundo a prototipação.

Copy link
Member

@RodrigoEC RodrigoEC left a comment

Choose a reason for hiding this comment

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

Ta quase tudo topppp, só pedi alguemas mudanças pontuais :3

Ah outra coisa, eu sei que talvez tenha um certo retrabalho, mas se tu puder traduzir o código (a documentação n precisa) pra ingles ia ser melhor pra manter a coerencia com o resto do código

Comment on lines 20 to 29
{props.info ? (
<>
<Divisor />
<Info>
{props.info}
</Info>
</>
) : ""}

</Card>
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
{props.info ? (
<>
<Divisor />
<Info>
{props.info}
</Info>
</>
) : ""}
</Card>
{props.info && (
<>
<Divisor />
<Info>
{props.info}
</Info>
</>
) }
</Card>

* Somente "titulo" é obrigatória, "info" é opcional.
* @returns
*/
const Activity = (props) => {
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
const Activity = (props) => {
const Activity = ({ titulo, info}) => {

Com isso a gente n precisa sempre fazer "props.atributo"

flex-grow: 1;
justify-content: center;
align-items: center;
width: 22.5em;
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
width: 22.5em;
width: 22.5rem;

Amg muda pf todos as medidas de em pra rem pf, pq em eh uma medida relativa, enquanto rem é a medida a partir da medida raiz

width: 22.5em;
min-height: fit-content;
height: 100%;
padding: 1.25em 1.25em;
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
padding: 1.25em 1.25em;
padding: 2em 1.25em;

@RodrigoEC RodrigoEC merged commit c6d76ab into OpenDevUFCG:develop Oct 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants