@squiz/resource-browser 3.3.0 → 3.3.1

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Change Log
 
+## 3.3.1
+
+### Patch Changes
+
+-   c8240b6: Minor styling updates
+-   Updated dependencies [c8240b6]
+    -   @squiz/resource-browser-ui-lib@1.2.1
+
 ## 3.3.0
 
 ### Minor Changes

package/lib/ResourceLauncher/ResourceLauncher.js

@@ -4,6 +4,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 const react_1 = __importDefault(require("react"));
+const sds_1 = require("@squiz/sds");
 const types_1 = require("../types");
 function ResourceLauncher({ sources, onSourceSelect }) {
     return (react_1.default.createElement("div", { className: "overflow-y-auto w-screen max-w-[400px] min-h-[290px] flex-1 grow-[3] border-r border-gray-300 bg-gray-100 pl-6 pr-6 pb-6 pt-4" },
@@ -11,7 +12,9 @@ function ResourceLauncher({ sources, onSourceSelect }) {
             const SourceLauncher = source.plugin.renderResourceLauncher();
             return (react_1.default.createElement("li", { key: index, className: "flex items-stretch relative" },
                 react_1.default.createElement("div", { className: `squiz-rb-plugin squiz-rb-plugin--${source.plugin?.type} w-full` },
-                    react_1.default.createElement(SourceLauncher, { source: source, onSearch: (query) => onSourceSelect(source, { type: types_1.PluginLaunchModeType.Search, args: { query } }), onBrowse: (browseTo) => onSourceSelect(source, { type: types_1.PluginLaunchModeType.Browse, args: { browseTo } }) }))));
+                    react_1.default.createElement(SourceLauncher, { source: source, onSearch: (query) => onSourceSelect(source, { type: types_1.PluginLaunchModeType.Search, args: { query } }), onBrowse: (browseTo) => onSourceSelect(source, { type: types_1.PluginLaunchModeType.Browse, args: { browseTo } }) }),
+                    sources.length > 1 && index < sources.length - 1 && (react_1.default.createElement("div", { className: "pt-3 pb-3" },
+                        react_1.default.createElement(sds_1.Divider, null))))));
         }))));
 }
 exports.default = ResourceLauncher;

package/lib/index.css

@@ -5201,6 +5201,9 @@
 .squiz-rb-scope .rounded-lg:not(.squiz-rb-plugin *) {
   border-radius: 0.5rem;
 }
+.squiz-rb-scope .border:not(.squiz-rb-plugin *) {
+  border-width: 1px;
+}
 .squiz-rb-scope .border-0:not(.squiz-rb-plugin *) {
   border-width: 0px;
 }
@@ -5364,6 +5367,9 @@
 .squiz-rb-scope .pb-0:not(.squiz-rb-plugin *) {
   padding-bottom: 0px;
 }
+.squiz-rb-scope .pb-3:not(.squiz-rb-plugin *) {
+  padding-bottom: 0.75rem;
+}
 .squiz-rb-scope .pb-4:not(.squiz-rb-plugin *) {
   padding-bottom: 1rem;
 }
@@ -5893,7 +5899,12 @@
   width: 48px;
 }
 .squiz-rb-scope .sourceLauncher:not(.squiz-rb-plugin *) {
-  position: relative;
+  margin-bottom: 0.5rem;
+}
+.squiz-rb-scope .sourceLauncher__top-row:not(.squiz-rb-plugin *) {
+  display: flex;
+  align-items: flex-end;
+  justify-content: space-between;
   margin-bottom: 0.5rem;
 }
 .squiz-rb-scope .sourceLauncher__title:not(.squiz-rb-plugin *) {
@@ -5904,10 +5915,8 @@
   color: #4f4f4f;
 }
 .squiz-rb-scope .sourceLauncher__browse-group:not(.squiz-rb-plugin *) {
-  position: absolute;
-  right: 0;
-  top: 0;
   display: flex;
+  align-items: center;
 }
 .squiz-rb-scope .sourceLauncher__or:not(.squiz-rb-plugin *) {
   display: flex;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@squiz/resource-browser",
-    "version": "3.3.0",
+    "version": "3.3.1",
     "main": "lib/index.js",
     "types": "lib/index.d.ts",
     "private": false,
@@ -28,7 +28,7 @@
         "@react-types/shared": "^3.23.1",
         "@squiz/dx-json-schema-lib": "^1.67.0",
         "@squiz/generic-browser-lib": "1.67.2",
-        "@squiz/resource-browser-ui-lib": "^1.0.0",
+        "@squiz/resource-browser-ui-lib": "^1.2.1",
         "clsx": "^2.1.0",
         "expiry-map": "^2.0.0",
         "p-memoize": "^4.0.4",

package/src/ResourceLauncher/ResourceLauncher.spec.tsx

@@ -23,6 +23,20 @@ describe('ResourceLauncher', () => {
         expect(screen.queryAllByText('Resource Launcher UI').length).toEqual(2);
     });
 
+    it('should render the divider only for source items except the last one', () => {
+        const sources = [
+            mockSourceWithPlugin({ plugin: defaultPlugin }),
+            mockSourceWithPlugin({ id: '2', plugin: defaultPlugin }),
+            mockSourceWithPlugin({ id: '3', plugin: defaultPlugin }),
+        ];
+        const onSourceSelect = jest.fn();
+        const { container } = render(<ResourceLauncher sources={sources} onSourceSelect={onSourceSelect} />);
+
+        const dividerContainers = container.querySelectorAll('div.pt-3.pb-3');
+        // With 3 sources, there should be 2 dividers (in the first two <li>s)
+        expect(dividerContainers.length).toEqual(sources.length - 1);
+    });
+
     it('onSourceSelect query', async () => {
         const sources = [mockSourceWithPlugin({ plugin: defaultPlugin })];
         const onSourceSelect = jest.fn();

package/src/ResourceLauncher/ResourceLauncher.stories.tsx

@@ -0,0 +1,55 @@
+import React from 'react';
+import { StoryFn, Meta } from '@storybook/react';
+import ResourceLauncher from './ResourceLauncher';
+import sampleSources from '../__mocks__/sample-sources.json';
+import { ResourceBrowserSourceWithPlugin, PluginLaunchMode, PluginLaunchModeType } from '../types';
+
+const DummyLauncher = ({
+    source,
+    onSearch,
+    onBrowse,
+}: {
+    source: any;
+    onSearch: (query: string) => void;
+    onBrowse: (browseTo: string) => void;
+}) => (
+    <div className="p-2 border rounded">
+        <div>{source.name}</div>
+        <button onClick={() => onSearch('sample query')} className="mr-2">
+            Search
+        </button>
+        <button onClick={() => onBrowse('sample browse')}>Browse</button>
+    </div>
+);
+
+const sourcesWithDummyPlugin = (sampleSources as any[]).map((source) => ({
+    ...source,
+    plugin: {
+        ...source.plugin,
+        type: source.plugin?.type || 'dummy',
+        // renderResourceLauncher should return a component. Here we return a function that renders DummyLauncher.
+        renderResourceLauncher: () => DummyLauncher,
+    },
+})) as ResourceBrowserSourceWithPlugin[];
+
+export default {
+    title: 'Resource Launcher',
+    component: ResourceLauncher,
+} as Meta<typeof ResourceLauncher>;
+
+const Template: StoryFn<typeof ResourceLauncher> = (args) => {
+    return (
+        <div className="flex justify-center m-3">
+            <ResourceLauncher
+                {...args}
+                sources={sourcesWithDummyPlugin}
+                onSourceSelect={(source, mode: PluginLaunchMode) => {
+                    console.log(`Source ${source?.name} / ${source.id} was selected with mode:`, mode);
+                }}
+            />
+        </div>
+    );
+};
+
+export const Primary = Template.bind({});
+Primary.args = {};

package/src/ResourceLauncher/ResourceLauncher.tsx

@@ -1,4 +1,5 @@
 import React from 'react';
+import { Divider } from '@squiz/sds';
 import { ResourceBrowserSource, ResourceBrowserSourceWithPlugin, PluginLaunchMode, PluginLaunchModeType } from '../types';
 
 interface ResourceLauncherProps {
@@ -23,6 +24,11 @@ function ResourceLauncher({ sources, onSourceSelect }: ResourceLauncherProps) {
                                         onSourceSelect(source, { type: PluginLaunchModeType.Browse, args: { browseTo } })
                                     }
                                 />
+                                {sources.length > 1 && index < sources.length - 1 && (
+                                    <div className="pt-3 pb-3">
+                                        <Divider />
+                                    </div>
+                                )}
                             </div>
                         </li>
                     );