Merge pull request #7 from codesyntax/develop

Develop

Author: erral

.gitignore

@@ -41,4 +41,6 @@ var/
 /sources/
 /venv/
 constraints*.txt
-requirements*.txt
+requirements*.txt
+.mxdev_cache
+

news/7.feat

@@ -0,0 +1,4 @@
+- Add icons to views @libargutxi
+- Add row spacing configuration @libargutxi
+- Add behavior to set fetchpriority/loading attributes to images @libargutxi
+- Complete translations @erral

pyproject.toml

@@ -3,7 +3,7 @@ name = "cs_dynamicpages"
 dynamic = ["version"]
 description = "A new addon for Plone"
 readme = "README.md"
-requires-python = ">=3.10"
+requires-python = ">=3.10,<3.14"
 authors = [{ name = "CodeSyntax", email = "plone@codesyntax.com" }]
 keywords = ["CMS", "Plone", "Python"]
 classifiers = [

src/cs_dynamicpages/behaviors/configure.zcml

@@ -12,6 +12,14 @@
       />
 
   <!-- -*- extra stuff goes here -*- -->
+  <plone:behavior
+      name="cs_dynamicpages.row_vertical_spacing"
+      title="RowVerticalSpacing"
+      description="This behavior provides..."
+      factory=".row_vertical_spacing.RowVerticalSpacing"
+      provides=".row_vertical_spacing.IRowVerticalSpacing"
+      marker=".row_vertical_spacing.IRowVerticalSpacingMarker"
+      />
 
   <plone:behavior
       name="cs_dynamicpages.row_width"
@@ -51,6 +59,15 @@
       marker=".related_image.IRelatedImageMarker"
       />
 
+  <plone:behavior
+      name="cs_dynamicpages.fetchpriority_image"
+      title="FetchPriorityImage"
+      description="This behavior provides..."
+      factory=".fetchpriority_image.FetchPriorityImage"
+      provides=".fetchpriority_image.IFetchPriorityImage"
+      marker=".fetchpriority_image.IFetchPriorityImageMarker"
+      />
+
 
   <plone:behavior
       name="cs_dynamicpages.link_info"

src/cs_dynamicpages/behaviors/fetchpriority_image.py

@@ -0,0 +1,46 @@
+from cs_dynamicpages import _
+from plone import schema
+from plone.autoform.interfaces import IFormFieldProvider
+from plone.supermodel import model
+from Products.CMFPlone.utils import safe_hasattr
+from zope.component import adapter
+from zope.interface import implementer
+from zope.interface import Interface
+from zope.interface import provider
+
+
+class IFetchPriorityImageMarker(Interface):
+    pass
+
+
+@provider(IFormFieldProvider)
+class IFetchPriorityImage(model.Schema):
+    """ """
+
+    fetchpriority_image = schema.Bool(
+        title=_("Fetch priority image"),
+        description=_(
+            "Set this to true if you want to signal that this image has a 'high' fetch "
+            "priority, otherwise 'auto' will be used. When using 'high' the value of "
+            "the loading attribute will be set to 'eager', and 'lazy' when "
+            "it is not set."
+        ),
+        required=False,
+    )
+
+
+@implementer(IFetchPriorityImage)
+@adapter(IFetchPriorityImageMarker)
+class FetchPriorityImage:
+    def __init__(self, context):
+        self.context = context
+
+    @property
+    def fetchpriority_image(self):
+        if safe_hasattr(self.context, "fetchpriority_image"):
+            return self.context.fetchpriority_image
+        return None
+
+    @fetchpriority_image.setter
+    def fetchpriority_image(self, value):
+        self.context.fetchpriority_image = value

src/cs_dynamicpages/behaviors/link_info.py

@@ -1,5 +1,7 @@
 from cs_dynamicpages import _
 from plone import schema
+from plone.app.z3cform.widgets.link import LinkFieldWidget
+from plone.autoform import directives as form
 from plone.autoform.interfaces import IFormFieldProvider
 from plone.supermodel import model
 from Products.CMFPlone.utils import safe_hasattr
@@ -29,6 +31,11 @@ class ILinkInfo(model.Schema):
         required=False,
     )
 
+    form.widget(
+        "link_url",
+        LinkFieldWidget,
+    )
+
 
 @implementer(ILinkInfo)
 @adapter(ILinkInfoMarker)

src/cs_dynamicpages/behaviors/row_vertical_spacing.py

@@ -0,0 +1,92 @@
+from cs_dynamicpages import _
+from plone import schema
+from plone.autoform.interfaces import IFormFieldProvider
+from plone.supermodel import model
+from Products.CMFPlone.utils import safe_hasattr
+from zope.component import adapter
+from zope.interface import implementer
+from zope.interface import Interface
+from zope.interface import provider
+
+
+class IRowVerticalSpacingMarker(Interface):
+    pass
+
+
+@provider(IFormFieldProvider)
+class IRowVerticalSpacing(model.Schema):
+    padding_top = schema.Choice(
+        title=_("Row padding top"),
+        description=_("Select the padding top that this row will have"),
+        vocabulary="cs_dynamicpages.RowPaddingTop",
+        required=True,
+        default="pt-0",
+    )
+    padding_bottom = schema.Choice(
+        title=_("Row padding bottom"),
+        description=_("Select the padding bottom that this row will have"),
+        vocabulary="cs_dynamicpages.RowPaddingBottom",
+        required=True,
+        default="pb-0",
+    )
+    margin_top = schema.Choice(
+        title=_("Row margin top"),
+        description=_("Select the margin top that this row will have"),
+        vocabulary="cs_dynamicpages.RowMarginTop",
+        required=True,
+        default="mt-0",
+    )
+    margin_bottom = schema.Choice(
+        title=_("Row margin bottom"),
+        description=_("Select the margin bottom that this row will have"),
+        vocabulary="cs_dynamicpages.RowMarginBottom",
+        required=True,
+        default="mb-0",
+    )
+
+
+@implementer(IRowVerticalSpacing)
+@adapter(IRowVerticalSpacingMarker)
+class RowVerticalSpacing:
+    def __init__(self, context):
+        self.context = context
+
+    @property
+    def padding_top(self):
+        if safe_hasattr(self.context, "padding_top"):
+            return self.context.padding_top
+        return None
+
+    @padding_top.setter
+    def padding_top(self, value):
+        self.context.padding_top = value
+
+    @property
+    def padding_bottom(self):
+        if safe_hasattr(self.context, "padding_bottom"):
+            return self.context.padding_bottom
+        return None
+
+    @padding_bottom.setter
+    def padding_bottom(self, value):
+        self.context.padding_bottom = value
+
+    @property
+    def margin_top(self):
+        if safe_hasattr(self.context, "margin_top"):
+            return self.context.margin_top
+        return None
+
+    @margin_top.setter
+    def margin_top(self, value):
+        self.context.margin_top = value
+
+    @property
+    def margin_bottom(self):
+        if safe_hasattr(self.context, "margin_bottom"):
+            return self.context.margin_bottom
+        return None
+
+    @margin_bottom.setter
+    def margin_bottom(self, value):
+        self.context.margin_bottom = value

src/cs_dynamicpages/browser/body_class.py

@@ -0,0 +1,26 @@
+from plone import api
+from plone.app.layout.globals.layout import IBodyClassAdapter
+from zope.component import adapter
+from zope.interface import implementer
+from zope.interface import Interface
+
+
+@adapter(Interface, Interface)
+@implementer(IBodyClassAdapter)
+class DynamicViewFolderClasses:
+    def __init__(self, context, request):
+        self.context = context
+        self.request = request
+
+    def get_classes(self, template, view):
+        """Whenever we are in a dynamic-view, add a custom class
+        signaling that the user can edit the content
+        """
+        if template.id == "dynamic_view.pt":
+            can_edit = api.user.has_permission(
+                "Modify portal content", obj=self.context
+            )
+            if can_edit:
+                return ["can_edit"]
+            return []
+        return []

src/cs_dynamicpages/browser/configure.zcml

@@ -14,4 +14,6 @@
       type="plone"
       />
 
+  <adapter factory=".body_class.DynamicViewFolderClasses" />
+
 </configure>

src/cs_dynamicpages/browser/static/delete-row.js

@@ -16,7 +16,7 @@
     // Only run on dynamic-view with edit permissions
     if (
       !document.body.classList.contains("template-dynamic-view") ||
-      !document.body.classList.contains("userrole-manager")
+      !document.body.classList.contains("can_edit")
     ) {
       return;
     }
@@ -114,7 +114,10 @@
           if (modal) {
             modal.hide();
           }
-          sessionStorage.setItem('toast-message', 'Element deleted successfully.');
+          sessionStorage.setItem(
+            "toast-message",
+            "Element deleted successfully."
+          );
           // Refresh the page after successful update
           window.location.reload();
         });