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

Issue 27 - Componente de Calendário #302

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

Conversation

DominMFD
Copy link
Contributor

@DominMFD DominMFD commented Sep 8, 2024

Closes #27

Feature

Foi criado o componente de calendário

Visual evidences 🖼️

image

Calendario.pra.frente.mp4
Calenario.para.tras.mp4

https://github.com/use
Captura de tela 2024-09-08 102034
r-attachments/assets/59105c54-daf3-445a-bf67-b395a605add0

Calendario.mobile.mp4
Checklist
  • Issue linked
  • Build working correctly
  • Tests created

Copy link
Member

@Luiginicoletti Luiginicoletti left a comment

Choose a reason for hiding this comment

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

Mano maravilhoso, nem em toda minha vida eu conseguiria fazer isso na mão, parabens!

high-five-way-to-go-gif-by-smiledirectclub

src/components/Button/index.scss Outdated Show resolved Hide resolved
src/components/Button/index.scss Show resolved Hide resolved
src/styles/colors.scss Show resolved Hide resolved
Copy link
Contributor

@juliaam juliaam left a comment

Choose a reason for hiding this comment

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

Tá quebrando o calendário em certos tamanhos, além disso, nesse mesmo problema, parece que os anos e os meses ficam andando muito rápido e isso buga a exibição, mas por print não parece

image

Seria legal também ter uma corzinha indicando o dia atual, mesmo que seja selecionado outro dia, isso resultará em uma navegação melhor

tipo assim:
image

Quando você clica nos dias finais do mês anterior, ele seleciona do mês atual, ele deveria selecionar o dia daquele mês passado e a exibição ser do mês atual
image

src/layouts/components/SideMenu/index.scss Show resolved Hide resolved
src/components/Calendar/index.js Outdated Show resolved Hide resolved
Copy link
Contributor

@muritadb muritadb left a comment

Choose a reason for hiding this comment

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

capibara

@gustavogularte
Copy link
Contributor

Chega um momento que o scroll para de funcionar

Também acho que deveria ser possível selecionar os meses ou os anos clicando e não apenas no scroll até pq o scroll ta meio difícil de controlar e acaba indo de 3 em 3 ou mais

review.mp4

}

const totalDaysInMonth = dayjs(
`${this.year}-${this.month}-1`,
Copy link
Contributor

Choose a reason for hiding this comment

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

Não precisa do -1

this.year = year;

this.mountDayComposer();
this.firstDayInWeek = dayjs(`${this.year}-${this.month}-${this.day}`).day();
Copy link
Contributor

Choose a reason for hiding this comment

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

pq se chama firstDayInWeek sendo que ele não retorna o primeiro dia da semana e sim a posição que o dia selecionado está na semana?

}

const totalDaysInMonth = dayjs(
`${this.year}-${this.month}-1`,
Copy link
Contributor

Choose a reason for hiding this comment

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

Não precisa do -1

@use '~styles/base.scss';
@use '~styles/colors.scss' as colors;
@use '~styles/fonts.scss' as fonts;
@use '~styles/breakpoints.scss' as breakpoints;
Copy link
Contributor

Choose a reason for hiding this comment

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

É preciso importar tudo isso mesmo não usando?

@@ -0,0 +1,35 @@
@use '~styles/base.scss';
@use '~styles/colors.scss' as colors;
@use '~styles/fonts.scss' as fonts;
Copy link
Contributor

Choose a reason for hiding this comment

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

É preciso importar tudo isso mesmo não usando?

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.

Crianção do componente de calendário
6 participants