Skip to content

Conversation

@Mitch-At-Work
Copy link
Contributor

This pull request migrates the library, react-virtualizer from FluentUI core.

React-virtualizer provides virtualization using IntersectionObserver. The changes include adding the library to the codebase, setting up its configuration files, defining its build and test processes, and exporting its core components and utilities to match Fluentui-contrib's repo architecture.

Addition of the react-virtualizer library:

  • Library Initialization:

    • Added react-virtualizer to the .github/CODEOWNERS file for ownership tracking.
    • Created a README.md file explaining the purpose, benefits, and usage of the react-virtualizer library.
    • Added a package.json file specifying dependencies, peer dependencies, and metadata for the library.
  • Configuration Files:

    • Added .babelrc, .swcrc, and tsconfig.json files for build and compilation configurations. [1] [2] [3]
    • Configured ESLint with custom rules and parser settings for the library. [1] [2]
    • Added a jest.config.ts file for unit testing using Jest and SWC.
    • Set up Playwright for component testing with a configuration file and supporting files. [1] [2] [3]
  • Storybook Integration:

    • Configured Storybook for the library with main.ts and preview.tsx files. [1] [2]

Core functionality and exports:

  • Hooks and Utilities:

    • Exported hooks like useDynamicVirtualizerMeasure and utilities like scrollToItemDynamic for implementing virtualization features. [1] [2]
  • Components:

    • Exported core components such as Virtualizer and VirtualizerScrollView along with their types, styles, and utilities. [1] [2]
  • Project Configuration:

    • Added a project.json file defining build, lint, test, and Storybook tasks for the library.

@Mitch-At-Work Mitch-At-Work requested a review from a team as a code owner July 23, 2025 21:00
@dmytrokirpa dmytrokirpa self-requested a review July 24, 2025 12:09
@@ -0,0 +1,24 @@
{
"name": "@fluentui-contrib/react-virtualizer",
"version": "0.0.1",
Copy link
Contributor

Choose a reason for hiding this comment

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

lets start with "private": true and make public once ready for release

@Hotell
Copy link
Contributor

Hotell commented Jul 24, 2025

pipeline is failing on formatting issues, it seems we have different prettier setup in comparison with core repo.

please run yarn nx format

@Hotell Hotell changed the title Migrate React-Virtualizer into fluentui-contrib feat(react-virtualizer): migrate package from fluentui core repo Jul 24, 2025
@Hotell
Copy link
Contributor

Hotell commented Jul 24, 2025

just to confirm, it seems you used our generators to bootstrap a new package right ?

nit: our recommended flow is to bootstrap empty package 1st / add content as follow up PR -> https://github.com/microsoft/fluentui-contrib/blob/main/CONTRIBUTING.md#creating-a-new-package

@Hotell Hotell self-requested a review July 24, 2025 16:23
Copy link
Contributor

@Hotell Hotell left a comment

Choose a reason for hiding this comment

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

in general LGTM, needs pipeline fixes

@Mitch-At-Work Mitch-At-Work merged commit 19598fb into main Jul 24, 2025
8 checks passed
@Mitch-At-Work Mitch-At-Work deleted the user/mifraser/migrate-virtualizer branch July 24, 2025 21:45
marcosmoura added a commit to marcosmoura/fluentui-contrib that referenced this pull request Sep 8, 2025
…matic-dnd

* main: (211 commits)
  fix(docsite): add missing dependency for extract-storybook-llms target (microsoft#486)
  feat(react-19-tests): bootstrap react 19 integration tests (microsoft#468)
  feat(docsite): generate llms.txt (microsoft#476)
  docs(react-virtualizer): Update virtualizer stories to use the same storybook Meta format as contrib repo (microsoft#474)
  chore: bump react-components to latest and adjust types to support React 18 (microsoft#387)
  chore(deps-dev): bump @swc/core from 1.11.24 to 1.13.3 (microsoft#463)
  chore(deps-dev): bump patch-package from 7.0.0 to 7.0.2 (microsoft#465)
  chore(deps-dev): bump @swc/cli from 0.7.7 to 0.7.8 (microsoft#467)
  chore: migrate to nx 21.3.10 (microsoft#453)
  ci(dependabot): ignore major bumps (microsoft#461)
  chore(deps-dev): bump eslint-config-prettier from 10.1.5 to 10.1.8 (microsoft#460)
  chore(deps-dev): bump react-virtualized-auto-sizer and @types/react-virtualized-auto-sizer (microsoft#454)
  chore(deps): bump nrwl/nx-set-shas from 4.3.0 to 4.3.3 (microsoft#456)
  applying package updates
  feat(react-virtualizer): initial/stable release (microsoft#452)
  docs(gh): add issue templates (microsoft#442)
  feat(react-virtualizer): migrate package from fluentui core repo (microsoft#441)
  feat(react-gamepad-navigation): adding unit tests (microsoft#443)
  fix(react-gamepad-navigation): handling Gamepad API access denied (microsoft#416)
  fix(react-keytips): remove default event prevention in useHotkeys (microsoft#430)
  ...
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.

3 participants