apostrophe 4.5.0 → 4.5.2

package/CHANGELOG.md

@@ -1,5 +1,20 @@
 # Changelog
 
+## 4.5.2 (2024-07-11)
+
+### Fixes
+
+* Ensure that `apos.doc.walk` never gets caught in an infinite loop even if circular references are present in the data. This is a hotfix for an issue that can arise when the new support for breadcrumbs in search results is combined with a more inclusive projection for page ancestors.
+* Correct a longstanding bug in `apos.doc.walk` that led items to be listed twice in the `ancestors` array passed to the iterator.
+* Correct a longstanding bug in `apos.doc.walk` that led ancestors that are themselves arrays to be misrepresented as a series of objects in the `ancestors` array passed to the iterator.
+* For additional guarantees of reliability the `_dotPath` and `_ancestors` arguments to `apos.doc.walk`, which were always clearly documented as for internal use only, can no longer be passed in externally.
+
+## 4.5.1 (2024-07-11)
+
+### Changes
+
+* Allow tiptap rich-text widget to open modals for images and links without closing the toolbar.
+
 ## 4.5.0 (2024-07-10)
 
 ### Adds
@@ -14,6 +29,7 @@
 
 ### Fixes
 
+* Removes unnecessary, broadly applied line-height setting that may cause logged-in vs logged-out visual discrepencies.
 * Remove double GET request when saving image update.
 * Fix filter menu forgetting selecting filters and not instantiating them.
 * Remove blur emit for filter buttons and search bar to avoid re requesting when clicking outside…

package/modules/@apostrophecms/admin-bar/ui/apos/components/TheAposAdminBar.vue

@@ -89,6 +89,7 @@ export default {
   align-items: center;
   height: 35px;
   padding: 10px 20px;
+  line-height: var(--a-line-base);
   border-bottom: 1px solid var(--a-base-9);
 }
 

package/modules/@apostrophecms/admin-bar/ui/apos/components/TheAposAdminBarMenu.vue

@@ -176,6 +176,7 @@ export default {
 .apos-admin-bar__items {
   display: flex;
   flex-grow: 1;
+  line-height: var(--a-line-base);
   margin: 0;
   padding: 0;
 }

package/modules/@apostrophecms/doc/index.js

@@ -786,39 +786,40 @@ module.exports = {
       // will be `b`, the value will be `5`, the dotPath
       // will be the string `a.b`, and ancestors will be
       // [ { a: { b: 5 } } ].
-      //
-      // You do not need to pass the `_dotPath` and `_ancestors` arguments.
-      // They are used for recursive invocation.
-      walk(doc, iterator, _dotPath, _ancestors) {
-        // We do not use lodash here because of
-        // performance issues.
-        //
-        // Pruning big nested objects is not something we
-        // can afford to do slowly. -Tom
-        if (_dotPath !== undefined) {
-          _dotPath += '.';
-        } else {
-          _dotPath = '';
-        }
-        _ancestors = (_ancestors || []).concat(doc);
-        const remove = [];
-        for (const key in doc) {
-          const __dotPath = _dotPath + key.toString();
-          const ow = '_originalWidgets';
-          if (__dotPath === ow || __dotPath.substring(0, ow.length) === ow + '.') {
-            continue;
+      walk(doc, iterator) {
+        return walkBody(doc, iterator, undefined, []);
+        function walkBody(doc, iterator, _dotPath, _ancestors) {
+          if (_ancestors.includes(doc)) {
+            // No infinite loops on circular references
+            return;
           }
-          if (iterator(doc, key, doc[key], __dotPath, _ancestors) === false) {
-            remove.push(key);
+          // Don't use concat, doc can be an array in which case
+          // it is important to preserve the nesting
+          _ancestors = [ ..._ancestors, doc ];
+          if (_dotPath !== undefined) {
+            _dotPath += '.';
           } else {
-            const val = doc[key];
-            if (typeof val === 'object') {
-              self.walk(val, iterator, __dotPath, _ancestors.concat([ doc ]));
+            _dotPath = '';
+          }
+          const remove = [];
+          for (const key in doc) {
+            const __dotPath = _dotPath + key.toString();
+            const ow = '_originalWidgets';
+            if (__dotPath === ow || __dotPath.substring(0, ow.length) === ow + '.') {
+              continue;
+            }
+            if (iterator(doc, key, doc[key], __dotPath, _ancestors) === false) {
+              remove.push(key);
+            } else {
+              const val = doc[key];
+              if (typeof val === 'object') {
+                walkBody(val, iterator, __dotPath, _ancestors);
+              }
             }
           }
-        }
-        for (const key of remove) {
-          delete doc[key];
+          for (const key of remove) {
+            delete doc[key];
+          }
         }
       },
       // Retry the given "actor" async function until it

package/modules/@apostrophecms/image/ui/apos/components/AposMediaManagerEditor.vue

@@ -461,6 +461,7 @@ export default {
   .apos-media-editor__lip {
     display: flex;
     justify-content: flex-end;
+    line-height: var(--a-line-base);
 
     & > .apos-context-menu, & > .apos-button__wrapper {
       margin-left: 7.5px;

package/modules/@apostrophecms/modal/ui/apos/components/AposModal.vue

@@ -422,6 +422,7 @@ function close() {
     display: flex;
     align-items: center;
     padding: $spacing-double;
+    line-height: var(--a-line-base);
   }
 
   .apos-modal__header__main {

package/modules/@apostrophecms/modal/ui/apos/components/AposModalToolbar.vue

@@ -33,6 +33,7 @@ export default {
   .apos-toolbar {
     display: flex;
     justify-content: space-between;
+    line-height: var(--a-line-base);
   }
 
   .apos-toolbar__group {

package/modules/@apostrophecms/rich-text-widget/ui/apos/components/AposRichTextWidgetEditor.vue

@@ -13,7 +13,7 @@
         animation: 'fade',
         inertia: true,
         placement: 'bottom',
-        hideOnClick: true
+        hideOnClick: false
       }"
       :editor="editor"
     >

package/modules/@apostrophecms/schema/ui/apos/components/AposSchema.vue

@@ -89,6 +89,10 @@ export default {
 </script>
 
 <style lang="scss" scoped>
+  .apos-schema {
+    line-height: var(--a-line-base);
+  }
+
   .apos-schema :deep(.apos-field__wrapper) {
     max-width: $input-max-width;
   }

package/modules/@apostrophecms/ui/ui/apos/components/AposContextMenu.vue

@@ -243,6 +243,7 @@ async function setDropdownPosition() {
 .apos-context-menu__dropdown-content {
   z-index: $z-index-notifications;
   position: absolute;
+  line-height: var(--a-line-base);
   width: max-content;
 
   &[aria-hidden='true'] {

package/modules/@apostrophecms/ui/ui/apos/scss/global/_normalize.scss

@@ -1,4 +1,3 @@
 [class*='apos-'] {
-  line-height: var(--a-line-base);
   box-sizing: content-box;
 }

package/modules/@apostrophecms/ui/ui/apos/scss/global/_tables.scss

@@ -1,6 +1,7 @@
 .apos-table {
   width: 100%;
   border-collapse: collapse;
+  line-height: var(--a-line-base);
 }
 
 .apos-table__header {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "apostrophe",
-  "version": "4.5.0",
+  "version": "4.5.2",
   "description": "The Apostrophe Content Management System.",
   "main": "index.js",
   "scripts": {
@@ -148,4 +148,4 @@
   "browserslist": [
     "ie >= 10"
   ]
-}
+}