@apify/ui-library 0.59.0 → 0.59.1

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@apify/ui-library",
-    "version": "0.59.0",
+    "version": "0.59.1",
     "description": "React UI library used by apify.com",
     "license": "Apache-2.0",
     "type": "module",
@@ -67,5 +67,5 @@
         "recast": "^0.23.9",
         "typescript": "^5.1.6"
     },
-    "gitHead": "aff5f2f1a3db4fa021ade9588f4d923e59527e8b"
+    "gitHead": "16fdc799cf1670638596509d7af5e0a5421a598c"
 }

package/src/components/link.tsx

@@ -92,8 +92,8 @@ export const Link = forwardRef<HTMLElement, LinkProps>(({
     const effectiveRel = clsx(
         rel,
         isExternal && 'external',
-        (isExternal || target === '_blank') && 'noopener noreferrer',
-        (isExternal && !isTrusted) && 'nofollow',
+        (isExternal || target === '_blank') && 'noopener',
+        (isExternal && !isTrusted) && 'nofollow noreferrer',
     );
 
     return (

package/src/components/to_consolidate/markdown.tsx

@@ -417,21 +417,28 @@ const DefaultLinkRenderer = ({ node, href, ...props }: LinkRendererProps, { host
         && APIFY_HOSTNAMES.includes(urlParsed.hostname.toLowerCase())
         && urlParsed.protocol === 'https:'; // we want to disqualify links that have http: protocol. It's a mistake on users' side that we are penalized for.
 
-    let linkProps = {};
-    // If the link is an Apify link to the same hostname, we want to open in the same tab otherwise we want to open in a new tab
-    // If an external non-Apify link, we want to always open it in a new tab and add rel="noopener noreferrer nofollow"
-    // (and ugc if the link comes from the user-generated content)
-    if (isApifyLink && hasDifferentHostname) {
-        linkProps = { target: '_blank', rel: 'noopener noreferrer' };
-    } else if (!hasDifferentHostname && Link) { // Same host name, use the provided link for internal navigation
+    // Same host name, use the provided link for internal navigation
+    if (!hasDifferentHostname && Link) {
         return <Link to={urlParsed} {...props} />;
-    } else {
-        // USG - User Generated Content (new rel attribute for nofollow links)
-        // Google says:
-        // It’s valid to use nofollow with the new attributes — such as rel=”nofollow ugc”
-        // — if you wish to be backwards-compatible with services that don’t support the new attributes.
-        linkProps = { target: '_blank', rel: clsx('noopener noreferrer nofollow', isUserGeneratedContent && 'ugc') };
     }
+
+    let linkProps = {};
+
+    // If false, we want to open the link in the same tab (linkProps won't have any props)
+    if (hasDifferentHostname) {
+        // If the link is an Apify link to the different hostname, we want to open in a new tab
+        if (isApifyLink) {
+            linkProps = { target: '_blank', rel: 'noopener' };
+        } else {
+            // If an external non-Apify link, we want to always open it in a new tab and add rel="noopener noreferrer nofollow"
+            // USG - User Generated Content (new rel attribute for nofollow links)
+            // Google says:
+            // It’s valid to use nofollow with the new attributes — such as rel=”nofollow ugc”
+            // — if you wish to be backwards-compatible with services that don’t support the new attributes.
+            linkProps = { target: '_blank', rel: clsx('noopener noreferrer', 'nofollow', isUserGeneratedContent && 'ugc') };
+        }
+    }
+
     return <a href={href} {...props} {...linkProps} />;
 };