@portabletext/editor 1.18.2 → 1.18.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@portabletext/editor",
-  "version": "1.18.2",
+  "version": "1.18.4",
   "description": "Portable Text Editor made in React",
   "keywords": [
     "sanity",

package/src/editor/define-schema.ts

@@ -1,10 +1,5 @@
 import {Schema as SanitySchema} from '@sanity/schema'
-import {
-  defineField,
-  defineType,
-  type BlockDecoratorDefinition,
-  type ObjectSchemaType,
-} from '@sanity/types'
+import {defineField, defineType, type ObjectSchemaType} from '@sanity/types'
 import startCase from 'lodash.startcase'
 import type {PortableTextMemberSchemaTypes} from '../types/editor'
 import {createEditorSchema} from './create-editor-schema'
@@ -15,7 +10,6 @@ import {createEditorSchema} from './create-editor-schema'
 export type BaseDefinition = {
   name: string
   title?: string
-  icon?: BlockDecoratorDefinition['icon']
 }
 
 /**
@@ -57,7 +51,6 @@ export function compileSchemaDefinition<
         // fields to objects with the name `image`
         name: blockObject.name === 'image' ? 'tmp-image' : blockObject.name,
         title: blockObject.title,
-        icon: blockObject.icon,
         fields: [],
       }),
     ) ?? []
@@ -67,7 +60,6 @@ export function compileSchemaDefinition<
         type: 'object',
         name: inlineObject.name,
         title: inlineObject.title,
-        icon: inlineObject.icon,
         fields: [],
       }),
     ) ?? []
@@ -86,27 +78,23 @@ export function compileSchemaDefinition<
             definition?.decorators?.map((decorator) => ({
               title: decorator.title ?? startCase(decorator.name),
               value: decorator.name,
-              icon: decorator.icon,
             })) ?? [],
           annotations:
             definition?.annotations?.map((annotation) => ({
               name: annotation.name,
               type: 'object',
               title: annotation.title,
-              icon: annotation.icon,
             })) ?? [],
         },
         lists:
           definition?.lists?.map((list) => ({
             value: list.name,
             title: list.title ?? startCase(list.name),
-            icon: list.icon,
           })) ?? [],
         styles:
           definition?.styles?.map((style) => ({
             value: style.name,
             title: style.title ?? startCase(style.name),
-            icon: style.icon,
           })) ?? [],
       },
     ],

package/src/utils/looks-like-url.test.ts

@@ -4,10 +4,16 @@ import {looksLikeUrl} from './looks-like-url'
 test(looksLikeUrl.name, () => {
   expect(looksLikeUrl('https://example.com')).toBe(true)
   expect(looksLikeUrl('http://example.com')).toBe(true)
-  expect(looksLikeUrl('example.com')).toBe(true)
   expect(looksLikeUrl('mailto:foo@example.com')).toBe(true)
   expect(looksLikeUrl('tel:+123456789')).toBe(true)
-  expect(looksLikeUrl('http://example')).toBe(false)
-  expect(looksLikeUrl('http:example')).toBe(false)
+  expect(looksLikeUrl('https://example')).toBe(true)
+  expect(looksLikeUrl('http://example')).toBe(true)
+  expect(looksLikeUrl('http:example')).toBe(true)
+
+  expect(looksLikeUrl('http: example')).toBe(false)
+  expect(looksLikeUrl('https://example. com')).toBe(false)
+  expect(looksLikeUrl('example.com')).toBe(false)
+  expect(looksLikeUrl('example. com')).toBe(false)
   expect(looksLikeUrl('a:b')).toBe(false)
+  expect(looksLikeUrl('a: b')).toBe(false)
 })

package/src/utils/looks-like-url.ts

@@ -1,29 +1,13 @@
 export function looksLikeUrl(text: string) {
   let looksLikeUrl = false
   try {
-    try {
-      const url = new URL(text)
+    const url = new URL(text)
 
-      if (!sensibleProtocols.includes(url.protocol)) {
-        return false
-      }
-
-      const probablyHasTld =
-        url.hostname.length > 0 ? url.hostname.includes('.') : true
-
-      looksLikeUrl = probablyHasTld
-    } catch {
-      const url = new URL(`https://${text}`)
-
-      if (!sensibleProtocols.includes(url.protocol)) {
-        return false
-      }
-
-      const probablyHasTld =
-        url.hostname.length > 0 ? url.hostname.includes('.') : true
-
-      looksLikeUrl = probablyHasTld
+    if (!sensibleProtocols.includes(url.protocol)) {
+      return false
     }
+
+    looksLikeUrl = true
   } catch {}
   return looksLikeUrl
 }