Re: NSToolbar and SearchView

2010-04-29 Thread Fred Kiefer
Am 29.04.2010 00:39, schrieb Riccardo Mottola:
 using Grr, I notice that the search component is again displayed wrong.
 I had fixed that (other issues remained, but they were to be considered
 minor and Fred knows that we have them on the table). Did someone revert
 or work around my fix?

It was most likely Dougs commit 30143 that broke things for you. But as
you don't describe in detail what is wrong, it is hard to tell. And you
must be aware of that specific fix as you corrected some C99 issues you
had with it.
The only change you made to toolbars that I am aware of was the
_isFlexibleSpace method (By the way, your test is unneeded, the method
should just return NO to get the behaviour you want. This method is
overriden in the subclass). And nothing in that area has changed.
On the other hand all the layout code there is just a hack, I am not
surprised to see it broken by unrelated changes. We should rewrite the
whole area, if ever somebody has the time for that...

Fred


___
Gnustep-dev mailing list
Gnustep-dev@gnu.org
http://lists.gnu.org/mailman/listinfo/gnustep-dev


Re: NSToolbar and SearchView

2010-04-29 Thread Riccardo Mottola

Hi,

It was most likely Dougs commit 30143 that broke things for you. But as
you don't describe in detail what is wrong, it is hard to tell. And you
must be aware of that specific fix as you corrected some C99 issues you
had with it.
   
The search panels comes up way too small. I haven't checked precisely, 
but I believe it just displays at its minimum size.
I admit I dod not analyze Doug's change, I just mechanically fixed 
compiler issues.

The only change you made to toolbars that I am aware of was the
_isFlexibleSpace method (By the way, your test is unneeded, the method
should just return NO to get the behaviour you want. This method is
overriden in the subclass). And nothing in that area has changed.
On the other hand all the layout code there is just a hack, I am not
surprised to see it broken by unrelated changes. We should rewrite the
whole area, if ever somebody has the time for that...

   
Yes, the code is messy and complicated, but I believe after my change 
reasonably correct, even if limited, since it doesn't take flexible 
space and resizing well in account as you well know.

Understandable that some change broke the code unintended.

Riccardo


___
Gnustep-dev mailing list
Gnustep-dev@gnu.org
http://lists.gnu.org/mailman/listinfo/gnustep-dev


NSToolbar and SearchView

2010-04-28 Thread Riccardo Mottola

Hi,

using Grr, I notice that the search component is again displayed wrong. 
I had fixed that (other issues remained, but they were to be considered 
minor and Fred knows that we have them on the table). Did someone revert 
or work around my fix?


Riccardo


___
Gnustep-dev mailing list
Gnustep-dev@gnu.org
http://lists.gnu.org/mailman/listinfo/gnustep-dev