feat: implemented separator form field

packages/form-js-carbon-styles/test/spec/complex.json

@@ -600,6 +600,15 @@
         "suffixAdorner": "suffix"
       }
     },
+    {
+      "type": "separator",
+      "id": "Field_1a82jj4sdsd"
+    },
+    {
+      "type": "spacer",
+      "height": 30,
+      "id": "Field_1a82jjsdasd"
+    },
     {
       "action": "reset",
       "label": "reset",

packages/form-js-editor/assets/form-js-editor-base.css

@@ -710,6 +710,10 @@
   background: repeating-linear-gradient( 45deg, #ffffff, #ffffff 10px, #f5f5f5 10px, #f5f5f5 20px);
 }
 
+.fjs-editor-container .fjs-form-field-separator hr {
+  margin: 10px 2px;
+}
+
 /**
  * Custom Dragula Styles
  */

packages/form-js-editor/src/features/properties-panel/PropertiesPanelHeaderProvider.js

@@ -16,6 +16,7 @@ const labelsByType = {
   number: 'NUMBER',
   radio: 'RADIO',
   select: 'SELECT',
+  separator: 'SEPARATOR',
   spacer: 'SPACER',
   taglist: 'TAGLIST',
   text: 'TEXT VIEW',

packages/form-js-editor/src/features/properties-panel/groups/GeneralGroup.js

@@ -41,6 +41,10 @@ export default function GeneralGroup(field, editField, getService) {
     ...ReadonlyEntry({ field, editField })
   ];
 
+  if (entries.length === 0) {
+    return null;
+  }
+
   return {
     id: 'general',
     label: 'General',

packages/form-js-editor/test/spec/features/palette/Palette.spec.js

@@ -44,7 +44,7 @@ describe('palette', function() {
     const result = createPalette({ container });
 
     // then
-    expect(result.container.querySelectorAll('.fjs-palette-field')).to.have.length(14);
+    expect(result.container.querySelectorAll('.fjs-palette-field')).to.have.length(PALETTE_ENTRIES.length);
 
     expectEntries(result.container, PALETTE_ENTRIES);
   });
@@ -56,7 +56,7 @@ describe('palette', function() {
     const result = createPalette({ container });
 
     // then
-    expect(result.container.querySelectorAll('.fjs-palette-group')).to.have.length(4);
+    expect(result.container.querySelectorAll('.fjs-palette-group')).to.have.length(PALETTE_GROUPS.length);
 
     expectGroups(result.container, PALETTE_GROUPS);
   });

packages/form-js-playground/test/spec/form.json

@@ -50,6 +50,9 @@
       "label": "Checkbox",
       "key": "isGood"
     },
+    {
+      "type": "separator"
+    },
     {
       "type": "select",
       "label": "Select me",

packages/form-js-viewer/assets/form-js-base.css

@@ -430,6 +430,15 @@
   justify-content: center;
 }
 
+.fjs-container hr {
+  border: none;
+  border-top: solid 2px var(--color-borders-group);
+}
+
+.fjs-container .fjs-form-field-separator hr {
+  margin: 7px 2px;
+}
+
 .fjs-container .fjs-input[type='text'],
 .fjs-container .fjs-input[type='email'],
 .fjs-container .fjs-input[type='tel'],

packages/form-js-viewer/src/index.js

@@ -5,7 +5,7 @@ export * from './render';
 export * from './util';
 export * from './features';
 
-const schemaVersion = 11;
+const schemaVersion = 12;
 
 export {
   Form,

packages/form-js-viewer/src/render/components/form-fields/Separator.js

@@ -0,0 +1,22 @@
+import { formFieldClasses } from '../Util';
+
+const type = 'separator';
+
+export default function Separator() {
+
+  return (
+    <div class={ formFieldClasses(type) }>
+      <hr />
+    </div>
+  );
+}
+
+Separator.config = {
+  type,
+  keyed: false,
+  label: 'Separator',
+  group: 'presentation',
+  create: (options = {}) => ({
+    ...options
+  })
+};

packages/form-js-viewer/src/render/components/icons/index.js

@@ -8,6 +8,7 @@ import ColumnsIcon from './Group.svg';
 import NumberIcon from './Number.svg';
 import RadioIcon from './Radio.svg';
 import SelectIcon from './Select.svg';
+import SeparatorIcon from './Separator.svg';
 import SpacerIcon from './Spacer.svg';
 import TextIcon from './Text.svg';
 import TextfieldIcon from './Textfield.svg';
@@ -27,6 +28,7 @@ export const iconsByType = (type) => {
     number: NumberIcon,
     radio: RadioIcon,
     select: SelectIcon,
+    separator: SeparatorIcon,
     spacer: SpacerIcon,
     taglist: TaglistIcon,
     text: TextIcon,

packages/form-js-viewer/src/render/components/index.js

@@ -10,6 +10,7 @@ import Image from './form-fields/Image';
 import Numberfield from './form-fields/Number';
 import Radio from './form-fields/Radio';
 import Select from './form-fields/Select';
+import Separator from './form-fields/Separator';
 import Spacer from './form-fields/Spacer';
 import Taglist from './form-fields/Taglist';
 import Text from './form-fields/Text';
@@ -29,6 +30,7 @@ export {
   Numberfield,
   Radio,
   Select,
+  Separator,
   Spacer,
   Taglist,
   Text,
@@ -48,6 +50,7 @@ export const formFields = [
   Radio,
   Select,
   Spacer,
+  Separator,
   Taglist,
   Text,
   Textfield,

packages/form-js-viewer/test/spec/render/components/form-fields/Separator.spec.js

@@ -0,0 +1,69 @@
+import { render } from '@testing-library/preact/pure';
+
+import Separator from '../../../../../src/render/components/form-fields/Separator';
+
+import {
+  createFormContainer
+} from '../../../../TestHelper';
+
+import { WithFormContext } from './helper';
+
+let container;
+
+
+describe('Separator', function() {
+
+  beforeEach(function() {
+    container = createFormContainer();
+  });
+
+  afterEach(function() {
+    container.remove();
+  });
+
+  it('should render', function() {
+
+    // when
+    const { container } = createSeparator();
+
+    // then
+    const formField = container.querySelector('.fjs-form-field');
+
+    expect(formField).to.exist;
+    expect(formField.classList.contains('fjs-form-field-separator')).to.be.true;
+
+    const separator = formField.querySelector('hr');
+    expect(separator).to.exist;
+
+  });
+
+});
+
+// helpers //////////
+
+const defaultField = {
+  type: 'separator'
+};
+
+function createSeparator(options = {}) {
+  const {
+    data = {},
+    initialData = {},
+    properties = {},
+    field = defaultField
+  } = options;
+
+  return render(WithFormContext(
+    <Separator field={ field }
+    />,
+    {
+      ...options,
+      properties,
+      initialData,
+      data
+    }
+  ),
+  {
+    container: options.container || container.querySelector('.fjs-form')
+  });
+}

packages/form-js/test/spec/Form.spec.js

@@ -128,7 +128,7 @@ describe('viewer exports', function() {
   it('should expose schemaVersion', function() {
     expect(typeof schemaVersion).to.eql('number');
 
-    expect(schemaVersion).to.eql(11);
+    expect(schemaVersion).to.eql(12);
   });