Re: [PATCH]Replace Combo with ComboBox in GLayoutBox

2004-10-01 Thread Lars Gullik Bjønnes
John Spray <[EMAIL PROTECTED]> writes: | On Tue, 2004-09-28 at 23:36, Lars Gullik Bjønnes wrote: >> Hmm... did you compile this or not? | It was weird, actually, I could have sworn that I did compile it, but I | obviously didn't. Anyway, I've been using this for a day now, and I'm | pretty sure i

Re: [PATCH]Replace Combo with ComboBox in GLayoutBox

2004-09-29 Thread Lars Gullik Bjønnes
John Spray <[EMAIL PROTECTED]> writes: | On Tue, 2004-09-28 at 23:36, Lars Gullik Bjønnes wrote: >> Hmm... did you compile this or not? | It was weird, actually, I could have sworn that I did compile it, but I | obviously didn't. Anyway, I've been using this for a day now, and I'm | pretty sure i

Re: [PATCH]Replace Combo with ComboBox in GLayoutBox

2004-09-29 Thread John Spray
On Tue, 2004-09-28 at 23:36, Lars Gullik Bjønnes wrote: > Hmm... did you compile this or not? It was weird, actually, I could have sworn that I did compile it, but I obviously didn't. Anyway, I've been using this for a day now, and I'm pretty sure it's working okay. Attached version has a little

Re: [PATCH]Replace Combo with ComboBox in GLayoutBox

2004-09-28 Thread Lars Gullik Bjønnes
John Spray <[EMAIL PROTECTED]> writes: | On Tue, 2004-09-28 at 16:57, John Spray wrote: >> Revised patch attached. > | Oops, note that there's a "Manage" in there that needs to be a | "manage". Sorry. Hmm... did you compile this or not? PS. that took me 5 minutes to write... been at the pub

Re: [PATCH]Replace Combo with ComboBox in GLayoutBox

2004-09-28 Thread John Spray
On Tue, 2004-09-28 at 16:57, John Spray wrote: > Revised patch attached. Oops, note that there's a "Manage" in there that needs to be a "manage". Sorry.

Re: [PATCH]Replace Combo with ComboBox in GLayoutBox

2004-09-28 Thread John Spray
On Tue, 2004-09-28 at 08:06, Lars Gullik Bjønnes wrote: > | + Gtk::CellRendererText * cell = new Gtk::CellRendererText; > > Are you sure that no Gtk::manage is needed here? No, no I'm not :-) > Does this have to be a raw pointer? Who manages it? Let's just make it a non-pointer member of the GL

Re: [PATCH]Replace Combo with ComboBox in GLayoutBox

2004-09-28 Thread Alfredo Braunstein
Lars Gullik BjÃnnes wrote: > John Spray <[EMAIL PROTECTED]> writes: > > | Hi, >> > | Due to an unexpected bout of insomnia, > > You can sleep when you get old. And the good news is, if you don't sleep you'll get there sooner! ;-) Alfredo

Re: [PATCH]Replace Combo with ComboBox in GLayoutBox

2004-09-28 Thread Lars Gullik Bjønnes
John Spray <[EMAIL PROTECTED]> writes: | Hi, > | Due to an unexpected bout of insomnia, You can sleep when you get old. | I've replaced the deprecated | Combo with a ComboBox in GToolbar.[Ch]. nice. | Index: GToolbar.C | === | RCS

[PATCH]Replace Combo with ComboBox in GLayoutBox

2004-09-27 Thread John Spray
Hi, Due to an unexpected bout of insomnia, I've replaced the deprecated Combo with a ComboBox in GToolbar.[Ch]. John Index: ChangeLog === RCS file: /cvs/lyx/lyx-devel/src/frontends/gtk/ChangeLog,v retrieving revision 1.62 diff -u -3