devhatt / octopost

Octopost is an open source service from Devhat that was created to guide new developers for free
https://devhatt.github.io/octopost/
MIT License
76 stars 50 forks source link

fix: replace tag button with button component #423

Closed JpBurgarelli closed 6 months ago

JpBurgarelli commented 7 months ago

Closes #372

Feature Essa task é um refactor, portanto o objetivo dela é fazer ajustes pontuais no antigo SavBar que agora passou a ser chamado de ActionBar. Além da mudança do nome, foi excluido, também, o SubmitBotton. Por fim, foram realizadas pequenas alterações no module.scss desse componente para ficar em harmonia as novas diretivas do projeto, como o path alias. Additional info Acho que esse componente não precisa de evidências visuais.
github-actions[bot] commented 7 months ago

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 36.38% 131 / 360
🔵 Statements 37.69% 147 / 390
🔵 Functions 43.13% 66 / 153
🔵 Branches 47.72% 63 / 132
File Coverage
File Stmts % Branch % Funcs % Lines Uncovered Lines
Changed Files
src/components/ActionBar/ActionBar.tsx 100% 100% 0% 100%
src/pages/home/home.tsx 0% 100% 0% 0% 17-19, 30, 36
Generated in workflow #1013
JpBurgarelli commented 7 months ago

Essa task é um refactor e o ultimos commits trataram a respeito da mudança do nome (antigo SavBar) que agora passou a ser chamado de ActionBar. Além da mudança do nome, foi excluido, também, o SubmitBotton. Por fim, foram realizadas pequenas alterações no module.scss desse componente para ficar em harmonia as novas diretivas do projeto, como o path alias.

JpBurgarelli commented 7 months ago

Os botões continuam com a MESMA identidade visual, conforme projeto do Figma. O que foram alterados desde o ultimo push foi: adição dos buttons, nomes para as classes e buttons, pequenas estilizações no scss. Esse componente também era importado na Home e "envolvido" por um container, essa Container foi deletado também.

image

JpBurgarelli commented 7 months ago

Como o componente que agora chama ActionBar esta sendo importado dentro da Home e sendo envelopado por uma div, então, automaticamente essa Div se torna o wrapper do componente ActionBar, por isso tirei o max-width: 101rem, porque, agora, é usada div pai para delimitar o tamnho não há necessidade de eu manter o max-width, seria redudante.

image