Enlightenment CVS committal

Author  : codewarrior
Project : e17
Module  : proto

Dir     : e17/proto/exhibit/src/bin


Modified Files:
        exhibit.h exhibit_comment.c exhibit_main.c exhibit_menus.c 
        exhibit_tab.c 


Log Message:
- the comments box should play nice now. if any more bugs are found, please do 
say.


===================================================================
RCS file: /cvs/e/e17/proto/exhibit/src/bin/exhibit.h,v
retrieving revision 1.17
retrieving revision 1.18
diff -u -3 -r1.17 -r1.18
--- exhibit.h   28 Jul 2006 16:04:47 -0000      1.17
+++ exhibit.h   29 Jul 2006 02:49:24 -0000      1.18
@@ -64,7 +64,9 @@
    
    int    comments_visible;
 
-        Ex_Images im_view;
+   int im_view;
+   
+   int default_sort;
 };
 
 struct _Ex_Tab
@@ -89,6 +91,19 @@
    Etk_Tree_Col  *icol;
    
    Exhibit       *e;
+   
+   struct {
+      Etk_Bool    visible;
+      Etk_Widget *vbox;
+      Etk_Widget *frame;
+      Etk_Widget *table;
+      Etk_Widget *entry;
+      Etk_Widget *save;
+      Etk_Widget *revert;
+      Etk_Widget *vbox2;
+      Etk_Widget *hbox;
+      char       *text;
+   } comment;   
 };
 
 
@@ -113,19 +128,6 @@
    Etk_Widget    *zoombar;
    Etk_Widget    *menu;
    Etk_Widget    *win;
-
-   struct {
-      Etk_Bool    visible;
-      Etk_Widget *vbox;
-      Etk_Widget *frame;
-      Etk_Widget *table;
-      Etk_Widget *entry;
-      Etk_Widget *save;
-      Etk_Widget *revert;
-      Etk_Widget *vbox2;
-      Etk_Widget *hbox;
-      char       *text;
-   } comment;
    
    char           fav_path[PATH_MAX];
 
===================================================================
RCS file: /cvs/e/e17/proto/exhibit/src/bin/exhibit_comment.c,v
retrieving revision 1.5
retrieving revision 1.6
diff -u -3 -r1.5 -r1.6
--- exhibit_comment.c   28 Jul 2006 16:04:47 -0000      1.5
+++ exhibit_comment.c   29 Jul 2006 02:49:24 -0000      1.6
@@ -6,35 +6,47 @@
 void
 _ex_comment_show(Exhibit *e)
 {
-   if(e->comment.visible)
+   if(e->cur_tab->comment.visible)
      return;
    
-   e->comment.vbox = etk_vbox_new(ETK_FALSE, 0);
-   etk_paned_child2_set(ETK_PANED(e->hpaned), e->comment.vbox, ETK_TRUE);
-   
-   etk_box_pack_start(ETK_BOX(e->comment.vbox), e->cur_tab->scrolled_view, 
ETK_TRUE, ETK_TRUE, 0);
+   e->cur_tab->comment.vbox = etk_vbox_new(ETK_FALSE, 0);
+   if(evas_list_count(e->tabs) == 1)
+     {
+       /* we only have 1 tab, ie, no notebook */
+       etk_paned_child2_set(ETK_PANED(e->hpaned), e->cur_tab->comment.vbox, 
ETK_TRUE);
+     }
+   else
+     {
+       /* we have multiple tabs */
+       etk_notebook_page_child_set(ETK_NOTEBOOK(e->notebook), e->cur_tab->num, 
e->cur_tab->comment.vbox);
+     }
+
+   if (e->cur_tab->fit_window)
+     etk_box_pack_start(ETK_BOX(e->cur_tab->comment.vbox), 
e->cur_tab->alignment, ETK_TRUE, ETK_TRUE, 0);
+   else
+     etk_box_pack_start(ETK_BOX(e->cur_tab->comment.vbox), 
e->cur_tab->scrolled_view, ETK_TRUE, ETK_TRUE, 0);
    
-   e->comment.frame = etk_frame_new(_("Image Comments"));
-   etk_box_pack_start(ETK_BOX(e->comment.vbox), e->comment.frame, ETK_FALSE, 
ETK_FALSE, 3);
+   e->cur_tab->comment.frame = etk_frame_new(_("Image Comments"));
+   etk_box_pack_start(ETK_BOX(e->cur_tab->comment.vbox), 
e->cur_tab->comment.frame, ETK_FALSE, ETK_FALSE, 3);
    
-   e->comment.entry = etk_entry_new();
-   e->comment.save = etk_button_new_with_label("Save");
-   etk_signal_connect("clicked", ETK_OBJECT(e->comment.save), 
ETK_CALLBACK(_ex_comment_save_clicked_cb), e);
-   e->comment.revert = etk_button_new_with_label("Revert");
-   etk_signal_connect("clicked", ETK_OBJECT(e->comment.revert), 
ETK_CALLBACK(_ex_comment_revert_clicked_cb), e);
+   e->cur_tab->comment.entry = etk_entry_new();
+   e->cur_tab->comment.save = etk_button_new_with_label("Save");
+   etk_signal_connect("clicked", ETK_OBJECT(e->cur_tab->comment.save), 
ETK_CALLBACK(_ex_comment_save_clicked_cb), e);
+   e->cur_tab->comment.revert = etk_button_new_with_label("Revert");
+   etk_signal_connect("clicked", ETK_OBJECT(e->cur_tab->comment.revert), 
ETK_CALLBACK(_ex_comment_revert_clicked_cb), e);
    
-   e->comment.vbox2 = etk_vbox_new(ETK_FALSE, 0);
-   e->comment.hbox = etk_hbox_new(ETK_FALSE, 0);
+   e->cur_tab->comment.vbox2 = etk_vbox_new(ETK_FALSE, 0);
+   e->cur_tab->comment.hbox = etk_hbox_new(ETK_FALSE, 0);
    
-   etk_container_add(ETK_CONTAINER(e->comment.frame), e->comment.vbox2);
+   etk_container_add(ETK_CONTAINER(e->cur_tab->comment.frame), 
e->cur_tab->comment.vbox2);
    
-   etk_box_pack_start(ETK_BOX(e->comment.vbox2), e->comment.entry, ETK_TRUE, 
ETK_TRUE, 0);
-   etk_box_pack_start(ETK_BOX(e->comment.vbox2), e->comment.hbox, ETK_FALSE, 
ETK_FALSE, 0);
+   etk_box_pack_start(ETK_BOX(e->cur_tab->comment.vbox2), 
e->cur_tab->comment.entry, ETK_TRUE, ETK_TRUE, 0);
+   etk_box_pack_start(ETK_BOX(e->cur_tab->comment.vbox2), 
e->cur_tab->comment.hbox, ETK_FALSE, ETK_FALSE, 0);
    
-   etk_box_pack_start(ETK_BOX(e->comment.hbox), e->comment.revert, ETK_FALSE, 
ETK_FALSE, 0);
-   etk_box_pack_start(ETK_BOX(e->comment.hbox), e->comment.save, ETK_FALSE, 
ETK_FALSE, 0);
+   etk_box_pack_start(ETK_BOX(e->cur_tab->comment.hbox), 
e->cur_tab->comment.revert, ETK_FALSE, ETK_FALSE, 0);
+   etk_box_pack_start(ETK_BOX(e->cur_tab->comment.hbox), 
e->cur_tab->comment.save, ETK_FALSE, ETK_FALSE, 0);
    
-   e->comment.visible = ETK_TRUE;
+   e->cur_tab->comment.visible = ETK_TRUE;
    
    etk_widget_show_all(ETK_WIDGET(e->hpaned));   
 }
@@ -42,19 +54,35 @@
 void
 _ex_comment_hide(Exhibit *e)
 {
-   if(!e->comment.visible)
+   if(!e->cur_tab->comment.visible)
      return;
+
+   if(evas_list_count(e->tabs) == 1)
+     {
+       /* we only have 1 tab, ie, no notebook */
+       if (e->cur_tab->fit_window)
+         etk_paned_child2_set(ETK_PANED(e->hpaned), e->cur_tab->alignment, 
ETK_TRUE);
+       else
+         etk_paned_child2_set(ETK_PANED(e->hpaned), e->cur_tab->scrolled_view, 
ETK_TRUE);
+     }
+   else
+     {
+       /* we have multiple tabs */
+       if (e->cur_tab->fit_window)
+         etk_notebook_page_child_set(ETK_NOTEBOOK(e->notebook), 
e->cur_tab->num, e->cur_tab->alignment);
+       else
+         etk_notebook_page_child_set(ETK_NOTEBOOK(e->notebook), 
e->cur_tab->num, e->cur_tab->scrolled_view);
+     }   
+   
+   etk_object_destroy(ETK_OBJECT(e->cur_tab->comment.entry));
+   etk_object_destroy(ETK_OBJECT(e->cur_tab->comment.save));
+   etk_object_destroy(ETK_OBJECT(e->cur_tab->comment.revert));
+   etk_object_destroy(ETK_OBJECT(e->cur_tab->comment.hbox));
+   etk_object_destroy(ETK_OBJECT(e->cur_tab->comment.vbox2));
+   etk_object_destroy(ETK_OBJECT(e->cur_tab->comment.frame));
+   etk_object_destroy(ETK_OBJECT(e->cur_tab->comment.vbox));   
    
-   etk_paned_child2_set(ETK_PANED(e->hpaned), e->cur_tab->scrolled_view, 
ETK_TRUE);
-   etk_object_destroy(ETK_OBJECT(e->comment.entry));
-   etk_object_destroy(ETK_OBJECT(e->comment.save));
-   etk_object_destroy(ETK_OBJECT(e->comment.revert));
-   etk_object_destroy(ETK_OBJECT(e->comment.hbox));
-   etk_object_destroy(ETK_OBJECT(e->comment.vbox2));
-   etk_object_destroy(ETK_OBJECT(e->comment.frame));
-   etk_object_destroy(ETK_OBJECT(e->comment.vbox));   
-   
-   e->comment.visible = ETK_FALSE;
+   e->cur_tab->comment.visible = ETK_FALSE;
 }
 
 void
@@ -68,10 +96,10 @@
    if (_ex_file_is_jpg(file))
      if (_ex_comment_jpeg_read(file, &comment, &len) && (len != 0))
        {
-         etk_entry_text_set(ETK_ENTRY(e->comment.entry), comment);
+         etk_entry_text_set(ETK_ENTRY(e->cur_tab->comment.entry), comment);
          return;
        }
-   etk_entry_text_set(ETK_ENTRY(e->comment.entry), "");
+   etk_entry_text_set(ETK_ENTRY(e->cur_tab->comment.entry), "");
 }
 
 void
@@ -82,7 +110,7 @@
    int len;
    
    file = ((Ex_Tab *) e->cur_tab)->cur_file;
-   comment = etk_entry_text_get(ETK_ENTRY(e->comment.entry));
+   comment = etk_entry_text_get(ETK_ENTRY(e->cur_tab->comment.entry));
    if (comment)
      len = strlen(comment);
    else
===================================================================
RCS file: /cvs/e/e17/proto/exhibit/src/bin/exhibit_main.c,v
retrieving revision 1.50
retrieving revision 1.51
diff -u -3 -r1.50 -r1.51
--- exhibit_main.c      28 Jul 2006 16:04:47 -0000      1.50
+++ exhibit_main.c      29 Jul 2006 02:49:24 -0000      1.51
@@ -41,16 +41,38 @@
 void
 _ex_main_button_zoom_in_cb(Etk_Object *obj, void *data)
 {
-   Exhibit      *e;
+   Exhibit *e;
    
    e = data;
    
    if (e->cur_tab->fit_window)
      {
        if(evas_list_count(e->tabs) == 1)
-         etk_paned_child2_set(ETK_PANED(e->hpaned), e->cur_tab->scrolled_view, 
ETK_TRUE);
+         {
+            if(e->cur_tab->comment.visible)
+              {
+                 etk_paned_child2_set(ETK_PANED(e->hpaned), 
e->cur_tab->comment.vbox, ETK_TRUE);
+                 etk_container_remove(ETK_CONTAINER(e->cur_tab->comment.vbox), 
e->cur_tab->alignment);
+                 etk_container_remove(ETK_CONTAINER(e->cur_tab->comment.vbox), 
e->cur_tab->comment.frame);
+                 etk_box_pack_start(ETK_BOX(e->cur_tab->comment.vbox), 
e->cur_tab->scrolled_view, ETK_TRUE, ETK_TRUE, 0);
+                 etk_box_pack_start(ETK_BOX(e->cur_tab->comment.vbox), 
e->cur_tab->comment.frame, ETK_FALSE, ETK_FALSE, 3);
+              }
+            else
+              etk_paned_child2_set(ETK_PANED(e->hpaned), 
e->cur_tab->scrolled_view, ETK_TRUE);
+         }
        else
-         etk_notebook_page_child_set(ETK_NOTEBOOK(e->notebook), 
e->cur_tab->num, e->cur_tab->scrolled_view);
+         {
+            if(e->cur_tab->comment.visible)
+              {
+                 etk_notebook_page_child_set(ETK_NOTEBOOK(e->notebook), 
e->cur_tab->num, e->cur_tab->comment.vbox);
+                 etk_container_remove(ETK_CONTAINER(e->cur_tab->comment.vbox), 
e->cur_tab->alignment);
+                 etk_container_remove(ETK_CONTAINER(e->cur_tab->comment.vbox), 
e->cur_tab->comment.frame);
+                 etk_box_pack_start(ETK_BOX(e->cur_tab->comment.vbox), 
e->cur_tab->scrolled_view, ETK_TRUE, ETK_TRUE, 0);
+                 etk_box_pack_start(ETK_BOX(e->cur_tab->comment.vbox), 
e->cur_tab->comment.frame, ETK_FALSE, ETK_FALSE, 3);
+              }
+            else
+              etk_notebook_page_child_set(ETK_NOTEBOOK(e->notebook), 
e->cur_tab->num, e->cur_tab->scrolled_view);
+         }
          
         etk_widget_size_request_set(e->cur_tab->alignment, -1, -1);
         
etk_scrolled_view_add_with_viewport(ETK_SCROLLED_VIEW(e->cur_tab->scrolled_view),
 e->cur_tab->alignment);
@@ -69,16 +91,38 @@
 void
 _ex_main_button_zoom_out_cb(Etk_Object *obj, void *data)
 {
-   Exhibit      *e;
+   Exhibit *e;
    
    e = data;
    
    if (e->cur_tab->fit_window)
      {
-       if(evas_list_count(e->tabs) == 1)         
-         etk_paned_child2_set(ETK_PANED(e->hpaned), e->cur_tab->scrolled_view, 
ETK_TRUE);
+       if(evas_list_count(e->tabs) == 1)
+         {
+            if(e->cur_tab->comment.visible)
+              {
+                 etk_paned_child2_set(ETK_PANED(e->hpaned), 
e->cur_tab->comment.vbox, ETK_TRUE);
+                 etk_container_remove(ETK_CONTAINER(e->cur_tab->comment.vbox), 
e->cur_tab->alignment);
+                 etk_container_remove(ETK_CONTAINER(e->cur_tab->comment.vbox), 
e->cur_tab->comment.frame);
+                 etk_box_pack_start(ETK_BOX(e->cur_tab->comment.vbox), 
e->cur_tab->scrolled_view, ETK_TRUE, ETK_TRUE, 0);
+                 etk_box_pack_start(ETK_BOX(e->cur_tab->comment.vbox), 
e->cur_tab->comment.frame, ETK_FALSE, ETK_FALSE, 3);
+              }
+            else
+              etk_paned_child2_set(ETK_PANED(e->hpaned), 
e->cur_tab->scrolled_view, ETK_TRUE);
+         }
        else
-         etk_notebook_page_child_set(ETK_NOTEBOOK(e->notebook), 
e->cur_tab->num, e->cur_tab->scrolled_view);
+         {
+            if(e->cur_tab->comment.visible)
+              {
+                 etk_notebook_page_child_set(ETK_NOTEBOOK(e->notebook), 
e->cur_tab->num, e->cur_tab->comment.vbox);
+                 etk_container_remove(ETK_CONTAINER(e->cur_tab->comment.vbox), 
e->cur_tab->alignment);
+                 etk_container_remove(ETK_CONTAINER(e->cur_tab->comment.vbox), 
e->cur_tab->comment.frame);
+                 etk_box_pack_start(ETK_BOX(e->cur_tab->comment.vbox), 
e->cur_tab->scrolled_view, ETK_TRUE, ETK_TRUE, 0);
+                 etk_box_pack_start(ETK_BOX(e->cur_tab->comment.vbox), 
e->cur_tab->comment.frame, ETK_FALSE, ETK_FALSE, 3);
+              }
+            else            
+              etk_notebook_page_child_set(ETK_NOTEBOOK(e->notebook), 
e->cur_tab->num, e->cur_tab->scrolled_view);
+         }
        
         etk_widget_size_request_set(e->cur_tab->alignment, -1, -1);
         
etk_scrolled_view_add_with_viewport(ETK_SCROLLED_VIEW(e->cur_tab->scrolled_view),
 e->cur_tab->alignment);
@@ -104,9 +148,31 @@
    if (e->cur_tab->fit_window)
      {
        if(evas_list_count(e->tabs) == 1)
-         etk_paned_child2_set(ETK_PANED(e->hpaned), e->cur_tab->scrolled_view, 
ETK_TRUE);
+         {
+            if(e->cur_tab->comment.visible)
+              {
+                 etk_paned_child2_set(ETK_PANED(e->hpaned), 
e->cur_tab->comment.vbox, ETK_TRUE);
+                 etk_container_remove(ETK_CONTAINER(e->cur_tab->comment.vbox), 
e->cur_tab->alignment);
+                 etk_container_remove(ETK_CONTAINER(e->cur_tab->comment.vbox), 
e->cur_tab->comment.frame);          
+                 etk_box_pack_start(ETK_BOX(e->cur_tab->comment.vbox), 
e->cur_tab->scrolled_view, ETK_TRUE, ETK_TRUE, 0);
+                 etk_box_pack_start(ETK_BOX(e->cur_tab->comment.vbox), 
e->cur_tab->comment.frame, ETK_FALSE, ETK_FALSE, 3);
+              }
+            else                    
+              etk_paned_child2_set(ETK_PANED(e->hpaned), 
e->cur_tab->scrolled_view, ETK_TRUE);
+         }
        else
-         etk_notebook_page_child_set(ETK_NOTEBOOK(e->notebook), 
e->cur_tab->num, e->cur_tab->scrolled_view);
+         {
+            if(e->cur_tab->comment.visible)
+              {
+                 etk_notebook_page_child_set(ETK_NOTEBOOK(e->notebook), 
e->cur_tab->num, e->cur_tab->comment.vbox);
+                 etk_container_remove(ETK_CONTAINER(e->cur_tab->comment.vbox), 
e->cur_tab->alignment);
+                 etk_container_remove(ETK_CONTAINER(e->cur_tab->comment.vbox), 
e->cur_tab->comment.frame);
+                 etk_box_pack_start(ETK_BOX(e->cur_tab->comment.vbox), 
e->cur_tab->scrolled_view, ETK_TRUE, ETK_TRUE, 0);
+                 etk_box_pack_start(ETK_BOX(e->cur_tab->comment.vbox), 
e->cur_tab->comment.frame, ETK_FALSE, ETK_FALSE, 3);
+              }
+            else            
+              etk_notebook_page_child_set(ETK_NOTEBOOK(e->notebook), 
e->cur_tab->num, e->cur_tab->scrolled_view);
+         }
        
         etk_widget_size_request_set(e->cur_tab->alignment, -1, -1);
         
etk_scrolled_view_add_with_viewport(ETK_SCROLLED_VIEW(e->cur_tab->scrolled_view),
 e->cur_tab->alignment);
@@ -133,9 +199,31 @@
    
    etk_widget_size_request_set(e->cur_tab->alignment, 10, 10);
    if(evas_list_count(e->tabs) == 1)
-     etk_paned_child2_set(ETK_PANED(e->hpaned), e->cur_tab->alignment, 
ETK_TRUE);
-   else          
-     etk_notebook_page_child_set(ETK_NOTEBOOK(e->notebook), e->cur_tab->num, 
e->cur_tab->alignment);
+     {
+       if(e->cur_tab->comment.visible)
+         {
+            etk_paned_child2_set(ETK_PANED(e->hpaned), 
e->cur_tab->comment.vbox, ETK_TRUE);
+            etk_container_remove(ETK_CONTAINER(e->cur_tab->comment.vbox), 
e->cur_tab->scrolled_view);
+            etk_container_remove(ETK_CONTAINER(e->cur_tab->comment.vbox), 
e->cur_tab->comment.frame);       
+            etk_box_pack_start(ETK_BOX(e->cur_tab->comment.vbox), 
e->cur_tab->alignment, ETK_TRUE, ETK_TRUE, 0);            
+            etk_box_pack_start(ETK_BOX(e->cur_tab->comment.vbox), 
e->cur_tab->comment.frame, ETK_FALSE, ETK_FALSE, 3);
+         }
+       else
+         etk_paned_child2_set(ETK_PANED(e->hpaned), e->cur_tab->alignment, 
ETK_TRUE);
+     }
+   else
+     {
+       if(e->cur_tab->comment.visible)
+         {
+            etk_notebook_page_child_set(ETK_NOTEBOOK(e->notebook), 
e->cur_tab->num, e->cur_tab->comment.vbox);
+            etk_container_remove(ETK_CONTAINER(e->cur_tab->comment.vbox), 
e->cur_tab->scrolled_view);
+            etk_container_remove(ETK_CONTAINER(e->cur_tab->comment.vbox), 
e->cur_tab->comment.frame);
+            etk_box_pack_start(ETK_BOX(e->cur_tab->comment.vbox), 
e->cur_tab->alignment, ETK_TRUE, ETK_TRUE, 0);
+            etk_box_pack_start(ETK_BOX(e->cur_tab->comment.vbox), 
e->cur_tab->comment.frame, ETK_FALSE, ETK_FALSE, 3);
+         }
+       else    
+         etk_notebook_page_child_set(ETK_NOTEBOOK(e->notebook), 
e->cur_tab->num, e->cur_tab->alignment);
+     }
      
    etk_widget_size_request_set(e->cur_tab->image, -1, -1);
    
@@ -757,7 +845,6 @@
    e->tabs = NULL;
    e->slideshow.active = ETK_FALSE;
    e->slideshow.interval = 5.0;
-   e->comment.visible = ETK_FALSE;
    
    e->options = _ex_options_new();
    /* Now load saved options */
===================================================================
RCS file: /cvs/e/e17/proto/exhibit/src/bin/exhibit_menus.c,v
retrieving revision 1.19
retrieving revision 1.20
diff -u -3 -r1.19 -r1.20
--- exhibit_menus.c     28 Jul 2006 16:04:47 -0000      1.19
+++ exhibit_menus.c     29 Jul 2006 02:49:24 -0000      1.20
@@ -360,7 +360,7 @@
    
    etk_tree_row_fields_get(r, 
etk_tree_nth_col_get(ETK_TREE(e->cur_tab->itree), 0), NULL, &icol_string, 
etk_tree_nth_col_get(ETK_TREE(e->cur_tab->itree), 1),NULL);   
    
-   if(!e->comment.visible)
+   if(!e->cur_tab->comment.visible)
      {
        _ex_comment_show(e);
        _ex_comment_load(e);      
===================================================================
RCS file: /cvs/e/e17/proto/exhibit/src/bin/exhibit_tab.c,v
retrieving revision 1.13
retrieving revision 1.14
diff -u -3 -r1.13 -r1.14
--- exhibit_tab.c       28 Jul 2006 16:04:47 -0000      1.13
+++ exhibit_tab.c       29 Jul 2006 02:49:24 -0000      1.14
@@ -19,6 +19,7 @@
    tab->images = NULL;
    tab->e = e;
    tab->fit_window = ETK_FALSE;
+   tab->comment.visible = ETK_FALSE;
    
    tab->dtree = etk_tree_new();
    etk_widget_size_request_set(tab->dtree, 180, 120);



-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
enlightenment-cvs mailing list
enlightenment-cvs@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-cvs

Reply via email to