Skip to content

Conversation

JoaoFerreira-FrontEnd
Copy link
Contributor

@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd commented Mar 17, 2025

Issue number: Internal

What is the new behavior?

Adds a new property to datetime, showAdjacentDays, that when true will show the last days of the previous month and the first days of the next month. This will just occupy empty "cells" at the beginning of the month "table" and add rows to the table until a maximum of 6 rows are displayed.

Changes

  • add styles for adjacent day button
  • add showAdjacentDays property to datetime component
  • change month generation to respect new property
  • add visual tests to new feature

Does this introduce a breaking change?

  • Yes
  • No

Other information

Preview

JoaoFerreira-FrontEnd and others added 30 commits February 24, 2025 18:44
- add showDaysOutsideCurrentMonth property to datetime  component;
- change month generation to respect new property;
- create specific test page for showDaysOutsideCurrentMonth;
Co-authored-by: Brandy Smith <brandyscarney@users.noreply.github.com>
Co-authored-by: Brandy Smith <brandyscarney@users.noreply.github.com>
…datetime.e2e.ts

Co-authored-by: Brandy Smith <brandyscarney@users.noreply.github.com>
…datetime.e2e.ts

Co-authored-by: Brandy Smith <brandyscarney@users.noreply.github.com>
Co-authored-by: Brandy Smith <brandyscarney@users.noreply.github.com>
…datetime.e2e.ts

Co-authored-by: Brandy Smith <brandyscarney@users.noreply.github.com>
Co-authored-by: Brandy Smith <brandyscarney@users.noreply.github.com>
…datetime.e2e.ts

Co-authored-by: Brandy Smith <brandyscarney@users.noreply.github.com>
Co-authored-by: Brandy Smith <brandyscarney@users.noreply.github.com>
…index.html

Co-authored-by: Brandy Smith <brandyscarney@users.noreply.github.com>
Co-authored-by: Brandy Smith <brandyscarney@users.noreply.github.com>
…datetime.e2e.ts

Co-authored-by: Brandy Smith <brandyscarney@users.noreply.github.com>
…datetime.e2e.ts

Co-authored-by: Brandy Smith <brandyscarney@users.noreply.github.com>
…datetime.e2e.ts

Co-authored-by: Brandy Smith <brandyscarney@users.noreply.github.com>
…datetime.e2e.ts

Co-authored-by: Brandy Smith <brandyscarney@users.noreply.github.com>
…datetime.e2e.ts

Co-authored-by: Brandy Smith <brandyscarney@users.noreply.github.com>
…datetime.e2e.ts

Co-authored-by: Brandy Smith <brandyscarney@users.noreply.github.com>
…datetime.e2e.ts

Co-authored-by: Brandy Smith <brandyscarney@users.noreply.github.com>
…datetime.e2e.ts

Co-authored-by: Brandy Smith <brandyscarney@users.noreply.github.com>
…datetime.e2e.ts

Co-authored-by: Brandy Smith <brandyscarney@users.noreply.github.com>
…datetime.e2e.ts

Co-authored-by: Brandy Smith <brandyscarney@users.noreply.github.com>
Copy link
Member

@brandyscarney brandyscarney left a comment

Choose a reason for hiding this comment

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

Looks good just requested a couple things!

@brandyscarney brandyscarney requested review from thetaPC and removed request for christian-bromann March 17, 2025 19:23
JoaoFerreira-FrontEnd and others added 3 commits March 19, 2025 10:08
Co-authored-by: Maria Hutt <thetaPC@users.noreply.github.com>
Co-authored-by: Maria Hutt <thetaPC@users.noreply.github.com>
Co-authored-by: Maria Hutt <thetaPC@users.noreply.github.com>
JoaoFerreira-FrontEnd and others added 3 commits March 19, 2025 15:44
Co-authored-by: Maria Hutt <thetaPC@users.noreply.github.com>
Co-authored-by: Maria Hutt <thetaPC@users.noreply.github.com>
Co-authored-by: Maria Hutt <thetaPC@users.noreply.github.com>
Copy link
Contributor

@thetaPC thetaPC left a comment

Choose a reason for hiding this comment

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

LGTM

@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd merged commit b67259e into feature-8.6 Mar 19, 2025
48 checks passed
@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd deleted the ROU-11118_V2 branch March 19, 2025 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: angular @ionic/angular package package: core @ionic/core package package: vue @ionic/vue package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants