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

DatePicker with defaultValue persists time zone #7069

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

Conversation

boutahlilsoufiane
Copy link
Contributor

@boutahlilsoufiane boutahlilsoufiane commented Sep 23, 2024

Closes #6005

✅ Pull Request Checklist:

  • Included link to corresponding React Spectrum GitHub Issue.
  • Added/updated unit tests and storybook for this change (for new code or code which already has tests).
  • Filled out test instructions.
  • Updated documentation (if it already exists for this component).
  • Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

🧢 Your Project:

ktabors and others added 30 commits March 21, 2024 14:04
* testing theme change and webkit-highlight-color change

* Fix iOS tap highlight

* moving webkit highlight to theme

* forgot to remove extra tsignore
* css rename to styles

* fix comment
* fix SearchFieldProps children type

* remove isEntering and isExiting from TooltipProps

* omit more props from Tooltip

* remove open props from Tooltip

Co-authored-by: Reid Barber <[email protected]>
* setup storybook docs

* add intro page

* Remove Picker autodocs

* add syntax highlighting to code snippet

* add missing autodocs

* Remove intro docs for now

* remove extra args from Dialog

* clean up source formatting

Co-authored-by: Reid Barber <[email protected]>
Co-authored-by: Rob Snow <[email protected]>
Co-authored-by: Devon Govett <[email protected]>
* starting a rollup add to test the built files

* rainbow and macros installed and working

Co-authored-by: Kyle Taborski <[email protected]>
* Add vite example

* Add svgs and fix config
…e for invalid validation (adobe#115)

* revert Field change when invalid

* Update Modal and Popover so corner with scrollbar is square
* update css prop in InlineAlert

* add webpack 5 example

* remove Provider

* use render from react-dom

* add gitignore

* cleanup package.json

* fix React import

* add icon to button

Co-authored-by: Reid Barber <[email protected]>
* add menu and actionmenu

* fix stories for actionbutton and checkboxgroup

* fix actionbutton resizing story

* Remove styles prop from code output

* remove handlers from code output

* add // code: null to config

* fix StaticColorDecorator

* cleanup

* improve dialog example

* add global padding to stories

* remove css file

* remove role from DialogProps type

* improve ActionButton story

* use single quotes for jsdoc @default

* cleanup Dialog stories and code output

Co-authored-by: Reid Barber <[email protected]>
* add storybook logo

* Use italics for tagline

Co-authored-by: Reid Barber <[email protected]>
* fix show code button z-index

* fix to use isolation: isolate instead

Co-authored-by: Reid Barber <[email protected]>
* Fix chromatic publish to storybook

this prevents the Intro page from being sent to Chromatic, looks like it causes issues for them

* Update change so it is isolated to chromatic run
snowystinger
snowystinger previously approved these changes Sep 26, 2024
@snowystinger
Copy link
Member

Looks like you might need to sign the CLA? https://react-spectrum.adobe.com/contribute.html#contributor-license-agreement

GrantRussell and others added 5 commits September 26, 2024 02:15
* pass through disabledKeys to ActionGroup

* fix linting issues

---------

Co-authored-by: Grant Russell <[email protected]>
* add s2 components to Parcel app

* add components to Webpack example

* add to verdaccio build

* build verdaccio on branch

* try adding yarn config

* fix yarn

* add build script to parcel all

* typo in path

* enable comment on PR

* comment on PR

* try removing typekits

* update webpack render to createRoot

* try removing TagGroup

* remove TagGroup from example apps

* add branch filters back

* update example app S2 versions to be "latest"

* add TagGroup back

* remove branch filters

* Revert "remove branch filters"

This reverts commit b7b54fe.

* Fix style publishing

* turn on verdaccio

* Revert "turn on verdaccio"

This reverts commit 3be5f9c.

---------

Co-authored-by: GitHub <[email protected]>
* Fix ComboBox and Picker getting stuck in down state

* Fix table types and export from package
@boutahlilsoufiane
Copy link
Contributor Author

boutahlilsoufiane commented Sep 26, 2024

@snowystinger thanks for the review, I have just signed the CLA.

@snowystinger
Copy link
Member

Thank, I'll close and reopen as it looks like it's stuck at the moment.

snowystinger and others added 8 commits September 26, 2024 18:17
* Remove extraneous hover props from s2

* Fix ts

* use the correct prop name
* S2 Table optimizations

* fix merge
* Add some missing docs descriptions

* Rename strange interface in props

* Fix sizes of pending button spinner

* Add docs for --tree-item-level custom property in Tree

* fix ts

* Require children for disclosures

* Make it so Accordion isn't labellable

* fix button typescript

* narrow test utils exports and future proof

* Remove isPending from ActionButton

---------

Co-authored-by: Robert Snow <[email protected]>
Co-authored-by: GitHub <[email protected]>
* disable snapshot for regular stories, remove s2 from title

* add chromatic stories from missing components

* fix lint

* add more chromatic stories for missing components and remove s2 prefix

* remove disable snapshot, update chromatic-fc

* Some docs improvements

---------

Co-authored-by: Devon Govett <[email protected]>
* S2 chromatic table

* fix cardview chromatic stories
* Export pressScale and focusRing utilities from S2

* Also export StyleString type
* Rename SegmentedControl props to match selection APIs

* fix chromatic stories

* ts strict

* Add AriaLabelingProps

* Pass value and defaultValue to RadioGroup

* fix merge conflicts
* add disclosure tests

* lint/cleanup

* Pass isExpanded to useDisclosure as well

---------

Co-authored-by: Robert Snow <[email protected]>
Co-authored-by: Devon Govett <[email protected]>
@snowystinger
Copy link
Member

trying again to get it unstuck

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.

Date pickers lose time zone when using the calendar after values are cleared