Skip to content

feat(#3815): Added the size property to Date Picker - #4355

Merged
ArakTaiRoth merged 1 commit into
devfrom
dustin/3815
Oct 16, 2026
Merged

ArakTaiRoth merged 1 commit into
devfrom
dustin/3815

Conversation

@ArakTaiRoth

Copy link
Copy Markdown
Collaborator

Before (the change)

There was a size property supported by the Date Picker component, but it didn't exist for the Angular and React wrappers, and so was unusable by the vast majority of our users.

After (the change)

size now added to Date Picker Angular and React wrappers. Examples have been created in documentation, and on the PR docs page for Date Picker.

Make sure that you've checked the boxes below before you submit the PR

  • I have read and followed the setup steps
  • I have created necessary unit tests
  • I have tested the functionality in both React and Angular.

Steps needed to test

Check out the documentation page for Date Picker, verify the properties table for both Angular and React, and check the new Sizes example on the main documentation page. Also can check the docs PR page for Date Picker for both Angular and React.

@ArakTaiRoth
ArakTaiRoth requested a review from twjeffery October 14, 2026 19:34
@github-actions

github-actions Bot commented Oct 14, 2026 •

Copy link
Copy Markdown
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-10-16 19:39 UTC

@github-actions

github-actions Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

Preview removed

All preview folders cleaned from gh-pages branch.

@twjeffery twjeffery left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@ArakTaiRoth Looks good. One thing to change, the space between the inputs should be --goa-space-l for the default size and --goa-space-m for compact.

@ArakTaiRoth
ArakTaiRoth merged commit 63fa390 into dev Oct 16, 2026
6 checks passed
@ArakTaiRoth
ArakTaiRoth deleted the dustin/3815 branch October 16, 2026 19:37
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.

2 participants