Open external links in a new tab

.vscode/extensions.json

@@ -0,0 +1,8 @@
+{
+  "recommendations": [
+    "nrwl.angular-console",
+    "esbenp.prettier-vscode",
+    "firsttris.vscode-jest-runner",
+    "dbaeumer.vscode-eslint"
+  ]
+}

README.md

@@ -1,56 +1,31 @@
 <h1 align="center">
-  <a href="https://datahub.io/">
-    <img alt="datahub" src="http://datahub.io/datahub-cube.svg" width="146">
-  </a>
+🌀 Portal.JS
+<br />
+Rapidly build rich data portals using a modern frontend framework
 </h1>
 
-<p align="center">
-  Bugs, issues and suggestions re DataHub Cloud ☁️ and DataHub OpenSource 🌀
-  <br />
-  <br /><a href="https://discord.gg/xfFDMPU9dC"><img src="https://dcbadge.vercel.app/api/server/xfFDMPU9dC" /></a>
-</p>
+* [What is Portal.JS ?](#What-is-Portal.JS)
+    * [Features](#Features)
+      * [For developers](#For-developers)
+* [Docs](#Docs)
+* [Community](#Community)
+* [Appendix](#Appendix)
+    * [What happened to Recline?](#What-happened-to-Recline?)
 
-## DataHub
+# What is Portal.JS
 
-This repo and issue tracker are for 
+🌀 Portal.JS is a framework for rapidly building rich data portal frontends using a modern frontend approach. Portal.JS can be used to present a single dataset or build a full-scale data catalog/portal.
 
-- DataHub Cloud ☁️ - https://datahub.io/
-- DataHub 🌀 - https://datahub.io/opensource
+Built in JavaScript and React on top of the popular [Next.js](https://nextjs.com/) framework. Portal.JS assumes a "decoupled" approach where the frontend is a separate service from the backend and interacts with backend(s) via an API. It can be used with any backend and has out of the box support for [CKAN](https://ckan.org/).
 
-### Issues
-
-Found a bug: 👉 https://github.com/datopian/datahub/issues/new
-
-### Discussions
-
-Got a suggestion, a question, want some support or just want to shoot the breeze 🙂
-
-Head to the discussion forum: 👉 https://github.com/datopian/datahub/discussions
-
-### Chat on Discord
-
-If you would prefer to get help via live chat check out our discord 👉
-
-[Discord](https://discord.gg/xfFDMPU9dC)
-
-### Docs
-
-https://datahub.io/docs
-
-## DataHub OpenSource 🌀
-
-DataHub 🌀 is a platform for rapidly creating rich data portal and publishing systems using a modern frontend approach. Datahub can be used to publish a single dataset or build a full-scale data catalog/portal.
-
-DataHub is built in JavaScript and React on top of the popular [Next.js](https://nextjs.com/) framework. DataHub assumes a "decoupled" approach where the frontend is a separate service from the backend and interacts with backend(s) via an API. It can be used with any backend and has out of the box support for [CKAN](https://ckan.org/), GitHub, Frictionless Data Packages and more.
-
-### Features
+## Features
 
 - 🗺️ Unified sites: present data and content in one seamless site, pulling datasets from a DMS (e.g. CKAN) and content from a CMS (e.g. Wordpress) with a common internal API.
 - 👩‍💻 Developer friendly: built with familiar frontend tech (JavaScript, React, Next.js).
 - 🔋 Batteries included: full set of portal components out of the box e.g. catalog search, dataset showcase, blog, etc.
 - 🎨 Easy to theme and customize: installable themes, use standard CSS and React+CSS tooling. Add new routes quickly.
 - 🧱 Extensible: quickly extend and develop/import your own React components
-- 📝 Well documented: full set of documentation plus the documentation of Next.js.
+- 📝 Well documented: full set of documentation plus the documentation of Next.js and Apollo.
 
 ### For developers
 
@@ -58,3 +33,25 @@ DataHub is built in JavaScript and React on top of the popular [Next.js](https:/
 - 🚀 Next.js framework: so everything in Next.js for free: Server Side Rendering, Static Site Generation, huge number of examples and integrations, etc.
   - Server Side Rendering (SSR) => Unlimited number of pages, SEO and more whilst still using React.
   - Static Site Generation (SSG) => Ultra-simple deployment, great performance, great lighthouse scores and more (good for small sites)
+
+#### **Check out the [Portal.JS website](https://portaljs.org/) for a gallery of live portals**
+
+___
+
+# Docs
+
+Access the Portal.JS documentation at:
+
+https://portaljs.org/docs
+
+- [Examples](https://portaljs.org/docs#examples)
+
+# Community
+
+If you have questions about anything related to Portal.JS, you're always welcome to ask our community on [GitHub Discussions](https://github.com/datopian/portal.js/discussions) or on our [Discord server](https://discord.gg/EeyfGrGu4U).
+
+# Appendix
+
+## What happened to Recline?
+
+Portal.JS used to be Recline(JS). If you are looking for the old Recline codebase it still exists:  see the [`recline` branch](https://github.com/datopian/portal.js/tree/recline). If you want context for the rename see [this issue](https://github.com/datopian/portal.js/issues/520).

packages/components/CHANGELOG.md

@@ -1,17 +1,5 @@
 # @portaljs/components
 
-## 1.0.0
-
-### Major Changes
-
-- [#1103](https://github.com/datopian/datahub/pull/1103) [`48cd812a`](https://github.com/datopian/datahub/commit/48cd812a488a069a419d8ecc67f24f94d4d1d1d6) Thanks [@demenech](https://github.com/demenech)! - Components API tidying up and storybook docs improvements.
-
-## 0.6.0
-
-### Minor Changes
-
-- [`a044f56e`](https://github.com/datopian/portaljs/commit/a044f56e3cbe0519ddf9d24d78b0bb7eac917e1c) Thanks [@luccasmmg](https://github.com/luccasmmg)! - Added plotly components
-
 ## 0.5.10
 
 ### Patch Changes

packages/components/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@portaljs/components",
-  "version": "1.0.0",
+  "version": "0.5.10",
   "type": "module",
   "description": "https://portaljs.org",
   "keywords": [
@@ -40,13 +40,11 @@
     "ol": "^7.4.0",
     "papaparse": "^5.4.1",
     "pdfjs-dist": "2.15.349",
-    "plotly.js": "^2.30.1",
     "postcss-url": "^10.1.3",
     "react": "^18.2.0",
     "react-dom": "^18.2.0",
     "react-hook-form": "^7.43.9",
     "react-leaflet": "^4.2.1",
-    "react-plotly.js": "^2.6.0",
     "react-query": "^3.39.3",
     "react-vega": "^7.6.0",
     "vega": "5.25.0",

packages/components/src/components/Catalog.tsx

@@ -7,12 +7,7 @@ export function Catalog({
   datasets,
   facets,
 }: {
-  datasets: {
-    _id: string | number;
-    metadata: { title: string; [k: string]: string | number };
-    url_path: string;
-    [k: string]: any;
-  }[];
+  datasets: any[];
   facets: string[];
 }) {
   const [indexFilter, setIndexFilter] = useState('');
@@ -61,7 +56,7 @@ export function Catalog({
     //Then check if the selectedValue for the given facet is included in the dataset metadata
     .filter((dataset) => {
       //Avoids a server rendering breakage
-      if (!watch() || Object.keys(watch()).length === 0) return true;
+      if (!watch() || Object.keys(watch()).length === 0) return true
       //This will filter only the key pairs of the metadata values that were selected as facets
       const datasetFacets = Object.entries(dataset.metadata).filter((entry) =>
         facets.includes(entry[0])
@@ -91,7 +86,9 @@ export function Catalog({
           className="p-2 ml-1 text-sm shadow border border-block"
           {...register(elem[0] + '.selectedValue')}
         >
-          <option value="">Filter by {elem[0]}</option>
+          <option value="">
+            Filter by {elem[0]}
+          </option>
           {(elem[1] as { possibleValues: string[] }).possibleValues.map(
             (val) => (
               <option
@@ -105,10 +102,10 @@ export function Catalog({
           )}
         </select>
       ))}
-      <ul className="mb-5 pl-6 mt-5 list-disc">
+      <ul className='mb-5 pl-6 mt-5 list-disc'>
         {filteredDatasets.map((dataset) => (
-          <li className="py-2" key={dataset._id}>
-            <a className="font-medium underline" href={dataset.url_path}>
+          <li className='py-2' key={dataset._id}>
+            <a className='font-medium underline' href={dataset.url_path}>
               {dataset.metadata.title
                 ? dataset.metadata.title
                 : dataset.url_path}
@@ -119,3 +116,4 @@ export function Catalog({
     </>
   );
 }
+

packages/components/src/components/Excel.tsx

@@ -4,14 +4,12 @@ import { read, utils } from 'xlsx';
 import { AgGridReact } from 'ag-grid-react';
 import 'ag-grid-community/styles/ag-grid.css';
 import 'ag-grid-community/styles/ag-theme-alpine.css';
-import { Data } from '../types/properties';
 
 export type ExcelProps = {
-  data: Required<Pick<Data, 'url'>>;
+  url: string;
 };
 
-export function Excel({ data }: ExcelProps) {
-  const url = data.url;
+export function Excel({ url }: ExcelProps) {
   const [isLoading, setIsLoading] = useState<boolean>(false);
   const [activeSheetName, setActiveSheetName] = useState<string>();
   const [workbook, setWorkbook] = useState<any>();

packages/components/src/components/FlatUiTable.tsx

@@ -2,7 +2,6 @@ import { QueryClient, QueryClientProvider, useQuery } from 'react-query';
 import Papa from 'papaparse';
 import { Grid } from '@githubocto/flat-ui';
 import LoadingSpinner from './LoadingSpinner';
-import { Data } from '../types/properties';
 
 const queryClient = new QueryClient();
 
@@ -37,25 +36,30 @@ export async function parseCsv(file: string, parsingConfig): Promise<any> {
 }
 
 export interface FlatUiTableProps {
-  data: Data;
-  uniqueId?: number;
+  url?: string;
+  data?: { [key: string]: number | string }[];
+  rawCsv?: string;
+  randomId?: number;
   bytes: number;
   parsingConfig: any;
 }
 export const FlatUiTable: React.FC<FlatUiTableProps> = ({
+  url,
   data,
-  uniqueId,
+  rawCsv,
   bytes = 5132288,
   parsingConfig = {},
 }) => {
-  uniqueId = uniqueId ?? Math.random();
+  const randomId = Math.random();
   return (
     // Provide the client to your App
     <QueryClientProvider client={queryClient}>
       <TableInner
         bytes={bytes}
+        url={url}
         data={data}
-        uniqueId={uniqueId}
+        rawCsv={rawCsv}
+        randomId={randomId}
         parsingConfig={parsingConfig}
       />
     </QueryClientProvider>
@@ -63,32 +67,33 @@ export const FlatUiTable: React.FC<FlatUiTableProps> = ({
 };
 
 const TableInner: React.FC<FlatUiTableProps> = ({
+  url,
   data,
-  uniqueId,
+  rawCsv,
+  randomId,
   bytes,
   parsingConfig,
 }) => {
-  const url = data.url;
-  const csv = data.csv;
-  const values = data.values;
-
-  if (values) {
+  if (data) {
     return (
       <div className="w-full" style={{ height: '500px' }}>
-        <Grid data={values} />
+        <Grid data={data} />
       </div>
     );
   }
   const { data: csvString, isLoading: isDownloadingCSV } = useQuery(
-    ['dataCsv', url, uniqueId],
+    ['dataCsv', url, randomId],
     () => getCsv(url as string, bytes),
     { enabled: !!url }
   );
   const { data: parsedData, isLoading: isParsing } = useQuery(
-    ['dataPreview', csvString, uniqueId],
+    ['dataPreview', csvString, randomId],
     () =>
-      parseCsv(csv ? (csv as string) : (csvString as string), parsingConfig),
-    { enabled: csv ? true : !!csvString }
+      parseCsv(
+        rawCsv ? (rawCsv as string) : (csvString as string),
+        parsingConfig
+      ),
+    { enabled: rawCsv ? true : !!csvString }
   );
   if (isParsing || isDownloadingCSV)
     <div className="w-full flex justify-center items-center h-[500px]">

packages/components/src/components/Iframe.tsx

@@ -1,17 +1,14 @@
-import { CSSProperties } from 'react';
-import { Data } from '../types/properties';
+import { CSSProperties } from "react";
 
 export interface IframeProps {
-  data: Required<Pick<Data, 'url'>>;
+  url: string;
   style?: CSSProperties;
 }
 
-export function Iframe({ data, style }: IframeProps) {
-  const url = data.url;
+export function Iframe({
+  url, style
+}: IframeProps) {
   return (
-    <iframe
-      src={url}
-      style={style ?? { width: `100%`, height: `100%` }}
-    ></iframe>
+    <iframe src={url} style={style ?? { width: `100%`, height: `100%` }}></iframe>
   );
 }

packages/components/src/components/LineChart.tsx

@@ -2,33 +2,31 @@ import { useEffect, useState } from 'react';
 import LoadingSpinner from './LoadingSpinner';
 import { VegaLite } from './VegaLite';
 import loadData from '../lib/loadData';
-import { Data } from '../types/properties';
 
 type AxisType = 'quantitative' | 'temporal';
 type TimeUnit = 'year' | undefined; // or ...
 
 export type LineChartProps = {
-  data: Omit<Data, 'csv'>;
+  data: Array<Array<string | number>> | string | { x: string; y: number }[];
   title?: string;
-  xAxis: string;
+  xAxis?: string;
   xAxisType?: AxisType;
-  xAxisTimeUnit?: TimeUnit;
-  yAxis: string;
+  xAxisTimeUnit: TimeUnit;
+  yAxis?: string;
   yAxisType?: AxisType;
   fullWidth?: boolean;
 };
 
 export function LineChart({
-  data,
+  data = [],
+  fullWidth = false,
   title = '',
-  xAxis,
+  xAxis = 'x',
   xAxisType = 'temporal',
   xAxisTimeUnit = 'year', //  TODO: defaults to undefined would probably work better... keeping it as it's for compatibility purposes
-  yAxis,
+  yAxis = 'y',
   yAxisType = 'quantitative',
 }: LineChartProps) {
-  const url = data.url;
-  const values = data.values;
   const [isLoading, setIsLoading] = useState<boolean>(false);
 
   //  By default, assumes data is an Array...
@@ -66,12 +64,13 @@ export function LineChart({
   } as any;
 
   useEffect(() => {
-    if (url) {
+    //  If data is string, assume it's a URL
+    if (typeof data === 'string') {
       setIsLoading(true);
 
       //  Manualy loading the data allows us to do other kinds
       //  of stuff later e.g. load a file partially
-      loadData(url).then((res: any) => {
+      loadData(data).then((res: any) => {
         setSpecData({ values: res, format: { type: 'csv' } });
         setIsLoading(false);
       });
@@ -79,8 +78,12 @@ export function LineChart({
   }, []);
 
   var vegaData = {};
-  if (values) {
-    vegaData = { table: values };
+  if (Array.isArray(data)) {
+    var dataObj;
+    dataObj = data.map((r) => {
+      return { x: r[0], y: r[1] };
+    });
+    vegaData = { table: dataObj };
   }
 
   return isLoading ? (
@@ -88,6 +91,6 @@ export function LineChart({
       <LoadingSpinner />
     </div>
   ) : (
-    <VegaLite data={vegaData} spec={spec} />
+    <VegaLite fullWidth={fullWidth} data={vegaData} spec={spec} />
   );
 }

packages/components/src/components/Map.tsx

@@ -2,7 +2,6 @@ import { CSSProperties, useEffect, useState } from 'react';
 import LoadingSpinner from './LoadingSpinner';
 import loadData from '../lib/loadData';
 import chroma from 'chroma-js';
-import { GeospatialData } from '../types/properties';
 import {
   MapContainer,
   TileLayer,
@@ -15,25 +14,26 @@ import * as L from 'leaflet';
 
 export type MapProps = {
   layers: {
-    data: GeospatialData;
+    data: string | GeoJSON.GeoJSON;
     name: string;
     colorScale?: {
       starting: string;
       ending: string;
     };
     tooltip?:
-      | {
-          propNames: string[];
-        }
-      | boolean;
+    | {
+      propNames: string[];
+    }
+    | boolean;
+    _id?: number;
   }[];
   title?: string;
   center?: { latitude: number | undefined; longitude: number | undefined };
   zoom?: number;
   style?: CSSProperties;
   autoZoomConfiguration?: {
-    layerName: string;
-  };
+    layerName: string
+  }
 };
 
 export function Map({
@@ -56,19 +56,17 @@ export function Map({
 
   useEffect(() => {
     const loadDataPromises = layers.map(async (layer) => {
-      const url = layer.data.url;
-      const geojson = layer.data.geojson;
       let layerData: any;
 
-      if (url) {
+      if (typeof layer.data === 'string') {
         //  If "data" is string, assume it's a URL
         setIsLoading(true);
-        layerData = await loadData(url).then((res: any) => {
+        layerData = await loadData(layer.data).then((res: any) => {
           return JSON.parse(res);
         });
       } else {
         //  Else, expect raw GeoJSON
-        layerData = geojson;
+        layerData = layer.data;
       }
 
       if (layer.colorScale) {
@@ -113,23 +111,23 @@ export function Map({
         //  Create the title box
         var info = new L.Control() as any;
 
-        info.onAdd = function () {
+        info.onAdd = function() {
           this._div = L.DomUtil.create('div', 'info');
           this.update();
           return this._div;
         };
 
-        info.update = function () {
+        info.update = function() {
           this._div.innerHTML = `<h4 style="font-weight: 600; background: #f9f9f9; padding: 5px; border-radius: 5px; color: #464646;">${title}</h4>`;
         };
 
         if (title) info.addTo(map.target);
-        if (!autoZoomConfiguration) return;
+        if(!autoZoomConfiguration) return;
 
         let layerToZoomBounds = L.latLngBounds(L.latLng(0, 0), L.latLng(0, 0));
 
         layers.forEach((layer) => {
-          if (layer.name === autoZoomConfiguration.layerName) {
+          if(layer.name === autoZoomConfiguration.layerName) {
             const data = layersData.find(
               (layerData) => layerData.name === layer.name
             )?.data;

packages/components/src/components/PdfViewer.tsx

@@ -1,24 +1,22 @@
 // Core viewer
 import { Viewer, Worker, SpecialZoomLevel } from '@react-pdf-viewer/core';
 import { defaultLayoutPlugin } from '@react-pdf-viewer/default-layout';
-import { Data } from '../types/properties';
 
 // Import styles
 import '@react-pdf-viewer/core/lib/styles/index.css';
 import '@react-pdf-viewer/default-layout/lib/styles/index.css';
 
 export interface PdfViewerProps {
-  data: Required<Pick<Data, 'url'>>;
+  url: string;
   layout: boolean;
   parentClassName?: string;
 }
 
 export function PdfViewer({
-  data,
+  url,
   layout = false,
-  parentClassName = 'h-screen',
+  parentClassName,
 }: PdfViewerProps) {
-  const url = data.url;
   const defaultLayoutPluginInstance = defaultLayoutPlugin();
   return (
     <Worker workerUrl="https://unpkg.com/pdfjs-dist@2.15.349/build/pdf.worker.js">

packages/components/src/components/Plotly.tsx

@@ -1,9 +0,0 @@
-import Plot, { PlotParams } from "react-plotly.js";
-
-export const Plotly: React.FC<PlotParams> = (props) => {
-  return (
-    <div>
-      <Plot {...props} />
-    </div>
-  );
-};

packages/components/src/components/PlotlyBarChart.tsx

@@ -1,153 +0,0 @@
-import { QueryClient, QueryClientProvider, useQuery } from 'react-query';
-import { Plotly } from './Plotly';
-import Papa, { ParseConfig } from 'papaparse';
-import LoadingSpinner from './LoadingSpinner';
-import { Data } from '../types/properties';
-
-const queryClient = new QueryClient();
-
-async function getCsv(url: string, bytes: number) {
-  const response = await fetch(url, {
-    headers: {
-      Range: `bytes=0-${bytes}`,
-    },
-  });
-  const data = await response.text();
-  return data;
-}
-
-async function parseCsv(
-  file: string,
-  parsingConfig: ParseConfig
-): Promise<any> {
-  return new Promise((resolve, reject) => {
-    Papa.parse(file, {
-      ...parsingConfig,
-      header: true,
-      dynamicTyping: true,
-      skipEmptyLines: true,
-      transform: (value: string): string => {
-        return value.trim();
-      },
-      complete: (results: any) => {
-        return resolve(results);
-      },
-      error: (error: any) => {
-        return reject(error);
-      },
-    });
-  });
-}
-
-export interface PlotlyBarChartProps {
-  data: Data;
-  uniqueId?: number;
-  bytes?: number;
-  parsingConfig?: ParseConfig;
-  xAxis: string;
-  yAxis: string;
-  // TODO: commented out because this doesn't work. I believe
-  // this would only make any difference on charts with multiple
-  // traces.
-  // lineLabel?: string;
-  title?: string;
-}
-
-export const PlotlyBarChart: React.FC<PlotlyBarChartProps> = ({
-  data,
-  bytes = 5132288,
-  parsingConfig = {},
-  xAxis,
-  yAxis,
-  // lineLabel,
-  title = '',
-}) => {
-  const uniqueId = Math.random();
-  return (
-    // Provide the client to your App
-    <QueryClientProvider client={queryClient}>
-      <PlotlyBarChartInner
-        data={data}
-        uniqueId={uniqueId}
-        bytes={bytes}
-        parsingConfig={parsingConfig}
-        xAxis={xAxis}
-        yAxis={yAxis}
-        // lineLabel={lineLabel ?? yAxis}
-        title={title}
-      />
-    </QueryClientProvider>
-  );
-};
-
-const PlotlyBarChartInner: React.FC<PlotlyBarChartProps> = ({
-  data,
-  uniqueId,
-  bytes,
-  parsingConfig,
-  xAxis,
-  yAxis,
-  // lineLabel,
-  title,
-}) => {
-  if (data.values) {
-    return (
-      <div className="w-full" style={{ height: '500px' }}>
-        <Plotly
-          layout={{
-            title,
-          }}
-          data={[
-            {
-              x: data.values.map((d) => d[xAxis]),
-              y: data.values.map((d) => d[yAxis]),
-              type: 'bar',
-              // name: lineLabel,
-            },
-          ]}
-        />
-      </div>
-    );
-  }
-  const { data: csvString, isLoading: isDownloadingCSV } = useQuery(
-    ['dataCsv', data.url, uniqueId],
-    () => getCsv(data.url as string, bytes ?? 5132288),
-    { enabled: !!data.url }
-  );
-  const { data: parsedData, isLoading: isParsing } = useQuery(
-    ['dataPreview', csvString, uniqueId],
-    () =>
-      parseCsv(
-        data.csv ? (data.csv as string) : (csvString as string),
-        parsingConfig ?? {}
-      ),
-    { enabled: data.csv ? true : !!csvString }
-  );
-  if (isParsing || isDownloadingCSV)
-    <div className="w-full flex justify-center items-center h-[500px]">
-      <LoadingSpinner />
-    </div>;
-  if (parsedData)
-    return (
-      <div className="w-full" style={{ height: '500px' }}>
-        <Plotly
-          layout={{
-            title,
-          }}
-          data={[
-            {
-              x: parsedData.data.map((d: any) => d[xAxis]),
-              y: parsedData.data.map((d: any) => d[yAxis]),
-              type: 'bar',
-              // name: lineLabel, TODO: commented out because this doesn't work
-            },
-          ]}
-        />
-      </div>
-    );
-  return (
-    <div className="w-full flex justify-center items-center h-[500px]">
-      <LoadingSpinner />
-    </div>
-  );
-};

packages/components/src/components/PlotlyLineChart.tsx

@@ -1,155 +0,0 @@
-import { QueryClient, QueryClientProvider, useQuery } from 'react-query';
-import { Plotly } from './Plotly';
-import Papa, { ParseConfig } from 'papaparse';
-import LoadingSpinner from './LoadingSpinner';
-import { Data } from '../types/properties';
-
-const queryClient = new QueryClient();
-
-async function getCsv(url: string, bytes: number) {
-  const response = await fetch(url, {
-    headers: {
-      Range: `bytes=0-${bytes}`,
-    },
-  });
-  const data = await response.text();
-  return data;
-}
-
-async function parseCsv(
-  file: string,
-  parsingConfig: ParseConfig
-): Promise<any> {
-  return new Promise((resolve, reject) => {
-    Papa.parse(file, {
-      ...parsingConfig,
-      header: true,
-      dynamicTyping: true,
-      skipEmptyLines: true,
-      transform: (value: string): string => {
-        return value.trim();
-      },
-      complete: (results: any) => {
-        return resolve(results);
-      },
-      error: (error: any) => {
-        return reject(error);
-      },
-    });
-  });
-}
-
-export interface PlotlyLineChartProps {
-  data: Data;
-  bytes?: number;
-  parsingConfig?: ParseConfig;
-  xAxis: string;
-  yAxis: string;
-  lineLabel?: string;
-  title?: string;
-  uniqueId?: number;
-}
-
-export const PlotlyLineChart: React.FC<PlotlyLineChartProps> = ({
-  data,
-  bytes = 5132288,
-  parsingConfig = {},
-  xAxis,
-  yAxis,
-  lineLabel,
-  title = '',
-  uniqueId,
-}) => {
-  uniqueId = uniqueId ?? Math.random();
-  return (
-    // Provide the client to your App
-    <QueryClientProvider client={queryClient}>
-      <LineChartInner
-        data={data}
-        uniqueId={uniqueId}
-        bytes={bytes}
-        parsingConfig={parsingConfig}
-        xAxis={xAxis}
-        yAxis={yAxis}
-        lineLabel={lineLabel ?? yAxis}
-        title={title}
-      />
-    </QueryClientProvider>
-  );
-};
-
-const LineChartInner: React.FC<PlotlyLineChartProps> = ({
-  data,
-  uniqueId,
-  bytes,
-  parsingConfig,
-  xAxis,
-  yAxis,
-  lineLabel,
-  title,
-}) => {
-  const values = data.values;
-  const url = data.url;
-  const csv = data.csv;
-
-  if (values) {
-    return (
-      <div className="w-full" style={{ height: '500px' }}>
-        <Plotly
-          layout={{
-            title,
-          }}
-          data={[
-            {
-              x: values.map((d) => d[xAxis]),
-              y: values.map((d) => d[yAxis]),
-              mode: 'lines',
-              name: lineLabel,
-            },
-          ]}
-        />
-      </div>
-    );
-  }
-  const { data: csvString, isLoading: isDownloadingCSV } = useQuery(
-    ['dataCsv', url, uniqueId],
-    () => getCsv(url as string, bytes ?? 5132288),
-    { enabled: !!url }
-  );
-  const { data: parsedData, isLoading: isParsing } = useQuery(
-    ['dataPreview', csvString, uniqueId],
-    () =>
-      parseCsv(
-        csv ? (csv as string) : (csvString as string),
-        parsingConfig ?? {}
-      ),
-    { enabled: csv ? true : !!csvString }
-  );
-  if (isParsing || isDownloadingCSV)
-    <div className="w-full flex justify-center items-center h-[500px]">
-      <LoadingSpinner />
-    </div>;
-  if (parsedData)
-    return (
-      <div className="w-full" style={{ height: '500px' }}>
-        <Plotly
-          layout={{
-            title,
-          }}
-          data={[
-            {
-              x: parsedData.data.map((d: any) => d[xAxis]),
-              y: parsedData.data.map((d: any) => d[yAxis]),
-              mode: 'lines',
-              name: lineLabel,
-            },
-          ]}
-        />
-      </div>
-    );
-  return (
-    <div className="w-full flex justify-center items-center h-[500px]">
-      <LoadingSpinner />
-    </div>
-  );
-};

packages/components/src/components/Vega.tsx

@@ -1,7 +1,6 @@
 //  Wrapper for the Vega component
 import { Vega as VegaOg } from "react-vega";
-import { VegaProps } from "react-vega/lib/Vega";
 
-export function Vega(props: VegaProps) {
+export function Vega(props) {
   return <VegaOg {...props} />;
 }

packages/components/src/components/VegaLite.tsx

@@ -1,9 +1,8 @@
 //  Wrapper for the Vega Lite component
-import { VegaLite as VegaLiteOg } from 'react-vega';
-import { VegaLiteProps } from 'react-vega/lib/VegaLite';
-import applyFullWidthDirective from '../lib/applyFullWidthDirective';
+import { VegaLite as VegaLiteOg } from "react-vega";
+import applyFullWidthDirective from "../lib/applyFullWidthDirective";
 
-export function VegaLite(props: VegaLiteProps) {
+export function VegaLite(props) {
   const Component = applyFullWidthDirective({ Component: VegaLiteOg });
 
   return <Component {...props} />;

packages/components/src/index.ts

@@ -1,17 +1,12 @@
+export * from './components/Table';
 export * from './components/Catalog';
 export * from './components/LineChart';
 export * from './components/Vega';
 export * from './components/VegaLite';
 export * from './components/FlatUiTable';
+export * from './components/OpenLayers/OpenLayers';
 export * from './components/Map';
 export * from './components/PdfViewer';
 export * from "./components/Excel";
+export * from "./components/BucketViewer";
 export * from "./components/Iframe";
-export * from "./components/Plotly";
-export * from "./components/PlotlyLineChart";
-export * from "./components/PlotlyBarChart";
-// NOTE: components that are hidden for now
-// TODO: deprecate those components?
-// export * from './components/Table';
-// export * from "./components/BucketViewer";
-// export * from './components/OpenLayers/OpenLayers';

packages/components/src/types/properties.ts

@@ -1,18 +0,0 @@
-/*
- * All components should use this interface for
- * its data property.
- * Based on vega.
- *
- */
-
-type URL = string; // Just in case we want to transform it into an object with configurations
-export interface Data {
-  url?: URL;
-  values?: { [key: string]: number | string }[];
-  csv?: string;
-}
-
-export interface GeospatialData {
-  url?: URL;
-  geojson?: GeoJSON.GeoJSON;
-}

packages/components/stories/BucketViewer.stories.ts

@@ -1,6 +1,3 @@
-// NOTE: this component was renamed with .bkp so that it's hidden
-// from the Storybook app
-
 import { type Meta, type StoryObj } from '@storybook/react';
 
 import {

packages/components/stories/Catalog.stories.ts

@@ -10,14 +10,11 @@ const meta: Meta = {
   argTypes: {
     datasets: {
       description:
-        "Array of items to be displayed on the searchable list. Must have the following properties: \n\n \
-`_id`: item's unique id \n\n \
-`url_path`: href of the item \n\n \
-`metadata`: object with a `title` property, that will be displayed as the title of the item, together with any other custom fields that might or not be faceted.",
+        'Lists of datasets to be displayed in the list, will usually be automatically available',
     },
     facets: {
       description:
-        "Array of strings, which are name of properties in the datasets' `metadata`, which are going to be faceted.",
+        'List of frontmatter fields that should be used as filters, needs to match exactly with the field name',
     },
   },
 };
@@ -34,42 +31,7 @@ export const WithoutFacets: Story = {
       {
         _id: '07026b22d49916754df1dc8ffb9ccd1c31878aae',
         url_path: 'dataset-4',
-        metadata: {
-          title: 'Detecting Abusive Albanian',
-        },
-      },
-      {
-        _id: '42c86cf3c4fbbab11d91c2a7d6dcb8f750bc4e19',
-        url_path: 'dataset-1',
-        metadata: {
-          title: 'AbuseEval v1.0',
-        },
-      },
-      {
-        _id: '80001dd32a752421fdcc64e91fbd237dc31d6bb3',
-        url_path: 'dataset-2',
-        metadata: {
-          title:
-            'Abusive Language Detection on Arabic Social Media (Al Jazeera)',
-        },
-      },
-      {
-        _id: '96649d05d8193f4333b10015af76c6562971bd8c',
-        url_path: 'dataset-3',
-        metadata: {
-          title: 'CoRAL: a Context-aware Croatian Abusive Language Dataset',
-        },
-      },
-    ],
-  },
-};
-export const WithFacets: Story = {
-  name: 'Catalog with facets',
-  args: {
-    datasets: [
-      {
-        _id: '07026b22d49916754df1dc8ffb9ccd1c31878aae',
-        url_path: 'dataset-4',
+        file_path: 'content/dataset-4/index.md',
         metadata: {
           title: 'Detecting Abusive Albanian',
           'link-to-publication': 'https://arxiv.org/abs/2107.13592',
@@ -158,6 +120,107 @@ export const WithFacets: Story = {
         },
       },
     ],
-    facets: ['language', 'platform'],
   },
 };
+;
+
+export const WithFacets: Story = {
+  name: 'Catalog with facets',
+  args: {
+    datasets: [
+      {
+        _id: '07026b22d49916754df1dc8ffb9ccd1c31878aae',
+        url_path: 'dataset-4',
+        file_path: 'content/dataset-4/index.md',
+        metadata: {
+          title: 'Detecting Abusive Albanian',
+          'link-to-publication': 'https://arxiv.org/abs/2107.13592',
+          'link-to-data': 'https://doi.org/10.6084/m9.figshare.19333298.v1',
+          'task-description':
+            'Hierarchical (offensive/not; untargeted/targeted; person/group/other)',
+          'details-of-task':
+            'Detect and categorise abusive language in social media data',
+          'size-of-dataset': 11874,
+          'percentage-abusive': 13.2,
+          language: 'Albanian',
+          'level-of-annotation': ['Posts'],
+          platform: ['Instagram', 'Youtube'],
+          medium: ['Text'],
+          reference:
+            'Nurce, E., Keci, J., Derczynski, L., 2021. Detecting Abusive Albanian. arXiv:2107.13592',
+        },
+      },
+      {
+        _id: '42c86cf3c4fbbab11d91c2a7d6dcb8f750bc4e19',
+        url_path: 'dataset-1',
+        file_path: 'content/dataset-1/index.md',
+        metadata: {
+          title: 'AbuseEval v1.0',
+          'link-to-publication':
+            'http://www.lrec-conf.org/proceedings/lrec2020/pdf/2020.lrec-1.760.pdf',
+          'link-to-data': 'https://github.com/tommasoc80/AbuseEval',
+          'task-description':
+            'Explicitness annotation of offensive and abusive content',
+          'details-of-task':
+            'Enriched versions of the OffensEval/OLID dataset with the distinction of explicit/implicit offensive messages and the new dimension for abusive messages. Labels for offensive language: EXPLICIT, IMPLICT, NOT; Labels for abusive language: EXPLICIT, IMPLICT, NOTABU',
+          'size-of-dataset': 14100,
+          'percentage-abusive': 20.75,
+          language: 'English',
+          'level-of-annotation': ['Tweets'],
+          platform: ['Twitter'],
+          medium: ['Text'],
+          reference:
+            'Caselli, T., Basile, V., Jelena, M., Inga, K., and Michael, G. 2020. "I feel offended, don’t be abusive! implicit/explicit messages in offensive and abusive language". The 12th Language Resources and Evaluation Conference (pp. 6193-6202). European Language Resources Association.',
+        },
+      },
+      {
+        _id: '80001dd32a752421fdcc64e91fbd237dc31d6bb3',
+        url_path: 'dataset-2',
+        file_path: 'content/dataset-2/index.md',
+        metadata: {
+          title:
+            'Abusive Language Detection on Arabic Social Media (Al Jazeera)',
+          'link-to-publication': 'https://www.aclweb.org/anthology/W17-3008',
+          'link-to-data':
+            'http://alt.qcri.org/~hmubarak/offensive/AJCommentsClassification-CF.xlsx',
+          'task-description':
+            'Ternary (Obscene, Offensive but not obscene, Clean)',
+          'details-of-task': 'Incivility',
+          'size-of-dataset': 32000,
+          'percentage-abusive': 0.81,
+          language: 'Arabic',
+          'level-of-annotation': ['Posts'],
+          platform: ['AlJazeera'],
+          medium: ['Text'],
+          reference:
+            'Mubarak, H., Darwish, K. and Magdy, W., 2017. Abusive Language Detection on Arabic Social Media. In: Proceedings of the First Workshop on Abusive Language Online. Vancouver, Canada: Association for Computational Linguistics, pp.52-56.',
+        },
+      },
+      {
+        _id: '96649d05d8193f4333b10015af76c6562971bd8c',
+        url_path: 'dataset-3',
+        file_path: 'content/dataset-3/index.md',
+        metadata: {
+          title: 'CoRAL: a Context-aware Croatian Abusive Language Dataset',
+          'link-to-publication':
+            'https://aclanthology.org/2022.findings-aacl.21/',
+          'link-to-data':
+            'https://github.com/shekharRavi/CoRAL-dataset-Findings-of-the-ACL-AACL-IJCNLP-2022',
+          'task-description':
+            'Multi-class based on context dependency categories (CDC)',
+          'details-of-task': 'Detectioning CDC from abusive comments',
+          'size-of-dataset': 2240,
+          'percentage-abusive': 100,
+          language: 'Croatian',
+          'level-of-annotation': ['Posts'],
+          platform: ['Posts'],
+          medium: ['Newspaper Comments'],
+          reference:
+            'Ravi Shekhar, Mladen Karan and Matthew Purver (2022). CoRAL: a Context-aware Croatian Abusive Language Dataset. Findings of the ACL: AACL-IJCNLP.',
+        },
+      },
+    ],
+    facets: ['language', 'platform']
+  },
+};
+;

packages/components/stories/Excel.stories.ts

@@ -4,13 +4,13 @@ import { Excel, ExcelProps } from '../src/components/Excel';
 
 // More on how to set up stories at: https://storybook.js.org/docs/react/writing-stories/introduction
 const meta: Meta = {
-  title: 'Components/Tabular/Excel',
+  title: 'Components/Excel',
   component: Excel,
   tags: ['autodocs'],
   argTypes: {
-    data: {
+    url: {
       description:
-        'Object with a `url` property pointing to the Excel file to be displayed, e.g.: `{ url: "https://url.to/data.csv" }`',
+        'Url of the file to be displayed e.g.: "https://url.to/data.csv"',
     },
   },
 };
@@ -22,17 +22,13 @@ type Story = StoryObj<ExcelProps>;
 export const SingleSheet: Story = {
   name: 'Excel file with just one sheet',
   args: {
-    data: {
-      url: 'https://sheetjs.com/pres.xlsx',
-    },
+    url: 'https://sheetjs.com/pres.xlsx',
   },
 };
 
 export const MultipleSheet: Story = {
   name: 'Excel file with multiple sheets',
   args: {
-    data: {
-      url: 'https://storage.portaljs.org/IC-Gantt-Chart-Project-Template-8857.xlsx',
-    },
+    url: 'https://storage.portaljs.org/IC-Gantt-Chart-Project-Template-8857.xlsx',
   },
 };

packages/components/stories/FlatUiTable.stories.ts

@@ -4,31 +4,29 @@ import { FlatUiTable, FlatUiTableProps } from '../src/components/FlatUiTable';
 
 // More on how to set up stories at: https://storybook.js.org/docs/react/writing-stories/introduction
 const meta: Meta = {
-  title: 'Components/Tabular/FlatUiTable',
+  title: 'Components/FlatUiTable',
   component: FlatUiTable,
   tags: ['autodocs'],
   argTypes: {
     data: {
       description:
-        'Data to be displayed. \n\n \
-Must be an object with one of the following properties: `url`, `values` or `csv` \n\n \
-`url`: URL pointing to a CSV file. \n\n \
-`values`: array of objects. \n\n \
-`csv`: string with valid CSV. \n\n \
-',
+        'Data to be displayed in the table, must be setup as an array of key value pairs',
+    },
+    csv: {
+      description: 'CSV data as string.',
+    },
+    url: {
+      description:
+        'Fetch the data from a CSV file remotely. only the first 5MB of data will be displayed',
     },
     bytes: {
       description:
-        'Fetch the data from a CSV file remotely. Only the first <bytes> of data will be displayed. Defaults to 5MB.',
+        'Fetch the data from a CSV file remotely. only the first <bytes> of data will be displayed',
     },
     parsingConfig: {
       description:
         'Configuration for parsing the CSV data. See https://www.papaparse.com/docs#config for more details',
     },
-    uniqueId: {
-      description:
-        'Provide a unique ID to help with cache revalidation of the fetched data.',
-    },
   },
 };
 
@@ -38,40 +36,34 @@ type Story = StoryObj<FlatUiTableProps>;
 
 // More on writing stories with args: https://storybook.js.org/docs/react/writing-stories/args
 export const FromColumnsAndData: Story = {
-  name: 'Table from array or objects',
+  name: 'Table data',
   args: {
-    data: {
-      values: [
-        { id: 1, lastName: 'Snow', firstName: 'Jon', age: 35 },
-        { id: 2, lastName: 'Lannister', firstName: 'Cersei', age: 42 },
-        { id: 3, lastName: 'Lannister', firstName: 'Jaime', age: 45 },
-        { id: 4, lastName: 'Stark', firstName: 'Arya', age: 16 },
-        { id: 7, lastName: 'Clifford', firstName: 'Ferrara', age: 44 },
-        { id: 8, lastName: 'Frances', firstName: 'Rossini', age: 36 },
-        { id: 9, lastName: 'Roxie', firstName: 'Harvey', age: 65 },
-      ],
-    },
+    data: [
+      { id: 1, lastName: 'Snow', firstName: 'Jon', age: 35 },
+      { id: 2, lastName: 'Lannister', firstName: 'Cersei', age: 42 },
+      { id: 3, lastName: 'Lannister', firstName: 'Jaime', age: 45 },
+      { id: 4, lastName: 'Stark', firstName: 'Arya', age: 16 },
+      { id: 7, lastName: 'Clifford', firstName: 'Ferrara', age: 44 },
+      { id: 8, lastName: 'Frances', firstName: 'Rossini', age: 36 },
+      { id: 9, lastName: 'Roxie', firstName: 'Harvey', age: 65 },
+    ],
   },
 };
 
 export const FromRawCSV: Story = {
-  name: 'Table from inline CSV',
+  name: 'Table from raw CSV',
   args: {
-    data: {
-      csv: `
+    rawCsv: `
     Year,Temp Anomaly
     1850,-0.418
     2020,0.923
     `,
-    },
   },
 };
 
 export const FromURL: Story = {
   name: 'Table from URL',
   args: {
-    data: {
-      url: 'https://storage.openspending.org/alberta-budget/__os_imported__alberta_total.csv',
-    },
+    url: 'https://storage.openspending.org/alberta-budget/__os_imported__alberta_total.csv',
   },
 };

packages/components/stories/Iframe.stories.ts

@@ -3,17 +3,17 @@ import { type Meta, type StoryObj } from '@storybook/react';
 import { Iframe, IframeProps } from '../src/components/Iframe';
 
 const meta: Meta = {
-  title: 'Components/Embedding/Iframe',
+  title: 'Components/Iframe',
   component: Iframe,
   tags: ['autodocs'],
   argTypes: {
-    data: {
+    url: {
       description:
-        'Object with a `url` property pointing to the page to be embeded.',
+        'Page to display inside of the component',
     },
     style: {
       description:
-        'Style object of the component. See example at https://react.dev/learn#displaying-data. Defaults to `{ width: "100%", height: "100%" }`',
+        'Style of the component',
     },
   },
 };
@@ -25,9 +25,7 @@ type Story = StoryObj<IframeProps>;
 export const Normal: Story = {
   name: 'Iframe',
   args: {
-    data: {
-      url: 'https://app.powerbi.com/view?r=eyJrIjoiYzBmN2Q2MzYtYzE3MS00ODkxLWE5OWMtZTQ2MjBlMDljMDk4IiwidCI6Ijk1M2IwZjgzLTFjZTYtNDVjMy04MmM5LTFkODQ3ZTM3MjMzOSIsImMiOjh9',
-    },
-    style: { width: `100%`, height: `100%` },
+    url: 'https://app.powerbi.com/view?r=eyJrIjoiYzBmN2Q2MzYtYzE3MS00ODkxLWE5OWMtZTQ2MjBlMDljMDk4IiwidCI6Ijk1M2IwZjgzLTFjZTYtNDVjMy04MmM5LTFkODQ3ZTM3MjMzOSIsImMiOjh9',
+    style: {width: `100%`, height: `100%`}
   },
 };

packages/components/stories/LineChart.stories.ts

@@ -4,36 +4,37 @@ import { LineChart, LineChartProps } from '../src/components/LineChart';
 
 // More on how to set up stories at: https://storybook.js.org/docs/react/writing-stories/introduction
 const meta: Meta = {
-  title: 'Components/Charts/LineChart',
+  title: 'Components/LineChart',
   component: LineChart,
   tags: ['autodocs'],
   argTypes: {
     data: {
       description:
-        'Data to be displayed. \n\n \
-Must be an object with one of the following properties: `url` or `values` \n\n \
-`url`: URL pointing to a CSV file. \n\n \
-`values`: array of objects \n\n',
+        'Data to be displayed.\n\n E.g.: [["1990", 1], ["1991", 2]] \n\nOR\n\n "https://url.to/data.csv"',
     },
     title: {
-      description: 'Title to display on the chart.',
+      description: 'Title to display on the chart. Optional.',
     },
     xAxis: {
       description:
-        'Name of the column header or object property that represents the X-axis on the data.',
+        'Name of the X axis on the data. Required when the "data" parameter is an URL.',
     },
     xAxisType: {
-      description: 'Type of the X-axis.',
+      description: 'Type of the X axis',
     },
     xAxisTimeUnit: {
-      description: 'Time unit of the X-axis, in case its type is `temporal.`',
+      description: 'Time unit of the X axis (optional)',
     },
     yAxis: {
       description:
-        'Name of the column header or object property that represents the Y-axis on the data.',
+        'Name of the Y axis on the data. Required when the "data" parameter is an URL.',
     },
     yAxisType: {
-      description: 'Type of the Y-axis',
+      description: 'Type of the Y axis',
+    },
+    fullWidth: {
+      description:
+        'Whether the component should be rendered as full bleed or not',
     },
   },
 };
@@ -46,27 +47,21 @@ type Story = StoryObj<LineChartProps>;
 export const FromDataPoints: Story = {
   name: 'Line chart from array of data points',
   args: {
-    data: {
-      values: [
-        { year: '1850', value: -0.41765878 },
-        { year: '1851', value: -0.2333498 },
-        { year: '1852', value: -0.22939907 },
-        { year: '1853', value: -0.27035445 },
-        { year: '1854', value: -0.29163003 },
-      ],
-    },
-    xAxis: 'year',
-    yAxis: 'value',
+    data: [
+      ['1850', -0.41765878],
+      ['1851', -0.2333498],
+      ['1852', -0.22939907],
+      ['1853', -0.27035445],
+      ['1854', -0.29163003],
+    ],
   },
 };
 
 export const FromURL: Story = {
   name: 'Line chart from URL',
   args: {
-    data: {
-      url: 'https://raw.githubusercontent.com/datasets/oil-prices/main/data/wti-year.csv',
-    },
     title: 'Oil Price x Year',
+    data: 'https://raw.githubusercontent.com/datasets/oil-prices/main/data/wti-year.csv',
     xAxis: 'Date',
     yAxis: 'Price',
   },

packages/components/stories/Map.stories.ts

@@ -4,34 +4,29 @@ import { Map, MapProps } from '../src/components/Map';
 
 // More on how to set up stories at: https://storybook.js.org/docs/react/writing-stories/introduction
 const meta: Meta = {
-  title: 'Components/Geospatial/Map',
+  title: 'Components/Map',
   component: Map,
   tags: ['autodocs'],
   argTypes: {
     layers: {
       description:
-        'Array of layers to be displayed on the map. Should be an object with: \n\n \
-`data`: object with either a `url` property pointing to a GeoJSON file or a `geojson` property with a GeoJSON object. \n\n \
-`name`: name of the layer. \n\n \
-`colorscale`: object with a `starting` and `ending` colors that will be used to create a gradient and color the map. \n\n \
-`tooltip`: `true` to show all available features on the tooltip, object with a `propNames` property as an array of strings to choose which features to display. \n\n',
+        'Data to be displayed.\n\n GeoJSON Object \n\nOR\n\n URL to GeoJSON Object',
     },
     title: {
-      description: 'Title to display on the map.',
+      description: 'Title to display on the map. Optional.',
     },
     center: {
       description: 'Initial coordinates of the center of the map',
     },
     zoom: {
-      description: 'Initial zoom level',
+      description: 'Zoom level',
     },
     style: {
-      description: "CSS styles to be applied to the map's container.",
+      description: "Styles for the container"
     },
     autoZoomConfiguration: {
-      description:
-        "Pass a layer's name to automatically zoom to the bounding area of a layer.",
-    },
+      description: "Configuration to auto zoom in the specified layer data"
+    }
   },
 };
 
@@ -45,9 +40,7 @@ export const GeoJSONPolygons: Story = {
   args: {
     layers: [
       {
-        data: {
-          url: 'https://d2ad6b4ur7yvpq.cloudfront.net/naturalearth-3.3.0/ne_10m_geography_marine_polys.geojson',
-        },
+        data: 'https://d2ad6b4ur7yvpq.cloudfront.net/naturalearth-3.3.0/ne_10m_geography_marine_polys.geojson',
         name: 'Polygons',
         tooltip: { propNames: ['name'] },
         colorScale: {
@@ -67,9 +60,7 @@ export const GeoJSONPoints: Story = {
   args: {
     layers: [
       {
-        data: {
-          url: 'https://opendata.arcgis.com/datasets/9c58741995174fbcb017cf46c8a42f4b_25.geojson',
-        },
+        data: 'https://opendata.arcgis.com/datasets/9c58741995174fbcb017cf46c8a42f4b_25.geojson',
         name: 'Points',
         tooltip: { propNames: ['Location'] },
       },
@@ -85,16 +76,12 @@ export const GeoJSONMultipleLayers: Story = {
   args: {
     layers: [
       {
-        data: {
-          url: 'https://opendata.arcgis.com/datasets/9c58741995174fbcb017cf46c8a42f4b_25.geojson',
-        },
+        data: 'https://opendata.arcgis.com/datasets/9c58741995174fbcb017cf46c8a42f4b_25.geojson',
         name: 'Points',
         tooltip: true,
       },
       {
-        data: {
-          url: 'https://d2ad6b4ur7yvpq.cloudfront.net/naturalearth-3.3.0/ne_10m_geography_marine_polys.geojson',
-        },
+        data: 'https://d2ad6b4ur7yvpq.cloudfront.net/naturalearth-3.3.0/ne_10m_geography_marine_polys.geojson',
         name: 'Polygons',
         tooltip: true,
         colorScale: {
@@ -107,23 +94,19 @@ export const GeoJSONMultipleLayers: Story = {
     center: { latitude: 45, longitude: 0 },
     zoom: 2,
   },
-};
+}
 
 export const GeoJSONMultipleLayersWithAutoZoomInSpecifiedLayer: Story = {
   name: 'GeoJSON polygons and points map with auto zoom in the points layer',
   args: {
     layers: [
       {
-        data: {
-          url: 'https://opendata.arcgis.com/datasets/9c58741995174fbcb017cf46c8a42f4b_25.geojson',
-        },
+        data: 'https://opendata.arcgis.com/datasets/9c58741995174fbcb017cf46c8a42f4b_25.geojson',
         name: 'Points',
         tooltip: true,
       },
       {
-        data: {
-          url: 'https://d2ad6b4ur7yvpq.cloudfront.net/naturalearth-3.3.0/ne_10m_geography_marine_polys.geojson',
-        },
+        data: 'https://d2ad6b4ur7yvpq.cloudfront.net/naturalearth-3.3.0/ne_10m_geography_marine_polys.geojson',
         name: 'Polygons',
         tooltip: true,
         colorScale: {
@@ -136,7 +119,7 @@ export const GeoJSONMultipleLayersWithAutoZoomInSpecifiedLayer: Story = {
     center: { latitude: 45, longitude: 0 },
     zoom: 2,
     autoZoomConfiguration: {
-      layerName: 'Points',
-    },
+      layerName: 'Points'
+    }
   },
 };

packages/components/stories/OpenLayers.stories.tsx

@@ -1,6 +1,3 @@
-// NOTE: this component was renamed with .bkp so that it's hidden
-// from the Storybook app
-
 import type { Meta, StoryObj } from '@storybook/react';
 import React from 'react';
 import OpenLayers from '../src/components/OpenLayers/OpenLayers';

packages/components/stories/PdfViewer.stories.ts

@@ -3,21 +3,19 @@ import type { Meta, StoryObj } from '@storybook/react';
 import { PdfViewer, PdfViewerProps } from '../src/components/PdfViewer';
 
 const meta: Meta = {
-  title: 'Components/Embedding/PdfViewer',
+  title: 'Components/PdfViewer',
   component: PdfViewer,
   tags: ['autodocs'],
   argTypes: {
-    data: {
-      description:
-        'Object with a `url` property pointing to the PDF file to be displayed, e.g.: `{ url: "https://cdn.filestackcontent.com/wcrjf9qPTCKXV3hMXDwK" }`.',
+    url: {
+      description: 'URL to PDF file',
     },
     parentClassName: {
-      description:
-        'HTML classes to be applied to the container of the PDF viewer. [Tailwind](https://tailwindcss.com/) classes, such as `h-96` to define the height of the component, can be used on this field.',
+      description: 'Classname for the parent div of the pdf viewer',
     },
-    layout: {
+    layour: {
       description:
-        'Set to `true` if you want to display a layout with zoom level, page count, printing button and other controls.',
+        'Set to true if you want to have a layout with zoom level, page count, printing button etc',
       defaultValue: false,
     },
   },
@@ -27,23 +25,26 @@ export default meta;
 
 type Story = StoryObj<PdfViewerProps>;
 
-export const PdfViewerStoryWithoutControlsLayout: Story = {
-  name: 'PDF Viewer without controls layout',
+export const PdfViewerStory: Story = {
+  name: 'PdfViewer',
   args: {
-    data: {
-      url: 'https://cdn.filestackcontent.com/wcrjf9qPTCKXV3hMXDwK',
-    },
-    parentClassName: 'h-96',
+    url: 'https://cdn.filestackcontent.com/wcrjf9qPTCKXV3hMXDwK',
   },
 };
 
-export const PdfViewerStoryWithControlsLayout: Story = {
-  name: 'PdfViewer with controls layout',
+export const PdfViewerStoryWithLayout: Story = {
+  name: 'PdfViewer with the default layout',
   args: {
-    data: {
-      url: 'https://cdn.filestackcontent.com/wcrjf9qPTCKXV3hMXDwK',
-    },
+    url: 'https://cdn.filestackcontent.com/wcrjf9qPTCKXV3hMXDwK',
+    layout: true,
+  },
+};
+
+export const PdfViewerStoryWithHeight: Story = {
+  name: 'PdfViewer with a custom height',
+  args: {
+    url: 'https://cdn.filestackcontent.com/wcrjf9qPTCKXV3hMXDwK',
+    parentClassName: 'h-96',
     layout: true,
-    parentClassName: 'h-96',
   },
 };

packages/components/stories/Plotly.stories.ts

@@ -1,49 +0,0 @@
-import type { Meta, StoryObj } from '@storybook/react';
-
-import { Plotly } from '../src/components/Plotly';
-
-// More on how to set up stories at: https://storybook.js.org/docs/react/writing-stories/introduction
-const meta: Meta = {
-  title: 'Components/Charts/Plotly',
-  component: Plotly,
-  tags: ['autodocs'],
-  argTypes: {
-    data: {
-      description:
-        "Plotly's `data` prop. You can find references on how to use these props at https://github.com/plotly/react-plotly.js/#basic-props.",
-    },
-    layout: {
-      description:
-        "Plotly's `layout` prop. You can find references on how to use these props at https://github.com/plotly/react-plotly.js/#basic-props.",
-    },
-  },
-};
-
-export default meta;
-
-type Story = StoryObj<any>;
-
-// More on writing stories with args: https://storybook.js.org/docs/react/writing-stories/args
-export const Primary: Story = {
-  name: 'Line chart',
-  args: {
-    data: [
-      {
-        x: [1, 2, 3],
-        y: [2, 6, 3],
-        type: 'scatter',
-        mode: 'lines+markers',
-        marker: { color: 'red' },
-      },
-    ],
-    layout: {
-      title: 'Chart built with Plotly',
-      xaxis: {
-        title: 'x Axis',
-      },
-      yaxis: {
-        title: 'y Axis',
-      },
-    },
-  },
-};

packages/components/stories/PlotlyBarChart.stories.ts

@@ -1,102 +0,0 @@
-import type { Meta, StoryObj } from '@storybook/react';
-
-import {
-  PlotlyBarChart,
-  PlotlyBarChartProps,
-} from '../src/components/PlotlyBarChart';
-
-// More on how to set up stories at: https://storybook.js.org/docs/react/writing-stories/introduction
-const meta: Meta = {
-  title: 'Components/Charts/PlotlyBarChart',
-  component: PlotlyBarChart,
-  tags: ['autodocs'],
-  argTypes: {
-    data: {
-      description:
-        'Data to be displayed. \n\n \
-Must be an object with one of the following properties: `url`, `values` or `csv` \n\n \
-`url`: URL pointing to a CSV file. \n\n \
-`values`: array of objects (check out [this example](/?path=/story/components-plotlybarchart--from-data-points)) \n\n \
-`csv`: string with valid CSV (check out [this example](/?path=/story/components-plotlybarchart--from-inline-csv)) \n\n \
-',
-    },
-    bytes: {
-      // TODO: likely this should be an extra option on the data parameter,
-      // specific to URLs
-      description:
-        "How many bytes to read from the url so that the entire file doesn's have to be fetched.",
-    },
-    parsingConfig: {
-      description:
-        'If using URL or CSV, this parsing config will be used to parse the data. Check https://www.papaparse.com/ for more info.',
-    },
-    title: {
-      description: 'Title to display on the chart.',
-    },
-    // TODO: commented out because this doesn't work
-    // lineLabel: {
-    //   description:
-    //     'Label to display on the line, Optional, will use yAxis if not provided',
-    // },
-    xAxis: {
-      description:
-        'Name of the column header or object property that represents the X-axis on the data.',
-    },
-    yAxis: {
-      description:
-        'Name of the column header or object property that represents the Y-axis on the data.',
-    },
-    uniqueId: {
-      description: 'Provide a unique ID to help with cache revalidation of the fetched data.'
-    }
-  },
-};
-
-export default meta;
-
-type Story = StoryObj<PlotlyBarChartProps>;
-
-export const FromDataPoints: Story = {
-  name: 'Bar chart from array of data points',
-  args: {
-    data: {
-      values: [
-        { year: '1850', temperature: -0.41765878 },
-        { year: '1851', temperature: -0.2333498 },
-        { year: '1852', temperature: -0.22939907 },
-        { year: '1853', temperature: -0.27035445 },
-        { year: '1854', temperature: -0.29163003 },
-      ],
-    },
-    xAxis: 'year',
-    yAxis: 'temperature',
-  },
-};
-
-export const FromURL: Story = {
-  name: 'Bar chart from URL',
-  args: {
-    title: 'Apple Stock Prices',
-    data: {
-      url: 'https://raw.githubusercontent.com/plotly/datasets/master/finance-charts-apple.csv',
-    },
-    xAxis: 'Date',
-    yAxis: 'AAPL.Open',
-  },
-};
-
-export const FromInlineCSV: Story = {
-  name: 'Bar chart from inline CSV',
-  args: {
-    title: 'Apple Stock Prices',
-    data: {
-      csv: `Date,AAPL.Open,AAPL.High,AAPL.Low,AAPL.Close,AAPL.Volume,AAPL.Adjusted,dn,mavg,up,direction
-2015-02-17,127.489998,128.880005,126.919998,127.830002,63152400,122.905254,106.7410523,117.9276669,129.1142814,Increasing
-2015-02-18,127.629997,128.779999,127.449997,128.720001,44891700,123.760965,107.842423,118.9403335,130.0382439,Increasing
-2015-02-19,128.479996,129.029999,128.330002,128.449997,37362400,123.501363,108.8942449,119.8891668,130.8840887,Decreasing
-2015-02-20,128.619995,129.5,128.050003,129.5,48948400,124.510914,109.7854494,120.7635001,131.7415509,Increasing`,
-    },
-    xAxis: 'Date',
-    yAxis: 'AAPL.Open',
-  },
-};

packages/components/stories/PlotlyLineChart.stories.ts

@@ -1,101 +0,0 @@
-import type { Meta, StoryObj } from '@storybook/react';
-
-import {
-  PlotlyLineChart,
-  PlotlyLineChartProps,
-} from '../src/components/PlotlyLineChart';
-
-const meta: Meta = {
-  title: 'Components/Charts/PlotlyLineChart',
-  component: PlotlyLineChart,
-  tags: ['autodocs'],
-  argTypes: {
-    data: {
-      description:
-        'Data to be displayed. \n\n \
-Must be an object with one of the following properties: `url`, `values` or `csv` \n\n \
-`url`: URL pointing to a CSV file. \n\n \
-`values`: array of objects. \n\n \
-`csv`: string with valid CSV. \n\n \
-',
-    },
-    bytes: {
-      // TODO: likely this should be an extra option on the data parameter,
-      // specific to URLs
-      description:
-        "How many bytes to read from the url so that the entire file doesn's have to be fetched.",
-    },
-    parsingConfig: {
-      description:
-        'If using URL or CSV, this parsing config will be used to parse the data. Check https://www.papaparse.com/ for more info',
-    },
-    title: {
-      description: 'Title to display on the chart.',
-    },
-    lineLabel: {
-      description:
-        'Label to display on the line, will use yAxis if not provided',
-    },
-    xAxis: {
-      description:
-        'Name of the column header or object property that represents the X-axis on the data.',
-    },
-    yAxis: {
-      description:
-        'Name of the column header or object property that represents the Y-axis on the data.',
-    },
-    uniqueId: {
-      description:
-        'Provide a unique ID to help with cache revalidation of the fetched data.',
-    },
-  },
-};
-
-export default meta;
-
-type Story = StoryObj<PlotlyLineChartProps>;
-
-export const FromDataPoints: Story = {
-  name: 'Line chart from array of data points',
-  args: {
-    data: {
-      values: [
-        { year: '1850', temperature: -0.41765878 },
-        { year: '1851', temperature: -0.2333498 },
-        { year: '1852', temperature: -0.22939907 },
-        { year: '1853', temperature: -0.27035445 },
-        { year: '1854', temperature: -0.29163003 },
-      ],
-    },
-    xAxis: 'year',
-    yAxis: 'temperature',
-  },
-};
-
-export const FromURL: Story = {
-  name: 'Line chart from URL',
-  args: {
-    title: 'Oil Price x Year',
-    data: {
-      url: 'https://raw.githubusercontent.com/datasets/oil-prices/main/data/wti-year.csv',
-    },
-    xAxis: 'Date',
-    yAxis: 'Price',
-  },
-};
-
-export const FromInlineCSV: Story = {
-  name: 'Bar chart from inline CSV',
-  args: {
-    title: 'Apple Stock Prices',
-    data: {
-      csv: `Date,AAPL.Open,AAPL.High,AAPL.Low,AAPL.Close,AAPL.Volume,AAPL.Adjusted,dn,mavg,up,direction
-2015-02-17,127.489998,128.880005,126.919998,127.830002,63152400,122.905254,106.7410523,117.9276669,129.1142814,Increasing
-2015-02-18,127.629997,128.779999,127.449997,128.720001,44891700,123.760965,107.842423,118.9403335,130.0382439,Increasing
-2015-02-19,128.479996,129.029999,128.330002,128.449997,37362400,123.501363,108.8942449,119.8891668,130.8840887,Decreasing
-2015-02-20,128.619995,129.5,128.050003,129.5,48948400,124.510914,109.7854494,120.7635001,131.7415509,Increasing`,
-    },
-    xAxis: 'Date',
-    yAxis: 'AAPL.Open',
-  },
-};

packages/components/stories/Table.stories.ts

@@ -1,13 +1,10 @@
-// NOTE: this component was renamed with .bkp so that it's hidden
-// from the Storybook app
-
 import type { Meta, StoryObj } from '@storybook/react';
 
 import { Table, TableProps } from '../src/components/Table';
 
 // More on how to set up stories at: https://storybook.js.org/docs/react/writing-stories/introduction
 const meta: Meta = {
-  title: 'Components/Tabular/Table',
+  title: 'Components/Table',
   component: Table,
   tags: ['autodocs'],
   argTypes: {

packages/components/stories/Vega.stories.ts

@@ -4,19 +4,9 @@ import { Vega } from '../src/components/Vega';
 
 // More on how to set up stories at: https://storybook.js.org/docs/react/writing-stories/introduction
 const meta: Meta = {
-  title: 'Components/Charts/Vega',
+  title: 'Components/Vega',
   component: Vega,
   tags: ['autodocs'],
-  argTypes: {
-    data: {
-      description:
-        "Vega's `data` prop. You can find references on how to use this prop at https://vega.github.io/vega/docs/data/",
-    },
-    spec: {
-      description:
-        "Vega's `spec` prop. You can find references on how to use this prop at https://vega.github.io/vega/docs/specification/",
-    },
-  },
 };
 
 export default meta;
@@ -25,7 +15,7 @@ type Story = StoryObj<any>;
 
 // More on writing stories with args: https://storybook.js.org/docs/react/writing-stories/args
 export const Primary: Story = {
-  name: 'Bar chart',
+  name: 'Chart built with Vega',
   args: {
     data: {
       table: [

packages/components/stories/VegaLite.stories.ts

@@ -4,7 +4,7 @@ import { VegaLite } from '../src/components/VegaLite';
 
 // More on how to set up stories at: https://storybook.js.org/docs/react/writing-stories/introduction
 const meta: Meta = {
-  title: 'Components/Charts/VegaLite',
+  title: 'Components/VegaLite',
   component: VegaLite,
   tags: ['autodocs'],
   argTypes: {
@@ -25,7 +25,7 @@ type Story = StoryObj<any>;
 
 // More on writing stories with args: https://storybook.js.org/docs/react/writing-stories/args
 export const Primary: Story = {
-  name: 'Bar chart',
+  name: 'Chart built with Vega Lite',
   args: {
     data: {
       table: [

packages/core/src/ui/SiteToc/SiteToc.tsx

@@ -46,8 +46,8 @@ export const SiteToc: React.FC<Props> = ({ currentPath, nav }) => {
 
   return (
     <nav data-testid="lhs-sidebar" className="flex flex-col space-y-3 text-sm">
-      {sortNavGroupChildren(nav).map((n, index) => (
-        <NavComponent key={index} item={n} isActive={false} />
+      {sortNavGroupChildren(nav).map((n) => (
+        <NavComponent item={n} isActive={false} />
       ))}
     </nav>
   );
@@ -96,8 +96,8 @@ const NavComponent: React.FC<{
             leaveTo="transform scale-95 opacity-0"
           >
             <Disclosure.Panel className="flex flex-col space-y-3 pl-5 mt-3">
-              {sortNavGroupChildren(item.children).map((subItem, index) => (
-                <NavComponent key={index} item={subItem} isActive={false} />
+              {sortNavGroupChildren(item.children).map((subItem) => (
+                <NavComponent item={subItem} isActive={false} />
               ))}
             </Disclosure.Panel>
           </Transition>

packages/remark-wiki-link/CHANGELOG.md

@@ -1,11 +1,5 @@
 # @portaljs/remark-wiki-link
 
-## 1.2.0
-
-### Minor Changes
-
-- [#1084](https://github.com/datopian/datahub/pull/1084) [`57952e08`](https://github.com/datopian/datahub/commit/57952e0817770138881e7492dc9f43e9910b56a8) Thanks [@mohamedsalem401](https://github.com/mohamedsalem401)! - Add image resize feature
-
 ## 1.1.2
 
 ### Patch Changes

packages/remark-wiki-link/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@portaljs/remark-wiki-link",
-  "version": "1.2.0",
+  "version": "1.1.2",
   "description": "Parse and render wiki-style links in markdown especially Obsidian style links.",
   "repository": {
     "type": "git",

packages/remark-wiki-link/src/lib/fromMarkdown.ts

@@ -1,23 +1,23 @@
-import { isSupportedFileFormat } from './isSupportedFileFormat';
+import { isSupportedFileFormat } from "./isSupportedFileFormat";
 
 const defaultWikiLinkResolver = (target: string) => {
   // for [[#heading]] links
   if (!target) {
     return [];
   }
-  let permalink = target.replace(/\/index$/, '');
+  let permalink = target.replace(/\/index$/, "");
   // TODO what to do with [[index]] link?
   if (permalink.length === 0) {
-    permalink = '/';
+    permalink = "/";
   }
   return [permalink];
 };
 
 export interface FromMarkdownOptions {
   pathFormat?:
-    | 'raw' // default; use for regular relative or absolute paths
-    | 'obsidian-absolute' // use for Obsidian-style absolute paths (with no leading slash)
-    | 'obsidian-short'; // use for Obsidian-style shortened paths (shortest path possible)
+  | "raw" // default; use for regular relative or absolute paths
+  | "obsidian-absolute" // use for Obsidian-style absolute paths (with no leading slash)
+  | "obsidian-short"; // use for Obsidian-style shortened paths (shortest path possible)
   permalinks?: string[]; // list of permalinks to match possible permalinks of a wiki link against
   wikiLinkResolver?: (name: string) => string[]; // function to resolve wiki links to an array of possible permalinks
   newClassName?: string; // class name to add to links that don't have a matching permalink
@@ -25,23 +25,14 @@ export interface FromMarkdownOptions {
   hrefTemplate?: (permalink: string) => string; // function to generate the href attribute of a link
 }
 
-export function getImageSize(size: string) {
-  // eslint-disable-next-line prefer-const
-  let [width, height] = size.split('x');
-
-  if (!height) height = width;
-
-  return { width, height };
-}
-
 // mdas-util-from-markdown extension
 // https://github.com/syntax-tree/mdast-util-from-markdown#extension
 function fromMarkdown(opts: FromMarkdownOptions = {}) {
-  const pathFormat = opts.pathFormat || 'raw';
+  const pathFormat = opts.pathFormat || "raw";
   const permalinks = opts.permalinks || [];
   const wikiLinkResolver = opts.wikiLinkResolver || defaultWikiLinkResolver;
-  const newClassName = opts.newClassName || 'new';
-  const wikiLinkClassName = opts.wikiLinkClassName || 'internal';
+  const newClassName = opts.newClassName || "new";
+  const wikiLinkClassName = opts.wikiLinkClassName || "internal";
   const defaultHrefTemplate = (permalink: string) => permalink;
 
   const hrefTemplate = opts.hrefTemplate || defaultHrefTemplate;
@@ -53,9 +44,9 @@ function fromMarkdown(opts: FromMarkdownOptions = {}) {
   function enterWikiLink(token) {
     this.enter(
       {
-        type: 'wikiLink',
+        type: "wikiLink",
         data: {
-          isEmbed: token.isType === 'embed',
+          isEmbed: token.isType === "embed",
           target: null, // the target of the link, e.g. "Foo Bar#Heading" in "[[Foo Bar#Heading]]"
           alias: null, // the alias of the link, e.g. "Foo" in "[[Foo Bar|Foo]]"
           permalink: null, // TODO shouldn't this be named just "link"?
@@ -89,18 +80,18 @@ function fromMarkdown(opts: FromMarkdownOptions = {}) {
     } = wikiLink;
     // eslint-disable-next-line no-useless-escape
     const wikiLinkWithHeadingPattern = /^(.*?)(#.*)?$/u;
-    const [, path, heading = ''] = target.match(wikiLinkWithHeadingPattern);
+    const [, path, heading = ""] = target.match(wikiLinkWithHeadingPattern);
 
     const possibleWikiLinkPermalinks = wikiLinkResolver(path);
 
     const matchingPermalink = permalinks.find((e) => {
       return possibleWikiLinkPermalinks.find((p) => {
-        if (pathFormat === 'obsidian-short') {
+        if (pathFormat === "obsidian-short") {
           if (e === p || e.endsWith(p)) {
             return true;
           }
-        } else if (pathFormat === 'obsidian-absolute') {
-          if (e === '/' + p) {
+        } else if (pathFormat === "obsidian-absolute") {
+          if (e === "/" + p) {
             return true;
           }
         } else {
@@ -115,19 +106,21 @@ function fromMarkdown(opts: FromMarkdownOptions = {}) {
     // TODO this is ugly
     const link =
       matchingPermalink ||
-      (pathFormat === 'obsidian-absolute'
-        ? '/' + possibleWikiLinkPermalinks[0]
+      (pathFormat === "obsidian-absolute"
+        ? "/" + possibleWikiLinkPermalinks[0]
         : possibleWikiLinkPermalinks[0]) ||
-      '';
+      "";
+    const isExternal = /^https?:\/\//.test(link);
 
     wikiLink.data.exists = !!matchingPermalink;
     wikiLink.data.permalink = link;
+
     // remove leading # if the target is a heading on the same page
-    const displayName = alias || target.replace(/^#/, '');
-    const headingId = heading.replace(/\s+/g, '-').toLowerCase();
+    const displayName = alias || target.replace(/^#/, "");
+    const headingId = heading.replace(/\s+/g, "-").toLowerCase();
     let classNames = wikiLinkClassName;
     if (!matchingPermalink) {
-      classNames += ' ' + newClassName;
+      classNames += " " + newClassName;
     }
 
     if (isEmbed) {
@@ -135,55 +128,48 @@ function fromMarkdown(opts: FromMarkdownOptions = {}) {
       if (!isSupportedFormat) {
         // Temporarily render note transclusion as a regular wiki link
         if (!format) {
-          wikiLink.data.hName = 'a';
+          wikiLink.data.hName = "a";
           wikiLink.data.hProperties = {
-            className: classNames + ' ' + 'transclusion',
-            href: hrefTemplate(link) + headingId,
+            className: classNames + " " + "transclusion",
+            href: hrefTemplate(link) + headingId
           };
-          wikiLink.data.hChildren = [{ type: 'text', value: displayName }];
+          wikiLink.data.hChildren = [{ type: "text", value: displayName }];
+
         } else {
-          wikiLink.data.hName = 'p';
+          wikiLink.data.hName = "p";
           wikiLink.data.hChildren = [
             {
-              type: 'text',
+              type: "text",
               value: `![[${target}]]`,
             },
           ];
         }
-      } else if (format === 'pdf') {
-        wikiLink.data.hName = 'iframe';
+      } else if (format === "pdf") {
+        wikiLink.data.hName = "iframe";
         wikiLink.data.hProperties = {
           className: classNames,
-          width: '100%',
+          width: "100%",
           src: `${hrefTemplate(link)}#toolbar=0`,
         };
       } else {
-        const hasDimensions = alias && /^\d+(x\d+)?$/.test(alias);
-        // Take the target as alt text except if alt name was provided [[target|alt text]]
-        const altText = hasDimensions || !alias ? target : alias;
-
-        wikiLink.data.hName = 'img';  
-        wikiLink.data.hProperties = {  
-            className: classNames,  
-            src: hrefTemplate(link),  
-            alt: altText
-        };  
-
-        if (hasDimensions) {  
-          const { width, height } = getImageSize(alias as string);  
-          Object.assign(wikiLink.data.hProperties, {  
-            width,  
-            height,  
-          }); 
-        }
+        wikiLink.data.hName = "img";
+        wikiLink.data.hProperties = {
+          className: classNames,
+          src: hrefTemplate(link),
+          alt: displayName,
+        };
       }
     } else {
-      wikiLink.data.hName = 'a';
+      wikiLink.data.hName = "a";
       wikiLink.data.hProperties = {
         className: classNames,
         href: hrefTemplate(link) + headingId,
+        target: "_blank",
       };
-      wikiLink.data.hChildren = [{ type: 'text', value: displayName }];
+      if(isExternal){
+        wikiLink.data.hProperties.target = "_blank"; // Open in a new tab
+      }
+      wikiLink.data.hChildren = [{ type: "text", value: displayName }];
     }
   }
 

packages/remark-wiki-link/src/lib/html.ts

@@ -1,24 +1,23 @@
-import { getImageSize } from './fromMarkdown';
-import { isSupportedFileFormat } from './isSupportedFileFormat';
+import { isSupportedFileFormat } from "./isSupportedFileFormat";
 
 const defaultWikiLinkResolver = (target: string) => {
   // for [[#heading]] links
   if (!target) {
     return [];
   }
-  let permalink = target.replace(/\/index$/, '');
+  let permalink = target.replace(/\/index$/, "");
   // TODO what to do with [[index]] link?
   if (permalink.length === 0) {
-    permalink = '/';
+    permalink = "/";
   }
   return [permalink];
 };
 
 export interface HtmlOptions {
   pathFormat?:
-  | 'raw' // default; use for regular relative or absolute paths
-  | 'obsidian-absolute' // use for Obsidian-style absolute paths (with no leading slash)
-  | 'obsidian-short'; // use for Obsidian-style shortened paths (shortest path possible)
+  | "raw" // default; use for regular relative or absolute paths
+  | "obsidian-absolute" // use for Obsidian-style absolute paths (with no leading slash)
+  | "obsidian-short"; // use for Obsidian-style shortened paths (shortest path possible)
   permalinks?: string[]; // list of permalinks to match possible permalinks of a wiki link against
   wikiLinkResolver?: (name: string) => string[]; // function to resolve wiki links to an array of possible permalinks
   newClassName?: string; // class name to add to links that don't have a matching permalink
@@ -29,11 +28,11 @@ export interface HtmlOptions {
 // Micromark HtmlExtension
 // https://github.com/micromark/micromark#htmlextension
 function html(opts: HtmlOptions = {}) {
-  const pathFormat = opts.pathFormat || 'raw';
+  const pathFormat = opts.pathFormat || "raw";
   const permalinks = opts.permalinks || [];
   const wikiLinkResolver = opts.wikiLinkResolver || defaultWikiLinkResolver;
-  const newClassName = opts.newClassName || 'new';
-  const wikiLinkClassName = opts.wikiLinkClassName || 'internal';
+  const newClassName = opts.newClassName || "new";
+  const wikiLinkClassName = opts.wikiLinkClassName || "internal";
   const defaultHrefTemplate = (permalink: string) => permalink;
   const hrefTemplate = opts.hrefTemplate || defaultHrefTemplate;
 
@@ -42,21 +41,21 @@ function html(opts: HtmlOptions = {}) {
   }
 
   function enterWikiLink() {
-    let stack = this.getData('wikiLinkStack');
-    if (!stack) this.setData('wikiLinkStack', (stack = []));
+    let stack = this.getData("wikiLinkStack");
+    if (!stack) this.setData("wikiLinkStack", (stack = []));
 
     stack.push({});
   }
 
   function exitWikiLinkTarget(token) {
     const target = this.sliceSerialize(token);
-    const current = top(this.getData('wikiLinkStack'));
+    const current = top(this.getData("wikiLinkStack"));
     current.target = target;
   }
 
   function exitWikiLinkAlias(token) {
     const alias = this.sliceSerialize(token);
-    const current = top(this.getData('wikiLinkStack'));
+    const current = top(this.getData("wikiLinkStack"));
     current.alias = alias;
   }
 
@@ -97,6 +96,9 @@ function html(opts: HtmlOptions = {}) {
         : possibleWikiLinkPermalinks[0]) ||
       "";
 
+    const isExternal = /^https?:\/\//.test(link);
+    const openInNewTab = isExternal ? 'target="_blank"' : '';
+
     // remove leading # if the target is a heading on the same page
     const displayName = alias || target.replace(/^#/, "");
     // replace spaces with dashes and lowercase headings
@@ -112,9 +114,7 @@ function html(opts: HtmlOptions = {}) {
         // Temporarily render note transclusion as a regular wiki link
         if (!format) {
           this.tag(
-            `<a href="${hrefTemplate(
-              link + headingId
-            )}" class="${classNames} transclusion">`
+            `<a href="${hrefTemplate(link + headingId)}" class="${classNames} transclusion">`
           );
           this.raw(displayName);
           this.tag("</a>");
@@ -128,18 +128,11 @@ function html(opts: HtmlOptions = {}) {
           )}#toolbar=0" class="${classNames}" />`
         );
       } else {
-        const hasDimensions = alias && /^\d+(x\d+)?$/.test(alias);
-        // Take the target as alt text except if alt name was provided [[target|alt text]]
-        const altText = hasDimensions || !alias ? target : alias;
-        let imgAttributes = `src="${hrefTemplate(
-          link
-        )}" alt="${altText}" class="${classNames}"`;
-
-        if (hasDimensions) {
-          const { width, height } = getImageSize(alias as string);
-          imgAttributes += ` width="${width}" height="${height}"`;
-        }
-        this.tag(`<img ${imgAttributes} />`);
+        this.tag(
+          `<img src="${hrefTemplate(
+            link
+          )}" alt="${displayName}" class="${classNames}" />`
+        );
       }
     } else {
       this.tag(

packages/remark-wiki-link/test/micromarkExtensionWikiLink.spec.ts

@@ -48,7 +48,7 @@ describe("micromark-extension-wiki-link", () => {
           html({
             permalinks: ["/some/folder/Wiki Link"],
             pathFormat: "obsidian-short",
-          }) as any, // TODO type fix
+          }) as any // TODO type fix
         ],
       });
       expect(serialized).toBe(
@@ -75,7 +75,7 @@ describe("micromark-extension-wiki-link", () => {
           html({
             permalinks: ["/some/folder/Wiki Link"],
             pathFormat: "obsidian-absolute",
-          }) as any, // TODO type fix
+          }) as any // TODO type fix
         ],
       });
       expect(serialized).toBe(
@@ -97,14 +97,10 @@ describe("micromark-extension-wiki-link", () => {
     });
 
     test("parses a wiki link with heading and alias", () => {
-      const serialized = micromark(
-        "[[Wiki Link#Some Heading|Alias]]",
-        "ascii",
-        {
-          extensions: [syntax()],
-          htmlExtensions: [html() as any], // TODO type fix
-        }
-      );
+      const serialized = micromark("[[Wiki Link#Some Heading|Alias]]", "ascii", {
+        extensions: [syntax()],
+        htmlExtensions: [html() as any], // TODO type fix
+      });
       // note: lowercased and hyphenated heading
       expect(serialized).toBe(
         '<p><a href="Wiki Link#some-heading" class="internal new">Alias</a></p>'
@@ -138,7 +134,7 @@ describe("micromark-extension-wiki-link", () => {
         extensions: [syntax()],
         htmlExtensions: [html() as any], // TODO type fix
       });
-      expect(serialized).toBe('<p>![[My Image.xyz]]</p>');
+      expect(serialized).toBe("<p>![[My Image.xyz]]</p>");
     });
 
     test("parses and image ambed with a matching permalink", () => {
@@ -151,28 +147,6 @@ describe("micromark-extension-wiki-link", () => {
       );
     });
 
-    // TODO: Fix alt attribute
-    test("Can identify the dimensions of the image if exists", () => {
-      const serialized = micromark("![[My Image.jpg|200]]", "ascii", {
-        extensions: [syntax()],
-        htmlExtensions: [html({ permalinks: ["My Image.jpg"] }) as any], // TODO type fix
-      });
-      expect(serialized).toBe(
-        '<p><img src="My Image.jpg" alt="My Image.jpg" class="internal" width="200" height="200" /></p>'
-        );
-      });
-      
-    // TODO: Fix alt attribute
-      test("Can identify the dimensions of the image if exists", () => {
-      const serialized = micromark("![[My Image.jpg|200x200]]", "ascii", {
-        extensions: [syntax()],
-        htmlExtensions: [html({ permalinks: ["My Image.jpg"] }) as any], // TODO type fix
-      });
-      expect(serialized).toBe(
-        '<p><img src="My Image.jpg" alt="My Image.jpg" class="internal" width="200" height="200" /></p>'
-      );
-    });
-
     test("parses an image embed with a matching permalink and Obsidian-style shortedned path", () => {
       const serialized = micromark("![[My Image.jpg]]", {
         extensions: [syntax()],
@@ -180,7 +154,7 @@ describe("micromark-extension-wiki-link", () => {
           html({
             permalinks: ["/assets/My Image.jpg"],
             pathFormat: "obsidian-short",
-          }) as any, // TODO type fix
+          }) as any // TODO type fix
         ],
       });
       expect(serialized).toBe(
@@ -215,7 +189,7 @@ describe("micromark-extension-wiki-link", () => {
         extensions: [syntax()],
         htmlExtensions: [html() as any], // TODO type fix
       });
-      expect(serialized).toBe('<p>[[Wiki Link</p>');
+      expect(serialized).toBe("<p>[[Wiki Link</p>");
     });
 
     test("doesn't parse a wiki link with one missing closing bracket", () => {
@@ -223,7 +197,7 @@ describe("micromark-extension-wiki-link", () => {
         extensions: [syntax()],
         htmlExtensions: [html() as any], // TODO type fix
       });
-      expect(serialized).toBe('<p>[[Wiki Link]</p>');
+      expect(serialized).toBe("<p>[[Wiki Link]</p>");
     });
 
     test("doesn't parse a wiki link with a missing opening bracket", () => {
@@ -231,7 +205,7 @@ describe("micromark-extension-wiki-link", () => {
         extensions: [syntax()],
         htmlExtensions: [html() as any], // TODO type fix
       });
-      expect(serialized).toBe('<p>[Wiki Link]]</p>');
+      expect(serialized).toBe("<p>[Wiki Link]]</p>");
     });
 
     test("doesn't parse a wiki link in single brackets", () => {
@@ -239,7 +213,7 @@ describe("micromark-extension-wiki-link", () => {
         extensions: [syntax()],
         htmlExtensions: [html() as any], // TODO type fix
       });
-      expect(serialized).toBe('<p>[Wiki Link]</p>');
+      expect(serialized).toBe("<p>[Wiki Link]</p>");
     });
   });
 
@@ -251,7 +225,7 @@ describe("micromark-extension-wiki-link", () => {
           html({
             newClassName: "test-new",
             wikiLinkClassName: "test-wiki-link",
-          }) as any, // TODO type fix
+          }) as any // TODO type fix
         ],
       });
       expect(serialized).toBe(
@@ -277,7 +251,7 @@ describe("micromark-extension-wiki-link", () => {
             wikiLinkResolver: (page) => [
               page.replace(/\s+/, "-").toLowerCase(),
             ],
-          }) as any, // TODO type fix
+          }) as any // TODO type fix
         ],
       });
       expect(serialized).toBe(
@@ -356,5 +330,15 @@ describe("micromark-extension-wiki-link", () => {
       });
       expect(serialized).toBe(`<p><a href="li nk-w(i)th-àcèô íã_a(n)d_underline!:ª%@'*º$ °~./\\#li-nk-w(i)th-àcèô-íã_a(n)d_underline!:ª%@'*º$-°~./\\" class="internal new">li nk-w(i)th-àcèô íã_a(n)d_underline!:ª%@'*º$ °~./\\#LI NK-W(i)th-àcèô íã_a(n)d_uNdErlinE!:ª%@'*º$ °~./\\</a></p>`);
     });
-  });
+  })
+
+  describe("External links", () => {
+    test("parses an external link that opens in a new tab", () => {
+      const serialized = micromark("[google](https://www.google.com/)", "ascii", {
+        extensions: [syntax()],
+        htmlExtensions: [html() as any],
+      });
+      expect(serialized).toBe(`<p><a href="https://www.google.com/" target="_blank">google</a></p>`);
+    });
+  })
 });

packages/remark-wiki-link/test/remarkWikiLink.spec.ts

@@ -246,28 +246,6 @@ describe("remark-wiki-link", () => {
         expect(node.data?.hName).toEqual("img");
         expect((node.data?.hProperties as any).src).toEqual("My Image.png");
         expect((node.data?.hProperties as any).alt).toEqual("My Image.png");
-        expect((node.data?.hProperties as any).width).toBeUndefined();
-        expect((node.data?.hProperties as any).height).toBeUndefined();
-      });
-    });
-
-    test("Can identify the dimensions of the image if exists", () => {
-      const processor = unified().use(markdown).use(wikiLinkPlugin);
-
-      let ast = processor.parse("![[My Image.png|132x612]]");
-      ast = processor.runSync(ast);
-
-      expect(select("wikiLink", ast)).not.toEqual(null);
-
-      visit(ast, "wikiLink", (node: Node) => {
-        expect(node.data?.isEmbed).toEqual(true);
-        expect(node.data?.target).toEqual("My Image.png");
-        expect(node.data?.permalink).toEqual("My Image.png");
-        expect(node.data?.hName).toEqual("img");
-        expect((node.data?.hProperties as any).src).toEqual("My Image.png");
-        expect((node.data?.hProperties as any).alt).toEqual("My Image.png");
-        expect((node.data?.hProperties as any).width).toBe("132");
-        expect((node.data?.hProperties as any).height).toBe("612");
       });
     });
 
@@ -387,17 +365,13 @@ describe("remark-wiki-link", () => {
     test("parses a link with special characters and symbols", () => {
       const processor = unified().use(markdown).use(wikiLinkPlugin);
 
-      let ast = processor.parse(
-        "[[li nk-w(i)th-àcèô íã_a(n)d_underline!:ª%@'*º$ °~./\\#li-nk-w(i)th-àcèô íã_a(n)D_UNDERLINE!:ª%@'*º$ °~./\\]]"
-      );
+      let ast = processor.parse("[[li nk-w(i)th-àcèô íã_a(n)d_underline!:ª%@'*º$ °~./\\#li-nk-w(i)th-àcèô íã_a(n)D_UNDERLINE!:ª%@'*º$ °~./\\]]");
       ast = processor.runSync(ast);
       expect(select("wikiLink", ast)).not.toEqual(null);
 
       visit(ast, "wikiLink", (node: Node) => {
         expect(node.data?.exists).toEqual(false);
-        expect(node.data?.permalink).toEqual(
-          "li nk-w(i)th-àcèô íã_a(n)d_underline!:ª%@'*º$ °~./\\"
-        );
+        expect(node.data?.permalink).toEqual("li nk-w(i)th-àcèô íã_a(n)d_underline!:ª%@'*º$ °~./\\");
         expect(node.data?.alias).toEqual(null);
         expect(node.data?.hName).toEqual("a");
         expect((node.data?.hProperties as any).className).toEqual(
@@ -409,9 +383,9 @@ describe("remark-wiki-link", () => {
         expect((node.data?.hChildren as any)[0].value).toEqual(
           "li nk-w(i)th-àcèô íã_a(n)d_underline!:ª%@'*º$ °~./\\#li-nk-w(i)th-àcèô íã_a(n)D_UNDERLINE!:ª%@'*º$ °~./\\"
         );
-      });
+      })
     });
-  });
+  })
 
   describe("invalid wiki links", () => {
     test("doesn't parse a wiki link with two missing closing brackets", () => {
@@ -612,3 +586,4 @@ describe("remark-wiki-link", () => {
     });
   });
 });
+