Re: UI Suggestions

2012-09-20 Thread Liviu Andronic
Hello On Thu, Sep 20, 2012 at 7:02 AM, PJ Singh pjsingh5...@gmail.com wrote: Here are a few suggestions which I think will help give Lyx a more polished, refined appearance... Some of these suggestions seem to be related to: http://www.lyx.org/trac/ticket/5636 Liviu 1. Allow

Re: Problem with command-sequence

2012-09-20 Thread Jean-Marc Lasgouttes
Le 20/09/12 01:36, Lars Gullik Bjønnes a écrit : Right... but why now have LFUNS de async by default (when done as a script thingie), and a separate LFUN to make the async if wanted. Gui/Menu run LFUNS could still be async. Probably also part of that discussion which I completely missed. I do

Re: UI Suggestions

2012-09-20 Thread Jean-Marc Lasgouttes
Le 20/09/12 07:02, PJ Singh a écrit : 1. Allow configurable gradient backgrounds for menu toolbars. The gradient color information should be specified as parameters in the toolbar *.inc files under .../lyx/ui This is Qt business, in relation with OS theme support. 2.

Re: [LyX master] Calibrate log file parser

2012-09-20 Thread Jürgen Spitzmüller
2012/9/20 Lars Gullik Bjønnes lar...@gullik.org: 'closing' - why not const ref? Are there contex issues here, that makes creating a copy required? No. I'll change it. Thanks. Jürgen -- Lgb

Literate programming on LyX-2.0.4: no build button

2012-09-20 Thread Kayvan Sylvan
Hi guys, I'm back to playing with LyX and noweb and I installed the latest LyX on my Fedora 17 system. It works beautifully! Thank you all for the great work. I pulled up one of my files but I can't find the Build button or menu item. I can export to noweb and run build-script myself, but it

Re: Literate programming on LyX-2.0.4: no build button

2012-09-20 Thread Kayvan Sylvan
Kayvan Sylvan kayvansylvan at gmail.com writes: Hi guys, I'm back to playing with LyX and noweb and I installed the latest LyX on my Fedora 17 system. It works beautifully! Thank you all for the great work. I pulled up one of my files but I can't find the Build button or menu item. I can

[patch] LyX scrolling issue

2012-09-20 Thread Scott Kostyshak
Some have reported problems with LyX scrolling on Windows and Mac. Robert Schulze has git-bisected his way to the root cause of the particular scrolling problem he is experiencing: http://www.lyx.org/trac/ticket/8135 I am wondering if his problem is the same as others. The patch for current branch

[patch] fix warning on branch

2012-09-20 Thread Scott Kostyshak
This patch for branch fixes a warning that was fixed on master here: ea50cd71f9d Is it OK? Scott diff --git a/src/frontends/qt4/GuiSpellchecker.cpp b/src/frontends/qt4/GuiSpellchecker.cpp index 5dbdbbb..c428807 100644 --- a/src/frontends/qt4/GuiSpellchecker.cpp +++

Re: [patch] fix warning on branch

2012-09-20 Thread Scott Kostyshak
This patch for branch fixes a warning that was fixed on master here: ea50cd71f9d Is it OK? Scott This updated patch fixes another warning on branch, which was also fixed in master at that commit. Scott diff --git a/src/Lexer.cpp b/src/Lexer.cpp index d824da0..664e1b9 100644 ---

Re: [patch] fix warning on branch

2012-09-20 Thread Scott Kostyshak
On Thu, Sep 20, 2012 at 4:05 PM, Scott Kostyshak skost...@lyx.org wrote: This patch for branch fixes a warning that was fixed on master here: ea50cd71f9d Is it OK? Scott This updated patch fixes another warning on branch, which was also fixed in master at that commit. Scott Sorry for

PATCH: Changes to make LyX work on Mountain Lion

2012-09-20 Thread Stephan Witt
Hi, I've tried to build LyX on Mac OS X 10.8 with Xcode 4.3.3 + Qt 4.8.3 cocoa. There are some changes here and there required to make it work. I've attached them as one patch for review. I had to change: * the linker flags for frameworks * get rid of carbon calls deprecated since 10.8 * correct

Re: [patch] LyX bash completion

2012-09-20 Thread Pavel Sanda
Scott Kostyshak wrote: If the hardcoding is desired, then I will manually make the updates: I think hardcoding is fine here. Pavel

Re: [patch] LyX bash completion

2012-09-20 Thread Scott Kostyshak
On Thu, Sep 20, 2012 at 4:44 PM, Pavel Sanda sa...@lyx.org wrote: Scott Kostyshak wrote: If the hardcoding is desired, then I will manually make the updates: I think hardcoding is fine here. Pavel OK, updated patch attached. Thanks, Scott diff --git a/lib/scripts/bash_completion

Re: PATCH: Changes to make LyX work on Mountain Lion

2012-09-20 Thread Jean-Marc Lasgouttes
Le 20/09/12 22:40, Stephan Witt a écrit : * get rid of carbon calls deprecated since 10.8 For getting special paths, Package.cpp, what about using Qt functions directly? For ex. http://qt-project.org/forums/viewthread/10550 JMarc

Re: [patch] fix warning on branch

2012-09-20 Thread Lars Gullik Bjønnes
Scott Kostyshak skost...@lyx.org writes: | On Thu, Sep 20, 2012 at 4:05 PM, Scott Kostyshak skost...@lyx.org wrote: This patch for branch fixes a warning that was fixed on master here: ea50cd71f9d Is it OK? Scott This updated patch fixes another warning on branch, which was also fixed in

Re: Literate programming on LyX-2.0.4: no build button

2012-09-20 Thread Kayvan Sylvan
Kayvan Sylvan kayvansylvan at gmail.com writes: I found out that I can do: File - Export - More Formats and Options - Program But this is obviously more cumbersome than clicking a Build button. How can I get this functionality back? ---Kayvan After some more looking at the

Re: [patch] LyX bash completion

2012-09-20 Thread Richard Heck
On 09/20/2012 05:00 PM, Scott Kostyshak wrote: On Thu, Sep 20, 2012 at 4:44 PM, Pavel Sanda sa...@lyx.org wrote: Scott Kostyshak wrote: If the hardcoding is desired, then I will manually make the updates: I think hardcoding is fine here. Pavel OK, updated patch attached. Feel free to commit

Re: PATCH: Changes to make LyX work on Mountain Lion

2012-09-20 Thread Richard Heck
On 09/20/2012 04:40 PM, Stephan Witt wrote: Hi, I've tried to build LyX on Mac OS X 10.8 with Xcode 4.3.3 + Qt 4.8.3 cocoa. There are some changes here and there required to make it work. I've attached them as one patch for review. I had to change: * the linker flags for frameworks * get rid

Re: [patch] fix warning on branch

2012-09-20 Thread Richard Heck
All this looks fine to me once Lars's comments are handled. rh On 09/20/2012 05:49 PM, Lars Gullik Bjønnes wrote: Scott Kostyshak skost...@lyx.org writes: | On Thu, Sep 20, 2012 at 4:05 PM, Scott Kostyshak skost...@lyx.org wrote: This patch for branch fixes a warning that was fixed on

Re: [patch] LyX scrolling issue

2012-09-20 Thread Pavel Sanda
Scott Kostyshak wrote: void GuiWorkArea::update(int x, int y, int w, int h) { - viewport()-update(x, y, w, h); + viewport()-repaint(x, y, w, h); IIRC it was Andre's claim that we should use update instead of repaint. Maybe he would know why this slow downs LyX so much. Pavel

Re: [LyX master] Support for the Minion Pro LaTeX font

2012-09-20 Thread Richard Heck
On 09/19/2012 11:47 AM, Juergen Spitzmueller wrote: The branch, master, has been updated. - Log - commit e9ca6b6bf48b23559384e1108e01df68a9ee18b6 Author: Juergen Spitzmueller sp...@lyx.org Date: Wed Sep 19 17:46:55 2012 +0200

Re: Templates g-brief, g-brief2

2012-09-20 Thread Max Funk
My previous post did apparently not arrive. I repeat it with only the patch file attached. This is a patch of an issue with the g-brief-de.lyx, g-brief-en.lyx and g-brief2.lyx templates. I fixed the following Problems: - The g-brief templates require latex package europs. However, since

Re: [patch] fix warning on branch

2012-09-20 Thread Scott Kostyshak
On Thu, Sep 20, 2012 at 5:49 PM, Lars Gullik Bjønnes lar...@gullik.org wrote: btw. std::string::c_str() should not be used here, use std::string::data() instead. Lars, can you confirm that I made the right corrections? I changed c_str() to data() in two places. If they are right I will also

Re: [patch] LyX bash completion

2012-09-20 Thread Scott Kostyshak
On Thu, Sep 20, 2012 at 6:14 PM, Richard Heck rgh...@lyx.org wrote: On 09/20/2012 05:00 PM, Scott Kostyshak wrote: On Thu, Sep 20, 2012 at 4:44 PM, Pavel Sanda sa...@lyx.org wrote: Scott Kostyshak wrote: If the hardcoding is desired, then I will manually make the updates: I think

Re: UI Suggestions

2012-09-20 Thread Liviu Andronic
Hello On Thu, Sep 20, 2012 at 7:02 AM, PJ Singh wrote: > Here are a few suggestions which I think will help give Lyx a more polished, > refined appearance... > Some of these suggestions seem to be related to: http://www.lyx.org/trac/ticket/5636 Liviu > > 1. > Allow

Re: Problem with command-sequence

2012-09-20 Thread Jean-Marc Lasgouttes
Le 20/09/12 01:36, Lars Gullik Bjønnes a écrit : Right... but why now have LFUNS de async by default (when done as a script thingie), and a separate LFUN to make the async if wanted. Gui/Menu run LFUNS could still be async. Probably also part of that discussion which I completely missed. I do

Re: UI Suggestions

2012-09-20 Thread Jean-Marc Lasgouttes
Le 20/09/12 07:02, PJ Singh a écrit : 1. Allow configurable gradient backgrounds for menu toolbars. The gradient color information should be specified as parameters in the toolbar *.inc files under .../lyx/ui This is Qt business, in relation with OS theme support. 2.

Re: [LyX master] Calibrate log file parser

2012-09-20 Thread Jürgen Spitzmüller
2012/9/20 Lars Gullik Bjønnes : > 'closing' - why not const ref? Are there contex issues here, that makes > creating a copy required? No. I'll change it. Thanks. Jürgen > -- >Lgb >

Literate programming on LyX-2.0.4: no build button

2012-09-20 Thread Kayvan Sylvan
Hi guys, I'm back to playing with LyX and noweb and I installed the latest LyX on my Fedora 17 system. It works beautifully! Thank you all for the great work. I pulled up one of my files but I can't find the Build button or menu item. I can export to noweb and run build-script myself, but it

Re: Literate programming on LyX-2.0.4: no build button

2012-09-20 Thread Kayvan Sylvan
Kayvan Sylvan gmail.com> writes: > Hi guys, > > I'm back to playing with LyX and noweb and I installed the latest LyX on my Fedora 17 system. It works beautifully! Thank you all for the great work. > > I pulled up one of my files but I can't find the Build button or menu item. I can export

[patch] LyX scrolling issue

2012-09-20 Thread Scott Kostyshak
Some have reported problems with LyX scrolling on Windows and Mac. Robert Schulze has git-bisected his way to the root cause of the particular scrolling problem he is experiencing: http://www.lyx.org/trac/ticket/8135 I am wondering if his problem is the same as others. The patch for current branch

[patch] fix warning on branch

2012-09-20 Thread Scott Kostyshak
This patch for branch fixes a warning that was fixed on master here: ea50cd71f9d Is it OK? Scott diff --git a/src/frontends/qt4/GuiSpellchecker.cpp b/src/frontends/qt4/GuiSpellchecker.cpp index 5dbdbbb..c428807 100644 --- a/src/frontends/qt4/GuiSpellchecker.cpp +++

Re: [patch] fix warning on branch

2012-09-20 Thread Scott Kostyshak
> This patch for branch fixes a warning that was fixed on master here: > ea50cd71f9d > > Is it OK? > > Scott This updated patch fixes another warning on branch, which was also fixed in master at that commit. Scott diff --git a/src/Lexer.cpp b/src/Lexer.cpp index d824da0..664e1b9 100644 ---

Re: [patch] fix warning on branch

2012-09-20 Thread Scott Kostyshak
On Thu, Sep 20, 2012 at 4:05 PM, Scott Kostyshak wrote: >> This patch for branch fixes a warning that was fixed on master here: >> ea50cd71f9d >> >> Is it OK? >> >> Scott > > This updated patch fixes another warning on branch, which was also > fixed in master at that commit. > >

PATCH: Changes to make LyX work on Mountain Lion

2012-09-20 Thread Stephan Witt
Hi, I've tried to build LyX on Mac OS X 10.8 with Xcode 4.3.3 + Qt 4.8.3 cocoa. There are some changes here and there required to make it work. I've attached them as one patch for review. I had to change: * the linker flags for frameworks * get rid of carbon calls deprecated since 10.8 * correct

Re: [patch] LyX bash completion

2012-09-20 Thread Pavel Sanda
Scott Kostyshak wrote: > If the hardcoding is desired, then I will manually make the updates: I think hardcoding is fine here. Pavel

Re: [patch] LyX bash completion

2012-09-20 Thread Scott Kostyshak
On Thu, Sep 20, 2012 at 4:44 PM, Pavel Sanda wrote: > Scott Kostyshak wrote: >> If the hardcoding is desired, then I will manually make the updates: > > I think hardcoding is fine here. Pavel OK, updated patch attached. Thanks, Scott diff --git a/lib/scripts/bash_completion

Re: PATCH: Changes to make LyX work on Mountain Lion

2012-09-20 Thread Jean-Marc Lasgouttes
Le 20/09/12 22:40, Stephan Witt a écrit : * get rid of carbon calls deprecated since 10.8 For getting special paths, Package.cpp, what about using Qt functions directly? For ex. http://qt-project.org/forums/viewthread/10550 JMarc

Re: [patch] fix warning on branch

2012-09-20 Thread Lars Gullik Bjønnes
Scott Kostyshak writes: | On Thu, Sep 20, 2012 at 4:05 PM, Scott Kostyshak wrote: >>> This patch for branch fixes a warning that was fixed on master here: >>> ea50cd71f9d >>> >>> Is it OK? >>> >>> Scott >> >> This updated patch fixes another warning on

Re: Literate programming on LyX-2.0.4: no build button

2012-09-20 Thread Kayvan Sylvan
Kayvan Sylvan gmail.com> writes: > > I found out that I can do: > > File -> Export -> More Formats and Options -> Program > > But this is obviously more cumbersome than clicking a Build button. > > How can I get this functionality back? > > ---Kayvan > After some more looking at the

Re: [patch] LyX bash completion

2012-09-20 Thread Richard Heck
On 09/20/2012 05:00 PM, Scott Kostyshak wrote: On Thu, Sep 20, 2012 at 4:44 PM, Pavel Sanda wrote: Scott Kostyshak wrote: If the hardcoding is desired, then I will manually make the updates: I think hardcoding is fine here. Pavel OK, updated patch attached. Feel free to

Re: PATCH: Changes to make LyX work on Mountain Lion

2012-09-20 Thread Richard Heck
On 09/20/2012 04:40 PM, Stephan Witt wrote: Hi, I've tried to build LyX on Mac OS X 10.8 with Xcode 4.3.3 + Qt 4.8.3 cocoa. There are some changes here and there required to make it work. I've attached them as one patch for review. I had to change: * the linker flags for frameworks * get rid

Re: [patch] fix warning on branch

2012-09-20 Thread Richard Heck
All this looks fine to me once Lars's comments are handled. rh On 09/20/2012 05:49 PM, Lars Gullik Bjønnes wrote: Scott Kostyshak writes: | On Thu, Sep 20, 2012 at 4:05 PM, Scott Kostyshak wrote: This patch for branch fixes a warning that was fixed on

Re: [patch] LyX scrolling issue

2012-09-20 Thread Pavel Sanda
Scott Kostyshak wrote: > void GuiWorkArea::update(int x, int y, int w, int h) > { > - viewport()->update(x, y, w, h); > + viewport()->repaint(x, y, w, h); IIRC it was Andre's claim that we should use update instead of repaint. Maybe he would know why this slow downs LyX so much. Pavel

Re: [LyX master] Support for the Minion Pro LaTeX font

2012-09-20 Thread Richard Heck
On 09/19/2012 11:47 AM, Juergen Spitzmueller wrote: The branch, master, has been updated. - Log - commit e9ca6b6bf48b23559384e1108e01df68a9ee18b6 Author: Juergen Spitzmueller Date: Wed Sep 19 17:46:55 2012 +0200

Re: Templates g-brief, g-brief2

2012-09-20 Thread Max Funk
My previous post did apparently not arrive. I repeat it with only the patch file attached. This is a patch of an issue with the g-brief-de.lyx, g-brief-en.lyx and g-brief2.lyx templates. I fixed the following Problems: - The g-brief templates require latex package europs. However, since

Re: [patch] fix warning on branch

2012-09-20 Thread Scott Kostyshak
On Thu, Sep 20, 2012 at 5:49 PM, Lars Gullik Bjønnes wrote: > btw. std::string::c_str() should not be used here, > use std::string::data() instead. Lars, can you confirm that I made the right corrections? I changed c_str() to data() in two places. If they are right I will

Re: [patch] LyX bash completion

2012-09-20 Thread Scott Kostyshak
On Thu, Sep 20, 2012 at 6:14 PM, Richard Heck wrote: > On 09/20/2012 05:00 PM, Scott Kostyshak wrote: >> >> On Thu, Sep 20, 2012 at 4:44 PM, Pavel Sanda wrote: >>> >>> Scott Kostyshak wrote: If the hardcoding is desired, then I will manually make the