@eeacms/volto-group-block 8.1.0 → 9.0.0

package/CHANGELOG.md

@@ -4,15 +4,23 @@ All notable changes to this project will be documented in this file. Dates are d
 
 Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 
-### [8.1.0](https://github.com/eea/volto-group-block/compare/8.0.0...8.1.0) - 3 December 2025
+### [9.0.0](https://github.com/eea/volto-group-block/compare/8.1.1...9.0.0) - 10 February 2026
 
 #### :rocket: New Features
 
-- feat: Add possibility to skip blocks from character counting - refs #293783 [Teodor Voicu - [`c35d76b`](https://github.com/eea/volto-group-block/commit/c35d76bd03476bbcf8324b247ecf909c093d4117)]
+- feat: Persist charCount to block data and add maxCharsOverflowPercent - refs #294806 [Alin Voinea - [`b5bbd75`](https://github.com/eea/volto-group-block/commit/b5bbd7516cb8cb6bcff8541d711fd9fa74a4d92c)]
+
+### [8.1.1](https://github.com/eea/volto-group-block/compare/8.1.0...8.1.1) - 26 January 2026
 
 #### :hammer_and_wrench: Others
 
-- Release 8.1.0 [Alin Voinea - [`d711911`](https://github.com/eea/volto-group-block/commit/d71191102f8ef97375905049c7cbbe71ea9e357c)]
+- Fix blocks chooser layout when inside section block [Teodor - [`978ba35`](https://github.com/eea/volto-group-block/commit/978ba35214fc0b22bca04f22f051fdcd25d9812a)]
+### [8.1.0](https://github.com/eea/volto-group-block/compare/8.0.0...8.1.0) - 3 December 2025
+
+#### :rocket: New Features
+
+- feat: Add possibility to skip blocks from character counting - refs #293783 [Teodor Voicu - [`c35d76b`](https://github.com/eea/volto-group-block/commit/c35d76bd03476bbcf8324b247ecf909c093d4117)]
+
 ## [8.0.0](https://github.com/eea/volto-group-block/compare/7.1.1...8.0.0) - 3 December 2025
 
 #### :boom: Breaking Change

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@eeacms/volto-group-block",
-  "version": "8.1.0",
+  "version": "9.0.0",
   "description": "volto-group-block: Volto block to be used to group other blocks",
   "main": "src/index.js",
   "author": "European Environment Agency: IDM2 A-Team",

package/src/components/manage/Blocks/Group/CounterComponent.jsx

@@ -56,7 +56,7 @@ const countTextInEachBlock =
       : countCharsWithSpaces(foundText);
   };
 
-const countTextInBlocks = (blocksObject, ignoreSpaces, maxChars) => {
+export const countTextInBlocks = (blocksObject, ignoreSpaces, maxChars) => {
   const { countTextIn, skipBlocksInGroups = [] } =
     config.blocks?.blocksConfig?.group || {};
   // use obj ref to update value - if you send number it will not be updated
@@ -82,12 +82,21 @@ const countTextInBlocks = (blocksObject, ignoreSpaces, maxChars) => {
 };
 
 const CounterComponent = ({ data, setSidebarTab, setSelectedBlock }) => {
-  const { maxChars, ignoreSpaces } = data;
-  const charCount = countTextInBlocks(data?.data, ignoreSpaces, maxChars);
+  const maxChars = parseInt(data.maxChars) || 0;
+  const maxCharsOverflowPercent = parseInt(data.maxCharsOverflowPercent) || 0;
+  const { ignoreSpaces } = data;
+  const charCount =
+    data.charCount ?? countTextInBlocks(data?.data, ignoreSpaces, maxChars);
+
+  const overflowLimit =
+    maxCharsOverflowPercent > 0
+      ? Math.ceil((maxChars * (100 + maxCharsOverflowPercent)) / 100)
+      : maxChars;
+
   const counterClass =
     charCount < Math.ceil(maxChars / 1.05)
       ? 'info'
-      : charCount < maxChars
+      : charCount <= overflowLimit
       ? 'warning'
       : 'danger';
 

package/src/components/manage/Blocks/Group/CounterComponent.test.jsx

@@ -231,4 +231,119 @@ describe('CounterComponent', () => {
     expect(setSidebarTab).toHaveBeenCalledWith(1);
     expect(setSelectedBlock).toHaveBeenCalled();
   });
+
+  it('should render warning class when chars are within overflow zone (maxCharsOverflowPercent=10)', () => {
+    const { container, getByText } = render(
+      <CounterComponent
+        data={{
+          maxChars: 100,
+          maxCharsOverflowPercent: 10,
+          data: {
+            blocks: {
+              block1: { '@type': 'text', plaintext: 'a'.repeat(105) },
+            },
+            blocks_layout: {
+              items: ['block1'],
+            },
+          },
+        }}
+        setSidebarTab={setSidebarTab}
+        setSelectedBlock={setSelectedBlock}
+      />,
+    );
+    expect(getByText('5 characters over the limit')).toBeInTheDocument();
+    expect(container.querySelector('.counter.warning')).toBeInTheDocument();
+  });
+
+  it('should render danger class when chars exceed overflow zone (maxCharsOverflowPercent=10)', () => {
+    const { container, getByText } = render(
+      <CounterComponent
+        data={{
+          maxChars: 100,
+          maxCharsOverflowPercent: 10,
+          data: {
+            blocks: {
+              block1: { '@type': 'text', plaintext: 'a'.repeat(111) },
+            },
+            blocks_layout: {
+              items: ['block1'],
+            },
+          },
+        }}
+        setSidebarTab={setSidebarTab}
+        setSelectedBlock={setSelectedBlock}
+      />,
+    );
+    expect(getByText('11 characters over the limit')).toBeInTheDocument();
+    expect(container.querySelector('.counter.danger')).toBeInTheDocument();
+  });
+
+  it('should render warning class when chars equal maxChars with no overflow percent', () => {
+    const { container, getByText } = render(
+      <CounterComponent
+        data={{
+          maxChars: 100,
+          data: {
+            blocks: {
+              block1: { '@type': 'text', plaintext: 'a'.repeat(100) },
+            },
+            blocks_layout: {
+              items: ['block1'],
+            },
+          },
+        }}
+        setSidebarTab={setSidebarTab}
+        setSelectedBlock={setSelectedBlock}
+      />,
+    );
+    expect(getByText('0 characters remaining out of 100')).toBeInTheDocument();
+    expect(container.querySelector('.counter.warning')).toBeInTheDocument();
+  });
+
+  it('should render danger class when negative overflow percent falls back to maxChars', () => {
+    const { container, getByText } = render(
+      <CounterComponent
+        data={{
+          maxChars: 100,
+          maxCharsOverflowPercent: -5,
+          data: {
+            blocks: {
+              block1: { '@type': 'text', plaintext: 'a'.repeat(104) },
+            },
+            blocks_layout: {
+              items: ['block1'],
+            },
+          },
+        }}
+        setSidebarTab={setSidebarTab}
+        setSelectedBlock={setSelectedBlock}
+      />,
+    );
+    expect(getByText('4 characters over the limit')).toBeInTheDocument();
+    expect(container.querySelector('.counter.danger')).toBeInTheDocument();
+  });
+
+  it('should use persisted charCount from data instead of recomputing', () => {
+    const { container, getByText } = render(
+      <CounterComponent
+        data={{
+          maxChars: 100,
+          charCount: 42,
+          data: {
+            blocks: {
+              block1: { '@type': 'text', plaintext: 'a'.repeat(90) },
+            },
+            blocks_layout: {
+              items: ['block1'],
+            },
+          },
+        }}
+        setSidebarTab={setSidebarTab}
+        setSelectedBlock={setSelectedBlock}
+      />,
+    );
+    // Should use charCount=42 from data, not 90 from blocks
+    expect(getByText('58 characters remaining out of 100')).toBeInTheDocument();
+    expect(container.querySelector('.counter.info')).toBeInTheDocument();
+  });
 });

package/src/components/manage/Blocks/Group/DefaultBody.jsx

@@ -1,6 +1,7 @@
 import { Button } from 'semantic-ui-react';
 import { BlocksForm, Icon, RenderBlocks } from '@plone/volto/components';
 import EditBlockWrapper from './EditBlockWrapper';
+import { countTextInBlocks } from './CounterComponent';
 import { useLocation } from 'react-router-dom';
 
 import helpSVG from '@plone/volto/icons/help.svg';
@@ -48,21 +49,38 @@ const GroupBlockDefaultBody = (props) => {
         onSelectBlock(id, isMultipleSelection, e, selectedBlock);
       }}
       onChangeFormData={(newFormData) => {
-        onChangeBlock(block, {
+        const newData = {
           ...data,
           data: newFormData,
-        });
+        };
+        if (data.maxChars) {
+          newData.charCount = countTextInBlocks(
+            newFormData,
+            data.ignoreSpaces,
+            data.maxChars,
+          );
+        }
+        onChangeBlock(block, newData);
       }}
       onChangeField={(id, value) => {
         if (['blocks', 'blocks_layout'].indexOf(id) > -1) {
           blockState[id] = value;
-          onChangeBlock(block, {
+          const newChildData = {
+            ...data.data,
+            ...blockState,
+          };
+          const newData = {
             ...data,
-            data: {
-              ...data.data,
-              ...blockState,
-            },
-          });
+            data: newChildData,
+          };
+          if (data.maxChars) {
+            newData.charCount = countTextInBlocks(
+              newChildData,
+              data.ignoreSpaces,
+              data.maxChars,
+            );
+          }
+          onChangeBlock(block, newData);
         } else {
           onChangeField(id, value);
         }

package/src/components/manage/Blocks/Group/Edit.jsx

@@ -18,7 +18,7 @@ import PropTypes from 'prop-types';
 import { Segment } from 'semantic-ui-react';
 import EditSchema from './EditSchema';
 
-import CounterComponent from './CounterComponent';
+import CounterComponent, { countTextInBlocks } from './CounterComponent';
 import './editor.less';
 import { defineMessages, injectIntl } from 'react-intl';
 import { compose } from 'redux';
@@ -115,20 +115,29 @@ const Edit = (props) => {
     });
     const selectedIndex =
       data.data.blocks_layout.items.indexOf(selectedBlock) + 1;
-    onChangeBlock(block, {
-      ...data,
-      data: {
-        ...data?.data,
-        ...newBlockData,
-        blocks_layout: {
-          items: [
-            ...data.data.blocks_layout.items.slice(0, selectedIndex),
-            ...pastedBlocks,
-            ...data.data.blocks_layout.items.slice(selectedIndex),
-          ],
-        },
+    const newChildData = {
+      ...data?.data,
+      ...newBlockData,
+      blocks_layout: {
+        items: [
+          ...data.data.blocks_layout.items.slice(0, selectedIndex),
+          ...pastedBlocks,
+          ...data.data.blocks_layout.items.slice(selectedIndex),
+        ],
       },
-    });
+    };
+    const newData = {
+      ...data,
+      data: newChildData,
+    };
+    if (data.maxChars) {
+      newData.charCount = countTextInBlocks(
+        newChildData,
+        data.ignoreSpaces,
+        data.maxChars,
+      );
+    }
+    onChangeBlock(block, newData);
   };
 
   React.useEffect(() => {
@@ -137,10 +146,14 @@ const Edit = (props) => {
       childBlocksForm.blocks_layout.items[0] !== selectedBlock
     ) {
       setSelectedBlock(childBlocksForm.blocks_layout.items[0]);
-      onChangeBlock(block, {
+      const newData = {
         ...data,
         data: childBlocksForm,
-      });
+      };
+      if (data.maxChars) {
+        newData.charCount = 0;
+      }
+      onChangeBlock(block, newData);
     }
   }, [onChangeBlock, childBlocksForm, selectedBlock, block, data, data_blocks]);
 
@@ -218,10 +231,24 @@ const Edit = (props) => {
             title={props.intl.formatMessage(messages.sectionGroupSettings)}
             formData={data}
             onChangeField={(id, value) => {
-              props.onChangeBlock(props.block, {
+              const newData = {
                 ...props.data,
                 [id]: value,
-              });
+              };
+              const effectiveMaxChars =
+                id === 'maxChars' ? value : props.data.maxChars;
+              const effectiveIgnoreSpaces =
+                id === 'ignoreSpaces' ? value : props.data.ignoreSpaces;
+              if (effectiveMaxChars) {
+                newData.charCount = countTextInBlocks(
+                  props.data?.data,
+                  effectiveIgnoreSpaces,
+                  effectiveMaxChars,
+                );
+              } else {
+                delete newData.charCount;
+              }
+              props.onChangeBlock(props.block, newData);
             }}
           />
         )}

package/src/components/manage/Blocks/Group/LayoutSchema.jsx

@@ -11,6 +11,7 @@ const Schema = {
         'allowedBlocks',
         'as',
         'maxChars',
+        'maxCharsOverflowPercent',
         'ignoreSpaces',
         'readOnlySettings',
         'disableInnerButtons',
@@ -68,6 +69,13 @@ const Schema = {
       type: 'integer',
       factory: 'Integer',
     },
+    maxCharsOverflowPercent: {
+      title: 'Maximum Characters Overflow Percent',
+      description:
+        'The percentage of characters that can overflow the maximum characters limit before showing a warning: 0-100',
+      type: 'integer',
+      factory: 'Integer',
+    },
     ignoreSpaces: {
       title: 'Ignore spaces',
       description: 'Ignore spaces while calculating maximum characters',

package/src/components/manage/Blocks/Group/editor.less

@@ -71,8 +71,9 @@
   }
 
   .blocks-chooser {
-    right: 0;
-    left: auto;
+    position: absolute;
+    right: 0 !important;
+    left: auto !important;
     margin-top: 3rem;
   }