@kitconcept/volto-light-theme 1.0.0-rc.1 → 1.0.0-rc.2

package/CHANGELOG.md

@@ -8,6 +8,14 @@
 
 <!-- towncrier release notes start -->
 
+## 1.0.0-rc.2 (2023-07-07)
+
+### Bugfix
+
+- Add NewsItemView @iFlamieng [#127](https://github.com/kitconcept/volto-light-theme/pull/127)
+- Add support for margins in responsive. Improve the spacing in grids. @sneridagh [#129](https://github.com/kitconcept/volto-light-theme/pull/129)
+
+
 ## 1.0.0-rc.1 (2023-07-05)
 
 ### Bugfix

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kitconcept/volto-light-theme",
-  "version": "1.0.0-rc.1",
+  "version": "1.0.0-rc.2",
   "description": "Volto Light Theme by kitconcept",
   "main": "src/index.js",
   "repository": {

package/src/customizations/volto/components/theme/View/NewsItemView.jsx

@@ -0,0 +1,57 @@
+/**
+ * OVERRIDE NewsItemView.jsx
+ * REASON: BFS theme
+ * DATE: 2023-07-04
+ * DEVELOPER: @IFlameing
+ */
+
+/**
+ * NewsItemView view component.
+ * @module components/theme/View/NewsItemView
+ */
+
+import React from 'react';
+import PropTypes from 'prop-types';
+import RenderBlocks from '@plone/volto/components/theme/View/RenderBlocks';
+import { FormattedDate } from '@plone/volto/components';
+
+/**
+ * NewsItemView view component class.
+ * @function NewsItemView
+ * @params {object} content Content object.
+ * @returns {string} Markup of the component.
+ */
+const NewsItemView = ({ content }) => {
+  return (
+    <div id="page-document" className="ui container viewwrapper event-view">
+      <div className="dates">
+        {content?.effective ? (
+          <span className="day">
+            <FormattedDate date={content?.effective} />{' '}
+          </span>
+        ) : (
+          <span className="day">No date</span>
+        )}{' '}
+        <span className="headtitle">| {content?.head_title}</span>
+      </div>
+      <RenderBlocks content={content} />
+    </div>
+  );
+};
+
+/**
+ * Property types.
+ * @property {Object} propTypes Property types.
+ * @static
+ */
+NewsItemView.propTypes = {
+  content: PropTypes.shape({
+    title: PropTypes.string,
+    description: PropTypes.string,
+    text: PropTypes.shape({
+      data: PropTypes.string,
+    }),
+  }).isRequired,
+};
+
+export default NewsItemView;

package/src/theme/_bgcolor-blocks-layout.scss

@@ -31,7 +31,9 @@
     margin-bottom: $grid-block-vertical-spacing-bottom;
   }
   &.next--is--same--block-type.next--has--same--backgroundColor {
-    margin-bottom: -1rem;
+    // We rely on the grid gutter to keep the vertical spacing in case grid + grid
+    // So here we cancel the default block margin-bottom
+    margin-bottom: 0;
   }
 }
 

package/src/theme/_content.scss

@@ -5,3 +5,28 @@
   padding: 0;
   margin-top: 0;
 }
+
+.contenttype-news-item {
+  .documentFirstHeading {
+    margin-top: 0px;
+  }
+  .blocks-group-wrapper {
+    padding-top: 0px;
+  }
+  .dates {
+    max-width: var(--default-container-width) !important;
+    margin-top: 40px;
+    margin-right: auto;
+    margin-bottom: 40px;
+    margin-left: auto;
+    .day,
+    .headtitle {
+      font-size: 18px;
+
+      font-weight: 700;
+      letter-spacing: 1px;
+      line-height: 24px;
+      text-transform: uppercase;
+    }
+  }
+}

package/src/theme/_layout.scss

@@ -4,12 +4,12 @@
   --narrow-container-width: 620px;
 }
 
-// Container queries still do not work with CSS properties
+// Container queries still do not work with CSS custom properties
 // They should be exact numbers
 // For now, maintain in sync with the above
-$layout-container-width: 1440px;
-$default-container-width: 940px;
-$narrow-container-width: 620px;
+$layout-container-width: 1440px !default;
+$default-container-width: 940px !default;
+$narrow-container-width: 620px !default;
 
 @mixin narrow-container-width() {
   max-width: var(--narrow-container-width);
@@ -21,6 +21,13 @@ $narrow-container-width: 620px;
   max-width: var(--default-container-width);
   margin-right: auto;
   margin-left: auto;
+
+  // This is mostly aesthetical if you don't want to "see" the jump
+  // but a nice animation instead - PoC (WIP)
+  // @media only screen and (max-width: #{$default-container-width + 5px}) {
+  //   margin-right: 5px;
+  //   margin-left: 5px;
+  // }
 }
 
 @mixin layout-container-width() {
@@ -29,11 +36,24 @@ $narrow-container-width: 620px;
   margin-left: auto;
 }
 
-// One still cannot use variables or CSS properties directly on @container queries
+// One still cannot use CSS custom properties directly on @container queries
 @mixin adjustMarginsToContainer($width) {
+  // Next two ones are mostly aesthetical if you don't want to "see" the jump - PoC (WIP)
+  // transition: margin 1s ease;
+
+  // @container (max-width: #{$width + 5px}) {
+  //   margin-right: 5px;
+  //   margin-left: 5px;
+  // }
+
+  // @container (max-width: #{$width}) {
+  //   margin-right: 0px;
+  //   margin-left: 0px;
+  // }
+
   @container (max-width: #{$width - 1}) {
-    margin-right: 2rem;
-    margin-left: 2rem;
+    margin-right: 20px;
+    margin-left: 20px;
   }
 }
 
@@ -127,7 +147,9 @@ footer {
   & > pre,
   & > .block.code {
     @include narrow-container-width();
-    @include adjustMarginsToContainer($narrow-container-width + 2 * 36px);
+    // Why was this for? Removing for now (Victor: 2023-07-06)
+    // @include adjustMarginsToContainer($narrow-container-width + 2 * 36px);
+    @include adjustMarginsToContainer($narrow-container-width);
   }
 
   & > h1.documentFirstHeading,
@@ -146,6 +168,7 @@ footer {
   & > .table-of-contents,
   & > .slate blockquote {
     @include default-container-width();
+    @include adjustMarginsToContainer($default-container-width);
   }
 
   & > .block.teaser,
@@ -234,7 +257,18 @@ footer {
 .block.gridBlock .grid-items,
 .block.gridBlock h2.headline {
   @include default-container-width();
-  max-width: calc(var(--default-container-width) + 1rem);
+  // Adding 2 * 20px (one for each side) we force the container to extend the value of
+  // the gutter
+  max-width: calc(var(--default-container-width) + 2 * 20px);
+}
+
+.block.gridBlock {
+  .ui.grid > .column:not(.row):first-child {
+    padding-left: 20px;
+  }
+  .ui.grid > .column:not(.row):last-child {
+    padding-right: 20px;
+  }
 }
 
 // Fix for Image Grid with only one image
@@ -252,6 +286,11 @@ body.has-toolbar.has-sidebar .block .ui.basic.button.delete-button {
   margin-right: -30px !important;
 }
 
+// Listings edge case, conflicting with Pastanaga CSS
+.listing-item {
+  width: initial;
+}
+
 #page-add,
 #page-edit {
   .block-editor-accordion {

package/src/theme/collections/grid.variables

@@ -1,2 +1,7 @@
 // We want the separation between cells in grids to be 1rem only
-@gutterWidth: 1rem;
+@gutterWidth: 20px;
+@rowSpacing: 20px;
+
+// This has to be 2 * 20px because it's going to be 1/2 in the source.
+@stackableGutter: 2 * 20px;
+