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

Updating my fork with changes #1

Merged
merged 18 commits into from
Sep 17, 2020
Merged

Conversation

eljefe223
Copy link
Owner

Pull request checklist

  • Addresses an existing issue: Fixes #0000
  • Include a change request file using $ yarn change

Description of changes

(give an overview)

Focus areas to test

(optional)

kolaps33 and others added 18 commits September 16, 2020 18:32
* fixing navigation into cell which has multiple focusable elements

* updating changelog

* adding table keyboard navigation tests

* deleting from master

* adding table keyboard navigation tests

* deleting files from master, pushed by mistaked

* adding packages needed for a11y testing and generate documentation for behavior files

* Update packages/a11y-testing/LICENSE

Co-authored-by: Elizabeth Craig <ecraig12345@gmail.com>

* Update packages/a11y-testing/package.json

Co-authored-by: Elizabeth Craig <ecraig12345@gmail.com>

* improving based on review

* adressing comment regarding jest package

* impoving dependency package

Co-authored-by: Milan Turon <mituron@microsoft.com>
Co-authored-by: Elizabeth Craig <ecraig12345@gmail.com>
…unction component (#15054)

* Moving breadcrumb to react-next

* Reverting an accidental file.

* Change files

* Updating the api

* Updating example snapshots.
* changes

* undo

* Change files

* typing updates.

* fixing build break.
* Floating Pickers: Move to react-next

* update the api file
* Feat: convert card to be design system provider

* Change files
* remove un-used prop

* impl applyTo

* Change files

* fixes

* merge

* resolve comment

* update api

* call useFocusRects

* resolve comment

* udpate api
* fix: elementContains should behave like .contains()

* Change files
* style fixes

* update snapshots

* fix split button styles

* more fixes

* revert flow-grow and fix ghost hover/press background

* cleanup

* resolve comment

* fix ghost disabled background & fluid compound

* merge
* Add new package for spinbutton v8

* Adding src folder and empty index file.

* add version.ts files

* add @uifabric/set-version as a dependency

* change to react and add config folder

* Add package to CODEOWNERS file and update api-extractor.json

* update dependency version

* add etc folder

Co-authored-by: KHMakoto <humbertomakotomorimoto@gmail.com>
* Migrate ComboBox to react-next

* Change files

* Migrate ComboBoxOptionWrapper

* Add stub function component

* Disable tests that fail to compile for now

* Hoist default props

* Finalize partial migration

* Test fixes in progress

* Fix most tests

* Cleanup tests

* Fix more test cases

* Change files

* Fix build

* Revert unintended changes

* Revert unintended change

* Fix lint error

* Address feedback

* Update ref types

Co-authored-by: Elizabeth Craig <elcraig@microsoft.com>
* Adding compat exports to v7

* Change files

* Removing toggle, slider, pivot, link, and checkbox from @fluentui/react compat exports.

* Delete Toggle.ts

* Delete Slider.ts

* Delete Pivot.ts

* Delete Checkbox.ts

* Delete Link.ts

* Reverting an unintended change.

* Updating react-next compat index file

* Removing react-next's compat exports for toggle, slider, pivot, link, and checkbox
Upload images used in screener and other places to our CDN that we use for the website (http://fabricweb.azureedge.net/fabric-website/*). This will make them load faster than if we uploaded them to normal blob storages, which can be important for getting correct results in screener tests.

For testing purposes I have the images uploading as part of the PR build, but before check-in I plan to change it so the images upload **only on master builds** (mainly to avoid accidental changes). This means that if you want to use a new image, you'll have to check it in separately first, but that shouldn't be a big issue since it won't happen very often.

Also update other upload tasks to use the new AzureUpload task which gzips things. This should make the PR deploy site load faster.
* Replaced "drop-down menu" with "dropdown"

We're deviating from the Microsoft Writing Style Guide guidance (hyphenating drop-down), because compounded (dropdown) style is more commonly seen across other industry-leading design systems.

* Added explanation of when to use a dropdown

* Removed "list" from "dropdown list" references

* Removed erroneous guidance about form fields

Somehow guidance got checked in that doesn't map to our best practices. Correcting that, so we ensure everyone standardizes on the uses of asterisks for denoting required fields. And added a reminder note about ARIA properties to remind folks to consider the screenreader experience.

* Fix aria-required guidance

* Change files

* updates

Co-authored-by: jennbost <45047773+jennbost@users.noreply.github.com>
* Updating release notes for button.

* Button release notes

* Change files

* Update packages/react-next/RELEASE_NOTES.md

Co-authored-by: Makoto Morimoto <Humberto.Morimoto@microsoft.com>

* Update packages/react-next/RELEASE_NOTES.md

Co-authored-by: Makoto Morimoto <Humberto.Morimoto@microsoft.com>

Co-authored-by: Makoto Morimoto <Humberto.Morimoto@microsoft.com>
@eljefe223 eljefe223 merged commit 8912d32 into eljefe223:master Sep 17, 2020
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.

10 participants