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

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

@@ -2,6 +2,16 @@ 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)
 ------------------
 

{collective_tiles_carousel-1.1.0 → 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.1.0
+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,16 @@ 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)
 ------------------
 

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

@@ -15,7 +15,7 @@ long_description = "\n\n".join(
 
 setup(
     name="collective.tiles.carousel",
-    version="1.1.0",
+    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.1.0 → 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.1.0 → 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
@@ -257,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.1.0 → 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.1.0
+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,16 @@ 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)
 ------------------