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

Arquivo SCSS de definições de fontes para o projeto criado. #51

Merged
merged 13 commits into from
Feb 2, 2024

Conversation

DominMFD
Copy link
Contributor

@DominMFD DominMFD commented Jan 11, 2024

Closes #51

Feature

Criado o arquivo font.scss que irá conter todas as definições de fontes do projeto, utilizado mixins e funções reutilizáveis

Bugfix
  • Description
    N/A

  • Cause
    N/A

  • Solution
    N/A

Changelog O arquivo contem variáveis, mixins e funções para criar estilos facilmente aplicáveis e modulares, todos eles comentados explicando sua finalidade e com exemplos de como utilizar ao final do arquivo.
Visual evidences 🖼️
Checklist
  • Issue linked
  • Build working correctly
  • Tests created
Additional info É meu primeiro pull request nesse estilo, qualquer feedback coisa estou disposto a ouvir, tanto do pull request quanto do arquivo da task, qualquer melhoria ou sugestão pode me falar, pretendo seguir no projeto e ser ativo no discord, tenho o objetivo de crescer.

@DominMFD
Copy link
Contributor Author

Por favor me enviei feedbacks do pull request, qualquer variável, mixins e funções para adicionar ou alterar estarei disposto a isso, tentei fazer mixins e funções para os componentes que mais se repetem, utilizei o figma para essa referência.

src/styles/fonts.scss Outdated Show resolved Hide resolved
src/styles/fonts.scss Outdated Show resolved Hide resolved
src/styles/fonts.scss Outdated Show resolved Hide resolved
src/styles/fonts.scss Outdated Show resolved Hide resolved
src/styles/fonts.scss Outdated Show resolved Hide resolved
@Alecell
Copy link
Contributor

Alecell commented Jan 11, 2024

Iae man! No template de PR, as partes que não se encaixam no seu PR vc pode apagar tipo o bugfix e o visual reference!

Na parte ali no inicio "Closes" deve seguir o id da sua issue, por exemplo "Closes #51"

@DominMFD
Copy link
Contributor Author

Muito obrigado pelo ensinamento!

@DominMFD
Copy link
Contributor Author

Fiz a substituição dos mixins pelas variáveis como discutido na daily, qualquer alteração necessária estou a disposição, fiquei na duvida do nome das variáveis, se puder me dar um feedback sobre o nome delas.

src/styles/fonts.scss Outdated Show resolved Hide resolved
@DominMFD
Copy link
Contributor Author

Removi os espaços em branco, irei dar uma olhada sobre as configurações

src/home/index.html Outdated Show resolved Hide resolved
src/styles/fonts.scss Outdated Show resolved Hide resolved
src/styles/fonts.scss Outdated Show resolved Hide resolved
src/styles/fonts.scss Outdated Show resolved Hide resolved
src/styles/fonts.scss Outdated Show resolved Hide resolved
src/styles/fonts.scss Outdated Show resolved Hide resolved
src/styles/fonts.scss Outdated Show resolved Hide resolved
src/styles/fonts.scss Outdated Show resolved Hide resolved
src/styles/fonts.scss Outdated Show resolved Hide resolved
@DominMFD
Copy link
Contributor Author

Ranquei as coisas que eram pra rancar e estou esperando resposta do Eliabe.

@DominMFD
Copy link
Contributor Author

O import das fontes agora estão corretas com as fontes e os pesos, foi tudo revisado com o Eliabe.

src/styles/fonts.scss Outdated Show resolved Hide resolved
@Alecell Alecell merged commit ba5cf2a into devhatt:main Feb 2, 2024
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.

None yet

3 participants