"& ui_prefs.fullscreen" was redundant, state brings "ui_prefs.fullscreen & interface_prefs.hide_menubar_on_fullscreen" from caller. Now with tabs..I think it's ok!<br><br><div class="gmail_quote">
On Sat, Oct 22, 2011 at 6:03 PM, Dimitar Zhekov <span dir="ltr"><<a href="mailto:dimitar.zhekov@gmail.com">dimitar.zhekov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">On Thu, 20 Oct 2011 12:43:15 +0100<br>
Pedro Cirne <<a href="mailto:pedro@cirne100.eu">pedro@cirne100.eu</a>> wrote:<br>
<br>
> Here it is...I hope it's useful!<br>
><br>
> On Thu, Oct 20, 2011 at 1:55 AM, Lex Trotman <<a href="mailto:elextr@gmail.com">elextr@gmail.com</a>> wrote:<br>
><br>
> > On 20 October 2011 11:39, Pedro Cirne <<a href="mailto:pedro@cirne100.eu">pedro@cirne100.eu</a>> wrote:<br>
> > > I have edited the source code to make it works for me, is it useful for<br>
> > the<br>
> > > project?<br>
> ><br>
> > If you submit a patch/pull request we will certainly have a look at<br>
> > it.  Then you may not have to keep applying it each time Geany<br>
> > upgrades.<br>
<br>
</div>Looks fine to me, except that some of the changes are indented with<br>
spaces instead of tabs.<br>
<br>
I'm not quite sure why ui_menubar_showhide() should take a gboolean<br>
state, check it with & and hide the menu for state = 1 (or TRUE?). It<br>
may be the most effective way, but writing an extra line or too to make<br>
the code stupid and simple may be a good idea.<br>
<br>
A good thing to have, though I personnaly am not going to use it.<br>
<font color="#888888"><br>
--<br>
E-gards: Jimmy<br>
</font><div><div></div><div class="h5">_______________________________________________<br>
Geany mailing list<br>
<a href="mailto:Geany@uvena.de">Geany@uvena.de</a><br>
<a href="https://lists.uvena.de/cgi-bin/mailman/listinfo/geany" target="_blank">https://lists.uvena.de/cgi-bin/mailman/listinfo/geany</a><br>
</div></div></blockquote></div><br>