apostrophe 4.30.0-alpha.1 → 4.30.0

package/modules/@apostrophecms/image-widget/index.js

@@ -102,7 +102,7 @@ module.exports = {
         linkHref: {
           label: 'apostrophe:url',
           help: 'apostrophe:linkHrefHelp',
-          type: 'string',
+          type: 'url',
           required: true,
           if: {
             linkTo: '_url'
@@ -147,6 +147,7 @@ module.exports = {
     self.showPlaceholder = self.options.placeholder !== false;
     self.options.placeholder = true;
     self.determineBestAssetUrl('placeholder');
+    self.addCleanLinkHrefMigration();
   },
   handlers(self) {
     return {
@@ -159,6 +160,33 @@ module.exports = {
   },
   methods(self) {
     return {
+      // Clear link hrefs that use dangerous URL schemes (e.g.
+      // `javascript:`) and may have been stored before the linkHref
+      // schema field was changed to `type: 'url'`. Registered per
+      // module instance so subclasses of this widget are migrated
+      // under their own widget type and migration name.
+      addCleanLinkHrefMigration() {
+        self.apos.migration.add(
+          `${self.__meta.name}:clean-naughty-link-href`,
+          async () => {
+            await self.apos.migration.eachWidget({}, async (doc, widget, dotPath) => {
+              if (widget.type !== self.name) {
+                return;
+              }
+              if (!widget.linkHref) {
+                return;
+              }
+              if (!self.apos.launder.naughtyHref(widget.linkHref)) {
+                return;
+              }
+              await self.apos.doc.db.updateOne(
+                { _id: doc._id },
+                { $set: { [`${dotPath}.linkHref`]: '' } }
+              );
+            });
+          }
+        );
+      },
       validateAndAddSchemaLabels() {
         const linkWithType = self.options.linkWithType;
 

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

@@ -244,9 +244,7 @@ module.exports = {
               },
               setTotal (n) {
                 total = n;
-                const result = self.setTotal(job, n);
-                promises.push(result);
-                return result;
+                return self.setTotal(job, n);
               },
               setResults (_results) {
                 results = _results;
@@ -414,12 +412,12 @@ module.exports = {
       //
       // No promise is returned as this method just updates
       // the job tracking information in the background.
-      async setTotal(job, total) {
-        try {
-          await self.db.updateOne({ _id: job._id }, { $set: { total } });
-        } catch (err) {
-          self.apos.util.error(err);
-        }
+      setTotal(job, total) {
+        self.db.updateOne({ _id: job._id }, { $set: { total } }, function (err) {
+          if (err) {
+            self.apos.util.error(err);
+          }
+        });
       },
       // Mark the given job as ended. If `success`
       // is true the job is reported as an overall

package/modules/@apostrophecms/layout-widget/index.js

@@ -19,6 +19,10 @@ module.exports = {
     gap: '1.5rem',
     defaultCellHorizontalAlignment: null,
     defaultCellVerticalAlignment: null,
+    // Extra class name(s) to append to the rendered layout-widget area
+    // wrapper, in addition to the built-in `layout-widget` class. Accepts
+    // a string of space-separated class names.
+    className: '',
     injectStyles: true,
     minifyStyles: true
   },
@@ -98,6 +102,25 @@ module.exports = {
         validateAndIdentifyTypes() {
           const { column } = self.validateAndIdentifyTypes();
           self.columnWidgetName = column;
+        },
+        // Detect the widget-style "gap" field (any styles field whose
+        // CSS `property` resolves to `gap`). Only the first match is used.
+        // Also detect whether the @apostrophecms/styles module has a
+        // site-wide layout gap field configured (via the `layoutGap`
+        // preset / `layoutGapDefault: true` marker).
+        detectGapFields() {
+          const widgetGapFields = self.apos.styles
+            .fieldsWithProperty(self.schema, 'gap');
+          if (widgetGapFields.length > 1) {
+            self.apos.util.warn(
+              `[${self.__meta.name}] Multiple style fields produce the ` +
+              `CSS \`gap\` property (${widgetGapFields.join(', ')}). ` +
+              'Only the first one will be honoured as the widget-scope gap.'
+            );
+          }
+          self.widgetGapFieldName = widgetGapFields[0] || null;
+          self.globalGapEnabled = !!self.apos.modules['@apostrophecms/styles']
+            ?.layoutGapFieldName;
         }
       }
     };
@@ -118,6 +141,17 @@ module.exports = {
             defaultCellHorizontalAlignment: self.options.defaultCellHorizontalAlignment,
             defaultCellVerticalAlignment: self.options.defaultCellVerticalAlignment
           },
+          widgetGapFieldName: self.widgetGapFieldName || null,
+          widgetGapFieldUnit: self.widgetGapFieldName
+            ? (self.apos.styles
+              .getFieldByPath(self.schema, self.widgetGapFieldName)
+              ?.unit || '')
+            : '',
+          globalGapEnabled: !!self.globalGapEnabled,
+          // Opt-in flag read by the generic widget editor: when true,
+          // it broadcasts `apos-widget-live-preview` events on the apos bus
+          // during the style-only fast path.
+          subscribesToLivePreview: !!self.widgetGapFieldName,
           columnWidgetName: self.columnWidgetName
         };
       },
@@ -132,6 +166,7 @@ module.exports = {
           defaultCellHorizontalAlignment,
           defaultCellVerticalAlignment
         } = self.options;
+        const widgetGap = self.resolveWidgetGap(widget);
         return {
           ..._super(widget, { scene }),
           columns,
@@ -141,13 +176,53 @@ module.exports = {
           tablet,
           gap,
           defaultCellHorizontalAlignment,
-          defaultCellVerticalAlignment
+          defaultCellVerticalAlignment,
+          _gap: widgetGap,
+          _gapHasGlobal: !!self.globalGapEnabled
         };
       }
     };
   },
   methods(self) {
     return {
+      // Resolve the widget-scope gap for a widget instance, if this
+      // module declares a styles field with `property: 'gap'`. Returns
+      // the resolved value (with unit, when applicable). When the
+      // widget has no explicit value, falls back to the field's `def`.
+      // Returns `null` only when no widget gap field is
+      // configured, no widget value is set, and no `def` is declared
+      // on the field.
+      resolveWidgetGap(widget) {
+        if (!self.widgetGapFieldName || !widget) {
+          return null;
+        }
+        const field = self.apos.styles.getFieldByPath(
+          self.schema, self.widgetGapFieldName
+        );
+        let value = self.apos.util.get(widget, self.widgetGapFieldName);
+        if (value === null || value === undefined || value === '') {
+          if (field?.def === null || field?.def === undefined || field?.def === '') {
+            return null;
+          }
+          value = field.def;
+        }
+        if (field?.unit && typeof value !== 'string') {
+          return `${value}${field.unit}`;
+        }
+        return value;
+      },
+      // Determine whether the inline `--grid-gap` CSS variable should
+      // be omitted on the grid container so the global cascade
+      // (`var(--apos-layout-gap, …)`) can take effect. The inline var
+      // is omitted whenever:
+      //   - no widget-scope gap value is set, AND
+      //   - the global layout-gap field is configured.
+      shouldOmitInlineGap(widget, global) {
+        if (!self.globalGapEnabled) {
+          return false;
+        }
+        return self.resolveWidgetGap(widget) === null;
+      },
       publicCssNodes(req) {
         return [
           {
@@ -277,11 +352,13 @@ module.exports = {
         const mobileBreakpointPlus = mobileBreakpoint + 1;
         const tabletBreakpoint = self.options.tablet?.breakpoint || 1024;
         const tabletBreakpointPlus = tabletBreakpoint + 1;
+        const gapDefault = self.options.gap || '0';
         cssContent = cssContent
           .replace(/\{\$mobile\}/g, mobileBreakpoint)
           .replace(/\{\$mobile-plus\}/g, mobileBreakpointPlus)
           .replace(/\{\$tablet\}/g, tabletBreakpoint)
-          .replace(/\{\$tablet-plus\}/g, tabletBreakpointPlus);
+          .replace(/\{\$tablet-plus\}/g, tabletBreakpointPlus)
+          .replace(/\{\$gap-default\}/g, gapDefault);
 
         return self.processCss(cssContent, scene);
       },
@@ -371,6 +448,51 @@ module.exports = {
           (a.order ?? 0) - (b.order ?? 0)
         );
         return items[items.length - 1]._id;
+      },
+      // Compute the `--grid-gap: <value>;` declaration to inline on the
+      // grid container, or an empty string when the cascade should
+      // resolve gap via `var(--apos-layout-gap, …)` instead.
+      // Honours the priority order:
+      //   1. Widget-style gap (when set on this widget instance).
+      //   2. Static module option (BC) — when no global gap field is
+      //      configured, or it has no value.
+      //   3. Otherwise, omit the inline var so the global cascade wins.
+      // Must be invoked via the widget's own module namespace —
+      // `apos.modules[data.manager.__meta.name].gapInlineCss(...)` —
+      // so that `self` resolves to the actual subclass and picks up
+      // its `widgetGapFieldName` / `globalGapEnabled`.
+      gapInlineCss(widget, options, global) {
+        const widgetGap = self.resolveWidgetGap(widget);
+        if (widgetGap !== null) {
+          return ` --grid-gap: ${widgetGap};`;
+        }
+        if (self.shouldOmitInlineGap(widget, global)) {
+          return '';
+        }
+        const fallback = (options && options.gap) || self.options.gap || '0';
+        return ` --grid-gap: ${fallback};`;
+      },
+      // Build the `aposParentOptions` payload passed by the rendered
+      // layout-widget area to the in-place editor (AposAreaLayoutEditor).
+      // Includes the widget's resolved gap (from its `gap` styles field,
+      // when present) so the live editor's grid container reflects the
+      // saved per-widget value rather than only the static module
+      // option. Sets `gap: null` to signal the editor to omit
+      // `--grid-gap` so the global cascade resolves it through
+      // `:root { --apos-layout-gap }`. Must be invoked via the widget's
+      // own module namespace, like `gapInlineCss`.
+      parentOptionsForArea(widget, options, global) {
+        const opts = {
+          ...(options || {}),
+          widgetId: widget._id
+        };
+        const widgetGap = self.resolveWidgetGap(widget);
+        if (widgetGap !== null) {
+          opts.gap = widgetGap;
+        } else if (self.shouldOmitInlineGap(widget, global)) {
+          opts.gap = null;
+        }
+        return opts;
       }
     };
   }

package/modules/@apostrophecms/layout-widget/ui/apos/components/AposAreaLayoutEditor.vue

@@ -122,6 +122,7 @@
 
 <script>
 import { mapActions } from 'pinia';
+import get from 'lodash/get';
 import AposAreaEditorLogic from 'Modules/@apostrophecms/area/logic/AposAreaEditor.js';
 import walkWidgets from 'Modules/@apostrophecms/area/lib/walk-widgets.js';
 import { useWidgetStore } from 'Modules/@apostrophecms/ui/stores/widget.js';
@@ -144,7 +145,11 @@ export default {
     return {
       // 'layout' | 'focus' | 'content'
       layoutMode: 'content',
-      layoutDeviceMode: 'desktop'
+      layoutDeviceMode: 'desktop',
+      // Live snapshot of the parent layout-widget's in-modal style
+      // values, fed by `apos-widget-live-preview` bus events from
+      // AposWidgetEditor while the user drags style sliders.
+      liveWidgetData: null
     };
   },
   computed: {
@@ -152,11 +157,48 @@ export default {
       return window.apos.modules[this.moduleName] || {};
     },
     gridModuleOptions() {
-      return Object.assign(
-        {},
-        this.layoutModuleOptions.grid ?? {},
-        this.parentOptions
-      );
+      const baseGrid = this.layoutModuleOptions.grid ?? {};
+      const parent = this.parentOptions || {};
+      const opts = Object.assign({}, baseGrid, parent);
+      // Resolve `gap` priority for the live editor:
+      const liveGap = this.liveGapValue;
+      if (liveGap != null) {
+        opts.gap = liveGap;
+      } else if (parent.gap != null) {
+        opts.gap = parent.gap;
+      } else if (
+        parent.gap === null ||
+        this.layoutModuleOptions.globalGapEnabled
+      ) {
+        // Parent signalled "use the cascade" (omit `--grid-gap` and let
+        // `var(--apos-layout-gap, …)` resolve it on `:root`).
+        // The snapshot is recomputed whenever `layoutMode`
+        // changes, which is the only cadence the editor needs.
+        if (this.layoutMode !== 'content') {
+          opts.gap = this.resolvedCascadeGap();
+        } else {
+          opts.gap = undefined;
+        }
+      } else {
+        opts.gap = baseGrid.gap;
+      }
+      return opts;
+    },
+    // Reactive bridge to the parent layout-widget's styles modal.
+    liveGapValue() {
+      if (this.layoutMode !== 'content') {
+        return null;
+      }
+      const fieldName = this.layoutModuleOptions.widgetGapFieldName;
+      if (!fieldName || !this.liveWidgetData) {
+        return null;
+      }
+      const value = get(this.liveWidgetData, fieldName);
+      if (value === null || value === undefined || value === '') {
+        return null;
+      }
+      const unit = this.layoutModuleOptions.widgetGapFieldUnit || '';
+      return typeof value === 'string' ? value : `${value}${unit}`;
     },
     // Meta storage is not yet implemented, return a default meta object
     layoutMeta() {
@@ -259,6 +301,12 @@ export default {
         this.emphasizeGrid();
       } else {
         this.deEmphasizeGrid();
+        // Leaving layout mode: clear the control-suppression flag that was
+        // set on the focused widget when we entered layout mode. Without
+        // this the layout widget keeps `isSuppressingWidgetControls=true`
+        // and its side operation bar stays hidden even though it is the
+        // focused widget.
+        apos.bus.$emit('clear-focused-widget-control-suppression');
       }
     }
   },
@@ -266,6 +314,8 @@ export default {
     apos.bus.$on('apos-switch-layout-mode', this.switchLayoutMode);
     apos.bus.$on('apos-layout-col-delete', this.onRemoveLayoutColumn);
     apos.bus.$on('apos-edit-styles', this.editStyles);
+    apos.bus.$on('apos-widget-live-preview', this.onWidgetLivePreview);
+    apos.bus.$on('apos-widget-live-preview-end', this.onWidgetLivePreviewEnd);
     if (!this.hasLayoutColumnWidgets) {
       this.onCreateProvision();
     }
@@ -275,6 +325,8 @@ export default {
     apos.bus.$off('apos-switch-layout-mode', this.switchLayoutMode);
     apos.bus.$off('apos-layout-col-delete', this.onRemoveLayoutColumn);
     apos.bus.$off('apos-edit-styles', this.editStyles);
+    apos.bus.$off('apos-widget-live-preview', this.onWidgetLivePreview);
+    apos.bus.$off('apos-widget-live-preview-end', this.onWidgetLivePreviewEnd);
   },
   methods: {
     ...mapActions(useWidgetStore, [
@@ -284,6 +336,37 @@ export default {
       'removeEmphasizedWidget',
       'setFocusedWidget'
     ]),
+    // Read the current resolved value of the global cascade variable
+    // `--apos-layout-gap` from `:root`. Used by `gridModuleOptions` to
+    // hand layout/focus modes a real gap value.
+    resolvedCascadeGap() {
+      if (!document?.documentElement) {
+        return undefined;
+      }
+      const value = window.getComputedStyle(document.documentElement)
+        .getPropertyValue('--apos-layout-gap')
+        .trim();
+      return value || undefined;
+    },
+    onWidgetLivePreview({ widgetId, data }) {
+      if (!widgetId || widgetId !== this.parentOptions?.widgetId) {
+        return;
+      }
+      this.liveWidgetData = data;
+    },
+    onWidgetLivePreviewEnd({ widgetId, reason }) {
+      if (!widgetId || widgetId !== this.parentOptions?.widgetId) {
+        return;
+      }
+      // On `save` keep the live snapshot in place: clearing
+      // here would expose the old `parent.gap` value for one frame.
+      // On `cancel` / `unmount` drop the live value immediately so the
+      // editor falls back to the SSR-rendered parent options.
+      if (reason === 'save') {
+        return;
+      }
+      this.liveWidgetData = null;
+    },
     clickOnGrid() {
       if (this.parentOptions.widgetId) {
         this.setFocusedWidget(this.parentOptions.widgetId, this.areaId);

package/modules/@apostrophecms/layout-widget/ui/apos/components/AposGridLayout.vue

@@ -16,7 +16,7 @@
       data-mobile-auto="true"
       :style="{
         '--grid-columns': gridState.columns,
-        '--grid-gap': gridState.options.gap || '0',
+        '--grid-gap': gridState.options.gap ?? null,
         '--grid-rows': 'auto',
         '--mobile-grid-rows': 'auto',
         '--tablet-grid-rows': 'auto',
@@ -310,7 +310,7 @@ export default {
   display: grid;
   grid-template-columns: repeat(var(--grid-columns, 12), 1fr);
   grid-template-rows: repeat(var(--grid-rows), auto);
-  grid-gap: var(--grid-gap, 0);
+  gap: var(--grid-gap, var(--apos-layout-gap, 0));
   justify-items: var(--justify-items);
   /* stylelint-disable-next-line declaration-block-no-redundant-longhand-properties */
   align-items: var(--align-items);

package/modules/@apostrophecms/layout-widget/ui/apos/components/AposGridManager.vue

@@ -5,7 +5,7 @@
     data-apos-test="aposLayoutContainerClone"
     :style="{
       '--grid-columns': gridState.columns,
-      '--grid-gap': gridState.options.gap,
+      '--grid-gap': gridState.options.gap ?? null,
     }"
     @mousemove="onMouseMove($event)"
   >
@@ -938,7 +938,7 @@ $resize-ui-z-index: 2;
     inset: 0;
     grid-template-columns: repeat(var(--grid-columns, 12), 1fr);
     grid-template-rows: repeat(var(--grid-rows), auto);
-    grid-gap: var(--grid-gap, 0);
+    gap: var(--grid-gap, var(--apos-layout-gap, 0));
 
     &.is-moving,
     &.is-resizing {

package/modules/@apostrophecms/layout-widget/ui/apos/layout.css

@@ -1,3 +1,11 @@
+/*
+  Reserve the same vertical footprint for the SSR markup. 
+  This stylesheet is only injected on the apos scene in edit mode.
+*/
+.layout-widget > div {
+  min-height: 150px;
+}
+
 /* Tablet rules for admin interface */
 /* stylelint-disable-next-line media-feature-name-allowed-list */
 @media screen and (min-width: {$mobile-plus}px) and (max-width: {$tablet}px) {

package/modules/@apostrophecms/layout-widget/ui/src/layout.css

@@ -2,7 +2,7 @@
   display: grid;
   grid-template-columns: repeat(var(--grid-columns, 12), 1fr);
   grid-template-rows: repeat(var(--grid-rows), auto);
-  grid-gap: var(--grid-gap, 0);
+  gap: var(--grid-gap, var(--apos-layout-gap, {$gap-default}));
   justify-items: var(--justify-items);
   /* stylelint-disable-next-line declaration-block-no-redundant-longhand-properties */
   align-items: var(--align-items);

package/modules/@apostrophecms/layout-widget/views/widget.html

@@ -1,13 +1,13 @@
 {% area data.widget, 'columns' with { 
   aposStyle: '--grid-columns: ' + (data.options.columns or data.manager.options.columns) + ';' + 
-    ' --grid-gap: ' + (data.options.gap or data.manager.options.gap or '0') + ';' + 
+    apos.modules[data.manager.__meta.name].gapInlineCss(data.widget, data.options, data.global) + 
     ' --grid-rows: auto;' + 
     ' --mobile-grid-rows: auto;' +
     ' --tablet-grid-rows: auto;' + 
     ' --justify-items: ' + (data.options.defaultCellHorizontalAlignment or data.manager.options.defaultCellHorizontalAlignment or 'stretch') + ';' +
     ' --align-items: ' + (data.options.defaultCellVerticalAlignment or data.manager.options.defaultCellVerticalAlignment or 'stretch') + ';', 
-  aposClassName: 'layout-widget',
-  aposParentOptions: data.options | merge({ widgetId: data.widget._id }),
+  aposClassName: ('layout-widget' + ((' ' + (data.options.className or data.manager.options.className)) if (data.options.className or data.manager.options.className))),
+  aposParentOptions: apos.modules[data.manager.__meta.name].parentOptionsForArea(data.widget, data.options, data.global),
   aposAttrs: {
     'tablet-auto': true,
     'mobile-auto': true

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

@@ -295,7 +295,17 @@ module.exports = {
           async resetRequest(req) {
             const MIN_RESPONSE_TIME = 2000;
             const startTime = Date.now();
-            const site = (req.headers.host || '').replace(/:\d+$/, '');
+            // Refuse to construct reset URLs from the request's Host header,
+            // which is attacker-controlled. Operators must configure the
+            // baseUrl option (or the APOS_BASE_URL environment variable) so
+            // the link in the reset email points to the real site rather than
+            // wherever the attacker aimed their crafted request.
+            if (!self.apos.baseUrl) {
+              throw self.apos.error(
+                'invalid',
+                'The baseUrl option (or APOS_BASE_URL environment variable) must be configured to enable password reset'
+              );
+            }
             const email = self.apos.launder.string(req.body.email);
             if (!email.length) {
               throw self.apos.error('invalid', req.t('apostrophe:loginResetEmailRequired'));
@@ -319,22 +329,10 @@ module.exports = {
               user.passwordReset = reset;
               user.passwordResetAt = new Date();
               await self.apos.user.update(req, user, { permissions: false });
-              let port = (req.headers.host || '').split(':')[1];
-              if (!port || [ '80', '443' ].includes(port)) {
-                port = '';
-              } else {
-                port = `:${port}`;
-              }
-              const parsed = new URL(
-                req.absoluteUrl,
-                self.apos.baseUrl
-                  ? undefined
-                  : `${req.protocol}://${req.hostname}${port}`
-              );
-              parsed.pathname = self.login();
-              parsed.search = '?';
+              const parsed = new URL(self.login(), self.apos.baseUrl);
               parsed.searchParams.append('reset', reset);
               parsed.searchParams.append('email', user.email);
+              const site = parsed.hostname;
               try {
                 await self.email(req, 'passwordResetEmail', {
                   user,

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

@@ -212,7 +212,8 @@ const nonDraggableElements = [
   '.apos-field--inline-array-field',
   '.apos-field--inline-array-table-with-remove-button-field',
   '.apos-field--inline-array-table-field',
-  '.apos-input-color__sample-picker'
+  '.apos-input-color__sample-picker',
+  '.apos-input-array-inline-table'
 ];
 
 const resizeSides = [

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

@@ -16,10 +16,13 @@ const cheerio = require('cheerio');
 // widely trusted sites are already allowlisted.
 //
 // Your `allowlist` option is concatenated with `oembetter`'s standard
-// allowlist, plus wufoo.com, infogr.am, and slideshare.net.
+// allowlist, plus wufoo.com, infogr.am and slideshare.net.
 //
 // Your `endpoints` option is concatenated with `oembetter`'s standard
 // endpoints list.
+//
+// If you wish to completely override the behavior, set
+// `minimumAllowlist` and `minimumEndpoints` instead.
 
 module.exports = {
   options: {
@@ -42,28 +45,30 @@ module.exports = {
         // Don't permit oembed of untrusted sites, which could
         // lead to XSS attacks
 
-        self.oembetter.allowlist(self.oembetter.suggestedAllowlist.concat(
-          self.options.allowlist || [],
-          [
-            'wufoo.com',
-            'infogr.am',
-            'slideshare.net'
-          ])
-        );
+        const minimumAllowlist = self.options.minimumAllowlist || [
+          ...self.oembetter.suggestedAllowlist,
+          'wufoo.com',
+          'infogr.am',
+          'slideshare.net'
+        ];
+
+        self.oembetter.allowlist(minimumAllowlist.concat(self.options.allowlist || []));
+
+        const minimumEndpoints = self.options.minimumEndpoints || self.oembetter.suggestedEndpoints;
         self.oembetter.endpoints(
-          self.oembetter.suggestedEndpoints.concat(self.options.endpoints || [])
+          minimumEndpoints.concat(self.options.endpoints || [])
         );
       },
 
       // Enhances oembetter to support services better or to support services
-      // that have no oembed support by default. Called by `afterConstruct`.
-      // Extend this method to add additional `oembetter` filters.
+      // that have no oembed support by default.
+      //
+      // Extend or override this method to change or add oembetter filters.
 
       enhanceOembetter() {
         require('./lib/youtube.js')(self, self.oembetter);
         require('./lib/vimeo.js')(self, self.oembetter);
         require('./lib/wufoo.js')(self, self.oembetter);
-        require('./lib/infogram.js')(self, self.oembetter);
       },
 
       // This method fetches the specified URL, determines its best embedded

package/modules/@apostrophecms/recently-edited/ui/apos/components/AposRecentlyEditedIcon.vue

@@ -6,6 +6,8 @@
     :modifiers="['small', 'no-motion']"
     :tooltip="$t('apostrophe:recentlyEditedDocuments')"
     :icon-only="true"
+    :label="'apostrophe:recentlyEditedManagerOpen'"
+    :attrs="{ 'aria-label': $t('apostrophe:recentlyEditedManagerOpen') }"
     @click="open"
   />
 </template>