collective.tiles.carousel 1.0.4__tar.gz → 1.1.1__tar.gz

{collective_tiles_carousel-1.0.4 → collective_tiles_carousel-1.1.1}/CHANGES.rst

@@ -2,6 +2,23 @@ Changelog
 =========
 
 
+1.1.1 (2025-05-09)
+------------------
+
+- Show indicators and controls only if there are more than one slide.
+  [petschki]
+
+- Improve result calculation.
+  [petschki]
+
+
+1.1.0 (2024-12-12)
+------------------
+
+- Use new `ContentBrowserWidget` for Plone 6.1.
+  [petschki]
+
+
 1.0.4 (2024-05-02)
 ------------------
 

{collective_tiles_carousel-1.0.4 → collective_tiles_carousel-1.1.1}/PKG-INFO

@@ -1,6 +1,6 @@
-Metadata-Version: 2.1
+Metadata-Version: 2.2
 Name: collective.tiles.carousel
-Version: 1.0.4
+Version: 1.1.1
 Summary: Slider for plone.app.mosaic based on Bootstrap 5
 Home-page: https://github.com/collective/collective.tiles.carousel
 Author: Peter Holzer
@@ -47,6 +47,18 @@ Requires-Dist: plone.browserlayer; extra == "test"
 Requires-Dist: plone.testing; extra == "test"
 Requires-Dist: plone.app.robotframework[debug]; extra == "test"
 Requires-Dist: robotsuite; extra == "test"
+Dynamic: author
+Dynamic: author-email
+Dynamic: classifier
+Dynamic: description
+Dynamic: home-page
+Dynamic: keywords
+Dynamic: license
+Dynamic: project-url
+Dynamic: provides-extra
+Dynamic: requires-dist
+Dynamic: requires-python
+Dynamic: summary
 
 .. This README is meant for consumption by humans and pypi. Pypi can render rst files so please do not use Sphinx features.
    If you want to learn more about writing documentation, please check out: http://docs.plone.org/about/documentation_styleguide.html
@@ -108,6 +120,23 @@ Changelog
 =========
 
 
+1.1.1 (2025-05-09)
+------------------
+
+- Show indicators and controls only if there are more than one slide.
+  [petschki]
+
+- Improve result calculation.
+  [petschki]
+
+
+1.1.0 (2024-12-12)
+------------------
+
+- Use new `ContentBrowserWidget` for Plone 6.1.
+  [petschki]
+
+
 1.0.4 (2024-05-02)
 ------------------
 

{collective_tiles_carousel-1.0.4 → collective_tiles_carousel-1.1.1}/setup.py

@@ -15,7 +15,7 @@ long_description = "\n\n".join(
 
 setup(
     name="collective.tiles.carousel",
-    version="1.0.4",
+    version="1.1.1",
     description="Slider for plone.app.mosaic based on Bootstrap 5",
     long_description=long_description,
     # Get more from https://pypi.org/classifiers/

{collective_tiles_carousel-1.0.4 → collective_tiles_carousel-1.1.1}/src/collective/tiles/carousel/tests/robot/test_example.robot

@@ -23,13 +23,12 @@
 
 *** Settings *****************************************************************
 
-Resource  plone/app/robotframework/selenium.robot
-Resource  plone/app/robotframework/keywords.robot
+Resource  plone/app/robotframework/browser.robot
 
 Library  Remote  ${PLONE_URL}/RobotRemote
 
-Test Setup  Open test browser
-Test Teardown  Close all browsers
+Test Setup  Run keywords  Plone Test Setup
+Test Teardown  Run keywords  Plone Test Teardown
 
 
 *** Test Cases ***************************************************************
@@ -47,20 +46,19 @@ Scenario: As a member I want to be able to log into the website
 
 a login form
   Go To  ${PLONE_URL}/login_form
-  Wait until page contains  Login Name
-  Wait until page contains  Password
+  Get Text    //body    contains    Login Name
+  Get Text    //body    contains    Password
 
 
 # --- WHEN -------------------------------------------------------------------
 
 I enter valid credentials
-  Input Text  __ac_name  admin
-  Input Text  __ac_password  secret
-  Click Button  Log in
+  Type Text    //input[@name="__ac_name"]    admin
+  Type Text    //input[@name="__ac_password"]    secret
+  Click    //button[contains(text(), "Log in")]
 
 
 # --- THEN -------------------------------------------------------------------
 
 I am logged in
-  Wait until page contains  You are now logged in
-  Page should contain  You are now logged in
+  Get Text    //body    contains    You are now logged in

{collective_tiles_carousel-1.0.4 → collective_tiles_carousel-1.1.1}/src/collective/tiles/carousel/tile.pt

@@ -13,12 +13,11 @@
                  darkvariant python:view.data.get('darkvariant') and 'carousel-dark' or '';
                  crossfade python:view.data.get('crossfade') and 'carousel-fade' or '';
                  link_slides python:view.data.get('link_slides') != 'disabled';
-                 items_per_slide python:view.data.get('items_per_slide') and view.data.get('items_per_slide') or 1;
+                 items_per_slide python:view.data.get('items_per_slide', 1) or 1;
                  items_responsive python: items_per_slide > 1 and 'row-cols-2' or 'row-cols-1';
                  items_responsive python: items_per_slide == 3 and 'row-cols-1' or items_responsive;
                  autoplay_interval view/data/carousel_speed|nothing;
                ">
-
         <div class="tileHeader"
              tal:condition="python:view.data['title'] or view.data['description']"
         >
@@ -41,7 +40,7 @@
         >
 
           <div class="carousel-indicators"
-               tal:condition="view/data/indicators"
+               tal:condition="python:view.data.get('indicators')"
           >
             <tal:indicators repeat="item items">
               <button class="${python:repeat.item.index == 0 and 'active' or None}"
@@ -82,7 +81,7 @@
              href="#carousel-${view/id}"
              role="button"
              data-bs-slide="prev"
-             tal:condition="view/data/controls"
+             tal:condition="python:view.data.get('controls') and len(items) > 1"
           >
             <span class="carousel-control-prev-icon"
                   aria-hidden="true"
@@ -95,7 +94,7 @@
              href="#carousel-${view/id}"
              role="button"
              data-bs-slide="next"
-             tal:condition="view/data/controls"
+             tal:condition="python:view.data.get('controls') and len(items) > 1"
           >
             <span class="carousel-control-next-icon"
                   aria-hidden="true"

{collective_tiles_carousel-1.0.4 → collective_tiles_carousel-1.1.1}/src/collective/tiles/carousel/tile.py

@@ -1,4 +1,3 @@
-from collections import OrderedDict
 from collective.tiles.carousel import _
 from collective.tiles.carousel.interfaces import ICollectiveTilesCarouselLayer
 from collective.tiles.carousel.utils import parse_query_from_data
@@ -8,7 +7,6 @@ from plone.app.contenttypes.browser.link_redirect_view import NON_RESOLVABLE_URL
 from plone.app.contenttypes.interfaces import ICollection
 from plone.app.contenttypes.utils import replace_link_variables_by_paths
 from plone.app.z3cform.widgets.querystring import QueryStringFieldWidget
-from plone.app.z3cform.widgets.relateditems import RelatedItemsFieldWidget
 from plone.autoform import directives as form
 from plone.dexterity.interfaces import IDexterityContainer
 from plone.memoize import view
@@ -30,6 +28,16 @@ from zope.schema.vocabulary import SimpleTerm
 from zope.schema.vocabulary import SimpleVocabulary
 
 
+try:
+    from plone.app.z3cform.widgets.contentbrowser import (
+        ContentBrowserFieldWidget as CarouselItemsWidget,
+    )
+except ImportError:
+    from plone.app.z3cform.widgets.relateditems import (
+        RelatedItemsFieldWidget as CarouselItemsWidget,
+    )
+
+
 @provider(IContextSourceBinder)
 def image_scales(context):
     """Return custom source for image scales.
@@ -74,11 +82,12 @@ class ISliderTile(Schema):
 
     form.widget(
         "carousel_items",
-        RelatedItemsFieldWidget,
+        CarouselItemsWidget,
         vocabulary="plone.app.vocabularies.Catalog",
         pattern_options={
             "orderable": True,
             "recentlyUsed": True,
+            "upload": True,
         },
     )
 
@@ -247,50 +256,37 @@ class SliderTile(Tile):
 
     @property
     def items(self):
-        items = OrderedDict()
+        result = []
+
         if len(self.data.get("carousel_items") or []):
             for item in self.data["carousel_items"]:
                 if ICollection.providedBy(item.to_object):
-                    items.update(
-                        OrderedDict.fromkeys(
-                            [
-                                x.getObject()
-                                for x in item.to_object.results(
-                                    brains=True, batch=False
-                                )
-                            ]
+                    result += [
+                        x.getObject()
+                        for x in item.to_object.results(brains=True, batch=False)
+                    ]
+                elif IDexterityContainer.providedBy(item.to_object):
+                    result += [
+                        x.getObject()
+                        for x in api.content.find(
+                            path="/".join(item.to_object.getPhysicalPath()),
+                            sort_on="getObjPositionInParent",
+                            depth=1,
                         )
-                    )
-                    continue
-                if IDexterityContainer.providedBy(item.to_object):
-                    items.update(
-                        OrderedDict.fromkeys(
-                            [
-                                x.getObject()
-                                for x in api.content.find(
-                                    path="/".join(item.to_object.getPhysicalPath()),
-                                    sort_on="getObjPositionInParent",
-                                    depth=1,
-                                )
-                            ]
-                        )
-                    )
-                    continue
+                    ]
                 else:
-                    items[item.to_object] = None
+                    result.append(item.to_object)
 
         query = self.query
+        limit = self.data.get("limit") or 12
+
         if query:
-            items.update(
-                OrderedDict.fromkeys([x.getObject() for x in self.catalog(**query)])
-            )
+            # limit catalog query to our limit
+            query["sort_limit"] = limit
+            result += [x.getObject() for x in self.catalog(**query)]
 
-        result = []
-        limit = self.data.get("limit") or 12
-        for count, obj in enumerate(items.keys(), 1):
-            result.append(obj)
-            if count >= limit:
-                break
+        # limit result
+        result = result[:limit]
         ips = self.data.get("items_per_slide", 1) or 1
         slides = [
             result[i : i + ips]

{collective_tiles_carousel-1.0.4 → collective_tiles_carousel-1.1.1}/src/collective.tiles.carousel.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
-Metadata-Version: 2.1
+Metadata-Version: 2.2
 Name: collective.tiles.carousel
-Version: 1.0.4
+Version: 1.1.1
 Summary: Slider for plone.app.mosaic based on Bootstrap 5
 Home-page: https://github.com/collective/collective.tiles.carousel
 Author: Peter Holzer
@@ -47,6 +47,18 @@ Requires-Dist: plone.browserlayer; extra == "test"
 Requires-Dist: plone.testing; extra == "test"
 Requires-Dist: plone.app.robotframework[debug]; extra == "test"
 Requires-Dist: robotsuite; extra == "test"
+Dynamic: author
+Dynamic: author-email
+Dynamic: classifier
+Dynamic: description
+Dynamic: home-page
+Dynamic: keywords
+Dynamic: license
+Dynamic: project-url
+Dynamic: provides-extra
+Dynamic: requires-dist
+Dynamic: requires-python
+Dynamic: summary
 
 .. This README is meant for consumption by humans and pypi. Pypi can render rst files so please do not use Sphinx features.
    If you want to learn more about writing documentation, please check out: http://docs.plone.org/about/documentation_styleguide.html
@@ -108,6 +120,23 @@ Changelog
 =========
 
 
+1.1.1 (2025-05-09)
+------------------
+
+- Show indicators and controls only if there are more than one slide.
+  [petschki]
+
+- Improve result calculation.
+  [petschki]
+
+
+1.1.0 (2024-12-12)
+------------------
+
+- Use new `ContentBrowserWidget` for Plone 6.1.
+  [petschki]
+
+
 1.0.4 (2024-05-02)
 ------------------