On Fri, Dec 20, 2019 at 2:24 PM Fred Kiefer <fredkie...@gmx.de> wrote:
> > > > Am 20.12.2019 um 11:34 schrieb Sergii Stoian <stoyan...@gmail.com>: > > > > I've moved discussion to mail list as Fred asked me. > > > > On Fri, Dec 20, 2019 at 12:18 PM Fred Kiefer <notificati...@github.com> > wrote: > > Some of the changes of this pull request are obviously correct. What I > doubt is the overall change. Why would we want to display the title item > (that is the first item in a pull down) in the menu? > > This is the main difference between pull down and pop up and it is well > documented. > > > > Do you mean GNUstep documentation or Apple? > > Apple’s documentation. According to this, the first item of a pull down is > the title and not displayed when showing the menu. This is what GNUstep has > implemented up to now. > Give me a link please to that documentation. Here https://developer.apple.com/documentation/appkit/nspopupbutton/1532070-pullsdown?language=objc I've git this: "This property does not affect the contents of the menu; it affects only the style of the menu." > > I just tired this on Cocoa and the difference is still there. > > > > What particular difference did you observe from practical point of view? > What consequences do you expect? > > As I unterstand it you want to display the title as the first item in the > menu, but as a disabled one. This will look different than it did before. Fred, have you tried libs-gui with my change? The only difference is correct drawing of left and top border of button while mouse is down. On the contrary with my change popup button menu looks the same no matter what style was selected pulldown or normal. > Having this as a theme or with a user setable option would be fine for me. > But changing the default here is something that needs to be discussed more > widely. > > > Of course changing this would simplify the code here a lot and I would > love to see that, but this changes the whole appearance of pull down items > and this needs to be decided on a completely different level. > > > > So we're here. > > Any thoughts, guys? > > No, I personally don’t have much of a position on this point. Why do you > want to change the behaviour? I'll describe reasons and intentions in PR: https://github.com/gnustep/libs-gui/pull/43. -- Sergii Stoian
