@tpzdsp/next-toolkit 1.2.1 → 1.2.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tpzdsp/next-toolkit",
-  "version": "1.2.1",
+  "version": "1.2.3",
   "description": "A reusable React component library for Next.js applications",
   "type": "module",
   "private": false,

package/src/components/SlidingPanel/SlidingPanel.test.tsx

@@ -18,7 +18,7 @@ const hiddenPositionClasses = {
 describe('SlidingPanel', () => {
   it.each(positions)('should render closed panel by default at %s', (position) => {
     render(
-      <SlidingPanel position={position} tabLabel="Open Test">
+      <SlidingPanel position={position} tabLabel="Test">
         <p>Panel Content</p>
       </SlidingPanel>,
     );
@@ -38,7 +38,7 @@ describe('SlidingPanel', () => {
       const user = userEvent.setup();
 
       render(
-        <SlidingPanel position={position} tabLabel="Open Me">
+        <SlidingPanel position={position} tabLabel="Me">
           <div>My content</div>
         </SlidingPanel>,
       );
@@ -72,7 +72,7 @@ describe('SlidingPanel', () => {
 
     expect(panelContent).toBeVisible();
 
-    await user.click(screen.getByRole('button', { name: 'Close' }));
+    await user.click(screen.getByRole('button', { name: 'Close Trigger' }));
 
     await waitFor(() => {
       expect(panelContent).toBeInTheDocument();

package/src/components/SlidingPanel/SlidingPanel.tsx

@@ -117,7 +117,7 @@ export const SlidingPanel = ({
         style={getButtonStyle}
         onClick={() => setIsVisible((prev) => !prev)}
       >
-        {isVisible ? 'Close' : tabLabel}
+        {isVisible ? `Close ${tabLabel}` : `Open ${tabLabel}`}
       </button>
 
       <div

package/src/map/Map.tsx

@@ -18,6 +18,7 @@ export type MapComponentProps = {
   osMapsApiKey?: string;
   geocoderUrl: string;
   basePath: string;
+  isLoading?: boolean;
 };
 
 const positionTransforms: Record<PopupDirection, string> = {
@@ -41,7 +42,12 @@ const arrowStyles: Record<PopupDirection, string> = {
  *
  * @return {*}
  */
-export const MapComponent = ({ osMapsApiKey, geocoderUrl, basePath }: MapComponentProps) => {
+export const MapComponent = ({
+  osMapsApiKey,
+  geocoderUrl,
+  basePath,
+  isLoading,
+}: MapComponentProps) => {
   const [popupFeatures, setPopupFeatures] = useState([]);
   const [popupCoordinate, setPopupCoordinate] = useState<number[] | null>(null);
   const [popupPositionClass, setPopupPositionClass] = useState<PopupDirection>('bottom-right');
@@ -57,7 +63,6 @@ export const MapComponent = ({ osMapsApiKey, geocoderUrl, basePath }: MapCompone
     setMapConfig,
     map,
     isDrawing,
-    isSamplingPointsLoading,
   } = useMap();
 
   useEffect(() => {
@@ -199,7 +204,7 @@ export const MapComponent = ({ osMapsApiKey, geocoderUrl, basePath }: MapCompone
   return (
     <div className="flex flex-grow min-h-0">
       <div ref={mapRef} className="flex flex-grow relative z-10" id="map">
-        {isSamplingPointsLoading ? (
+        {isLoading ? (
           <div className="absolute inset-0 flex items-center justify-center bg-white/50 z-10">
             <div
               className="w-9 h-9 border-4 border-gray-200 border-t-blue-500 rounded-full

package/src/map/MapContext.tsx

@@ -25,7 +25,7 @@ export type MapContextType = {
   addLayer: (layer: Layer) => void;
   removeLayer: (name: string) => void;
   clearLayer: (name: string) => void;
-  resetMap: () => void;
+  resetMap: (onReset?: () => void) => void;
   getLayerByName: (name: string) => BaseLayer | undefined;
   aoi: Coordinate[][] | null;
   setAoi: Dispatch<SetStateAction<Coordinate[][] | null>>;
@@ -33,12 +33,6 @@ export type MapContextType = {
   setSelectedLayer: Dispatch<SetStateAction<Layer | null>>;
   isDrawing: boolean;
   setIsDrawing: Dispatch<SetStateAction<boolean>>;
-  samplingPoints: string[] | null;
-  setSamplingPoints: Dispatch<SetStateAction<string[] | null>>;
-  isTooManySamplingPoints: boolean;
-  setIsTooManySamplingPoints: Dispatch<SetStateAction<boolean>>;
-  isSamplingPointsLoading: boolean;
-  setIsSamplingPointsLoading: Dispatch<SetStateAction<boolean>>;
 };
 
 type MapProviderProps = {
@@ -66,9 +60,6 @@ export const MapProvider = ({ initialState = {}, children }: MapProviderProps) =
   const [aoi, setAoi] = useState<Coordinate[][] | null>(null);
   const [selectedLayer, setSelectedLayer] = useState<Layer | null>(null);
   const [isDrawing, setIsDrawing] = useState(false);
-  const [samplingPoints, setSamplingPoints] = useState<string[] | null>(null);
-  const [isTooManySamplingPoints, setIsTooManySamplingPoints] = useState(false);
-  const [isSamplingPointsLoading, setIsSamplingPointsLoading] = useState(false);
 
   const getLayers = useCallback(() => map?.getLayers().getArray(), [map]);
 
@@ -126,30 +117,32 @@ export const MapProvider = ({ initialState = {}, children }: MapProviderProps) =
     [getLayerByName],
   );
 
-  const resetMap = useCallback(() => {
-    if (map) {
-      const view = map.getView();
-
-      // Animate view to default
-      view.animate(
-        {
-          center: fromLonLat(DEFAULT_LON_LAT),
-          duration: DEFAULT_DURATION,
-        },
-        {
-          zoom: DEFAULT_ZOOM,
-          duration: DEFAULT_DURATION,
-        },
-      );
-    }
-
-    // Remove all defined layers
-    Object.values(LAYER_NAMES).forEach(clearLayer);
-
-    // Reset sampling points state
-    setSamplingPoints(null);
-    setIsTooManySamplingPoints(false);
-  }, [map, clearLayer]);
+  const resetMap = useCallback(
+    (onReset?: () => void) => {
+      if (map) {
+        const view = map.getView();
+
+        // Animate view to default
+        view.animate(
+          {
+            center: fromLonLat(DEFAULT_LON_LAT),
+            duration: DEFAULT_DURATION,
+          },
+          {
+            zoom: DEFAULT_ZOOM,
+            duration: DEFAULT_DURATION,
+          },
+        );
+      }
+
+      // Remove all defined layers
+      Object.values(LAYER_NAMES).forEach(clearLayer);
+
+      // Call app-specific reset logic
+      onReset?.();
+    },
+    [map, clearLayer],
+  );
 
   const contextValue = useMemo(
     () => ({
@@ -168,12 +161,6 @@ export const MapProvider = ({ initialState = {}, children }: MapProviderProps) =
       setSelectedLayer,
       isDrawing,
       setIsDrawing,
-      samplingPoints,
-      setSamplingPoints,
-      isTooManySamplingPoints,
-      setIsTooManySamplingPoints,
-      isSamplingPointsLoading,
-      setIsSamplingPointsLoading,
       clearLayer,
       resetMap,
       ...initialState,
@@ -188,9 +175,6 @@ export const MapProvider = ({ initialState = {}, children }: MapProviderProps) =
       aoi,
       selectedLayer,
       isDrawing,
-      samplingPoints,
-      isTooManySamplingPoints,
-      isSamplingPointsLoading,
       clearLayer,
       resetMap,
       initialState,