Merge pull request #573 from pixijs/v8-release

v8 Pre-Release Changes

Author: trezy

src/components/Application.tsx

@@ -137,7 +137,7 @@ const ApplicationImplementation = forwardRef<ApplicationRef, ApplicationProps>(f
 
             if (!root)
             {
-                root = createRoot(canvasElement, {}, handleInit);
+                root = createRoot(canvasElement, { onInit: handleInit });
             }
 
             // @ts-expect-error The value of `children` is fine, but `PixiReactChildNode` doesn't strictly adhere to the `ReactNode` structure.

src/core/createRoot.tsx

@@ -20,12 +20,6 @@ export function createRoot(
 
     /** @description Options to configure the tree. */
     options: CreateRootOptions = {},
-
-    /**
-     * @deprecated
-     * @description Callback to be fired when the application finishes initializing.
-     */
-    onInit?: (app: Application) => void,
 )
 {
     // Check against mistaken use of createRoot
@@ -93,7 +87,7 @@ export function createRoot(
                 applicationState.isInitialising = false;
                 applicationState.isInitialised = true;
                 applicationState = { ...applicationState };
-                (options.onInit ?? onInit)?.(applicationState.app);
+                options.onInit?.(applicationState.app);
             }
 
             Object.entries(applicationOptions).forEach(([key, value]) =>

src/hooks/useApp.ts

@@ -8,12 +8,6 @@ import { useIsomorphicLayoutEffect } from './useIsomorphicLayoutEffect';
 export function useTick<T>(
     /** @description The function to be called on each tick. */
     options: TickerCallback<T> | UseTickOptions<T>,
-
-    /**
-     * @deprecated
-     * @description Whether this callback is currently enabled.
-     */
-    enabled = true,
 )
 {
     const {
@@ -25,7 +19,7 @@ export function useTick<T>(
 
     let context: any;
 
-    let isEnabled = enabled;
+    let isEnabled: boolean = true;
 
     let priority: number | undefined;
 
@@ -69,7 +63,6 @@ export function useTick<T>(
     }, [
         callback,
         context,
-        isEnabled,
         isInitialised,
         priority,
     ]);

src/hooks/useTick.ts

@@ -1,15 +1,7 @@
-console.warn(`
-Be aware that you are using a beta version of Pixi React.
-- Things may be broken.
-- Things may (but shouldn't) change.
-- All functionality that's deprecated in the beta version WILL BE REMOVED for the production release.
-`);
-
 export { Application } from './components/Application';
 export { createRoot } from './core/createRoot';
 export * from './global';
 export { extend } from './helpers/extend';
-export { useApp } from './hooks/useApp';
 export { useApplication } from './hooks/useApplication';
 export { useExtend } from './hooks/useExtend';
 export { useTick } from './hooks/useTick';

src/index.ts

@@ -13,12 +13,6 @@ import { type PixiReactChildNode } from './PixiReactChildNode';
 
 export interface BaseApplicationProps
 {
-    /**
-     * @description Whether this application chould be attached to the dev tools. NOTE: This should only be enabled on one application at a time.
-     * @deprecated Pixi.js handles this automatically, making this option superfluous.
-     */
-    attachToDevTools?: boolean
-
     /** @description CSS classes to be applied to the Pixi Application's canvas element. */
     className?: string
 

src/typedefs/ApplicationProps.ts

@@ -6,7 +6,6 @@ import {
 import { Application } from '../../src/components/Application';
 import { createRoot } from '../../src/core/createRoot';
 import { extend } from '../../src/helpers/extend';
-import { useApp } from '../../src/hooks/useApp';
 import { useApplication } from '../../src/hooks/useApplication';
 import { useExtend } from '../../src/hooks/useExtend';
 import { useTick } from '../../src/hooks/useTick';
@@ -32,12 +31,6 @@ describe('exports', () =>
         expect(PixiReact.extend).toEqual(extend);
     });
 
-    it('exports the `useApp()` hook', () =>
-    {
-        expect(PixiReact).toHaveProperty('useApp');
-        expect(PixiReact.useApp).toEqual(useApp);
-    });
-
     it('exports the `useApplication()` hook', () =>
     {
         expect(PixiReact).toHaveProperty('useApplication');
@@ -61,7 +54,6 @@ describe('exports', () =>
         expect(PixiReact).toHaveProperty('Application');
         expect(PixiReact).toHaveProperty('createRoot');
         expect(PixiReact).toHaveProperty('extend');
-        expect(PixiReact).toHaveProperty('useApp');
         expect(PixiReact).toHaveProperty('useApplication');
         expect(PixiReact).toHaveProperty('useExtend');
         expect(PixiReact).toHaveProperty('useTick');