Conjunto de mudanças 6552 no repositório publico

Timestamp:

25/06/2012 15:36:45 (9 anos atrás)

Autor:

mazza

Mensagem:

edicao do carrosel: adicionar e remover painel

Localização:

PortalInterlegis/produtos/il.portalinterlegis/trunk/src/il/portalinterlegis

Arquivos:

• browser/boxes/carousel.py (modificado) (2 diffs)
• browser/boxes/carouseledit.pt (modificado) (1 diff)
• browser/boxes/templates/carousel-edit.html (modificado) (2 diffs)
• sass/_carousel-edit.scss (modificado) (2 diffs)
• static/index.html (modificado) (1 diff)
• static/js/carousel-edit.js (adicionado)
• static/js/editable-box.js (modificado) (1 diff)

PortalInterlegis/produtos/il.portalinterlegis/trunk/src/il/portalinterlegis/browser/boxes/carousel.py

@@ -38 +38 @@
         for i in range(NUMBER_OF_PRE_CREATED_BOXES):
             if i not in numbers:
-                numbers.append(i)
+                numbers.insert(0, i)
                 break
 
-    def remove_item(self, index):
+    def remove_item(self, id):
+        index = self._index_from_id(id)
         self.numbers.remove(index)
-        Box(ICarouselItem, i).erase_data(self.context)
+        Box(ICarouselItem, index).erase_data(self.context)
 
-    def reorder(self, order):
-        self.numbers[:] = [int(o.split('_')[-1]) for o in order.split(',')]
+    def reorder(self, ids):
+        self.numbers[:] = [self._index_from_id(id) for id in ids.split(',')]
+
+    def _index_from_id(self, id):
+        return int(id.split('_')[-1])
 
 
@@ -66 +70 @@
         if 'add' in self.request:
             self._carousel.add_item()
-        elif 'del' in self.request:
-            self._carousel.remove_item()
+        elif 'remove' in self.request:
+            self._carousel.remove_item(self.request['remove'])
         elif 'reorder' in self.request:
             self._carousel.reorder(self.request['reorder'])
+    # TODO: ajustar o render para retornar apenas feedback de sucesso ou falha
 
 class CarouselBox(BaseBox):

PortalInterlegis/produtos/il.portalinterlegis/trunk/src/il/portalinterlegis/browser/boxes/carouseledit.pt

@@ -10 +10 @@
 <metal:content-core fill-slot="content">
 
-    <script>
-    $(function() {
+  <h1>Carrossel</h1>
 
-    $( ".column" ).sortable({
-                        update: function(event, ui) {
-                                var order = $(this).sortable('toArray').toString();
-                $.post('edit-carousel', {reorder:order});
-                        }
-    });
+  <h2>Edição</h2>
 
-    $( ".portlet" ).addClass( "ui-widget ui-widget-content ui-helper-clearfix ui-corner-all" )
-            .find( ".portlet-header" )
-                .addClass( "ui-widget-header ui-corner-all" )
-                .prepend( "<span class='ui-icon ui-icon-minusthick'></span>")
-                .end()
-            .find( ".portlet-content" );
+  <ul>
+    <li>Arreste os paineis para alterar a ordem em que aparecem no carrosel.</li>
+    <li>Clique em nos ícones <img src="pencil_icon.png" width="16" height="16" alt="Lápis"/>
+      em cada painel para editá-lo.</li>
+    <li>Ao terminar volte para a página original.</li>
+  </ul>
 
-        $( ".portlet-header .ui-icon" ).click(function() {
-            $( this ).toggleClass( "ui-icon-minusthick" ).toggleClass( "ui-icon-plusthick" );
-            $( this ).parents( ".portlet:first" ).find( ".portlet-content" ).toggle();
-        });
-
-        $( ".column" ).disableSelection();
-
-
-    });
-    </script>
-
-    <h1>Carrossel</h1>
-
-    <h2>Edição</h2>
-
-    <ul>
-      <li>Arreste os paineis para alterar a ordem em que aparecem no carrosel.</li>
-      <li>Clique em nos ícones <img src="pencil_icon.png" width="16" height="16" alt="Lápis"/>
-        em cada painel para editá-lo.</li>
-      <li>Ao terminar salve suas mudanças e volte para a página original.</li>
-    </ul>
-
-    <div tal:content="structure view/carousel" />
+  <div tal:content="structure view/carousel" />
 
 </metal:content-core>

PortalInterlegis/produtos/il.portalinterlegis/trunk/src/il/portalinterlegis/browser/boxes/templates/carousel-edit.html

@@ -1 @@
-<div class="column" id="carousel-edit-panels">
+<div id="carousel-edit-panels">
   {%- for number, data, edit_href in items %}
   <div class="portlet" id="carousel_item_{{ number }}">
@@ -20 +20 @@
   {%- endfor %}
 </div>
+<form id="carousel-add-form" method="post" action="edit-carousel">
+  <button id="carousel-add-button" type="submit" name="add" value="_">
+    Adicionar um novo painel
+  </button>
+</form>

PortalInterlegis/produtos/il.portalinterlegis/trunk/src/il/portalinterlegis/sass/_carousel-edit.scss

@@ -1 +1 @@
 // TODO: reescrever completamente isso para alinhar-se com o carousel.scss !!!!
-.column {
+// TODO: simpificar o javascript para simplesmente usar templates e scss
+#carousel-edit-panels {
     width: 700px;
     float: left;
     padding-bottom: 100px;
+    margin-top: 10px;
 }
 .portlet {
     margin: 0 1em 1em 0;
     width: $carousel-width;
-    /* height: $carousel-height; */
+    .editable-box {
+        min-height: $carousel-height;
+    }
 }
 .portlet-header {
@@ -26 +30 @@
     height: $carousel-height !important;
 }
+
+#carousel-add-form {
+    padding: 10px 0;
+    font-size: 16px;
+}
+.carousel-add-button {
+    font-size: 12px;
+}
 .ui-sortable-placeholder * {
     visibility: hidden;

PortalInterlegis/produtos/il.portalinterlegis/trunk/src/il/portalinterlegis/static/index.html

@@ -20 +20 @@
   <!-- É importante que isto venha depois de unifyHeightsOfDottedCells -->
   <script src="js/editable-box.js"></script>
+
+  <!-- TODO: colocar isso apenas para carouseledit.pt -->
+  <script src="js/carousel-edit.js"></script>
+
 </head>
 <body>

PortalInterlegis/produtos/il.portalinterlegis/trunk/src/il/portalinterlegis/static/js/editable-box.js

@@ -27 +27 @@
               },
               onClose: function() {
-                  location.reload();
+                  // location.reload() would resend post data sometimes
+                  location = location.href
               }
           },