Am Fri, 06 Apr 2007 13:22:21 +0300
schrieb Виктор Кожухаров <[EMAIL PROTECTED]>:
> В чт, 2007-04-05 в 21:09 +0200, Hannes Janetzek написа:
> > Hi,
> > I made a patch so that hiding of the shelf is done by the shelf and
> > not by the theme. I think this will make it easier to have a
> > consistent and configurable shelf-hiding behaviour.
> > I added one theme-data item to the shelf group: "hidden_state_size"
> > which sets the height or width for the shelf when it is in hidden
> > state.
> >
> The only problem with this patch that i can see is, that it should
> work regardless of whether the theme has a "hidden_state_size" set.
> Currently, if a theme does not have that, things will mess up, bad.
> And it might be useful to increase the default hidden_state_size from
> 2 to 6, so as to minimize the chance of rolling over to another
> virtual desktop.
>
> Other than that, the patch is dandy, and if no one objects, I'll
> commit it.
Thanks.
I think the changes you have made are reasonable but one thing. The
shelf also disappears if i move the mouse out (the hide_timer starts)
and then move the mouse back in(while it is hiding or the hide_timer
is on). The attached patch should fix this.
> >
> > Regards,
> > Hannes
> >
> >
> >
> > -------------------------------------------------------------------------
> > 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-devel
> > mailing list enlightenment-devel@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
Index: e_shelf.c
===================================================================
RCS file: /var/cvs/e/e17/apps/e/src/bin/e_shelf.c,v
retrieving revision 1.54
diff -u -r1.54 e_shelf.c
--- e_shelf.c 8 Apr 2007 09:32:48 -0000 1.54
+++ e_shelf.c 8 Apr 2007 19:02:49 -0000
@@ -291,7 +291,8 @@
}
else if (!show && es->cfg->autohide && !es->hidden)
{
- if(!es->hide_timer)
+ es->hidden = 1;
+ if(!es->hide_timer)
es->hide_timer = ecore_timer_add(1.0, _e_shelf_cb_hide_timer, es);
}
}
@@ -1098,24 +1099,26 @@
es = data;
- es->hidden = 1;
- edje_object_signal_emit(es->o_base, "e,state,hidden", "e");
- if (es->instant_delay >= 0.0)
+ if(es->hidden)
{
- if (!es->instant_timer)
- es->instant_timer = ecore_timer_add(es->instant_delay, _e_shelf_cb_instant_hide_timer, es);
+
+ edje_object_signal_emit(es->o_base, "e,state,hidden", "e");
+ if (es->instant_delay >= 0.0)
+ {
+ if (!es->instant_timer)
+ es->instant_timer = ecore_timer_add(es->instant_delay, _e_shelf_cb_instant_hide_timer, es);
+ }
+ else
+ {
+ if(!es->hide_animator)
+ es->hide_animator = ecore_animator_add(_e_shelf_cb_hide_animator, es);
+ }
+ if (es->hide_timer)
+ {
+ ecore_timer_del(es->hide_timer);
+ es->hide_timer = NULL;
+ }
}
- else
- {
- if(!es->hide_animator)
- es->hide_animator = ecore_animator_add(_e_shelf_cb_hide_animator, es);
- }
- if (es->hide_timer)
- {
- ecore_timer_del(es->hide_timer);
- es->hide_timer = NULL;
- }
-
return 1;
}
-------------------------------------------------------------------------
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-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
