Fix next/prev buttons when nitf id is not a good css selector

CHANGES.rst

@@ -6,6 +6,10 @@ There's a frood who really knows where his towel is.
 2.0b5 (unreleased)
 ^^^^^^^^^^^^^^^^^^
 
+- Simplify slideshow template to avoid depending on context id;
+  this solves an issue when id ends with ".html".
+  [rodfersou]
+
 - Refactor static resources.
   [rodfersou]
 

src/collective/nitf/browser/__init__.py

@@ -42,9 +42,6 @@ class Slideshow(DefaultView):
 
     """Slideshow view of a News Article."""
 
-    def id(self):
-        return id(self)
-
     def js_resources(self):
         """Return a list of JS resources that are not available in the
         registry, but need to be loaded anyway. This way the slideshow

src/collective/nitf/browser/templates/slideshow.pt

@@ -1,7 +1,7 @@
-<div class="slideshow-container"
+<div id="nitf-slideshow"
+     class="slideshow-container"
      tal:define="images context/get_images"
-     tal:condition="images"
-     tal:attributes="id string:photogallery-${context/id}">
+     tal:condition="images">
   <div class="slideshow-player">
     <div class="cycle-player cycle-slideshow"
          data-cycle-slides="> div"
@@ -10,8 +10,8 @@
          data-allow-wrap="true"
          data-cycle-fx="fade"
          data-cycle-log="false"
-         tal:attributes="data-cycle-prev string:#photogallery-${context/id} .cycle-prev;
-                         data-cycle-next string:#photogallery-${context/id} .cycle-next">
+         data-cycle-prev="#nitf-slideshow .cycle-prev"
+         data-cycle-next="#nitf-slideshow .cycle-next">
       <div tal:repeat="image images">
         <img tal:replace="structure image/@@images/image/large" />
       </div>