Re: [LyX/master] Some fixups to row margins

2016-03-04 Thread Kornel Benko
Am Samstag, 5. März 2016 um 01:24:15, schrieb Scott Kostyshak 
> On Wed, Jul 22, 2015 at 12:01:34AM +0200, Jean-Marc Lasgouttes wrote:
> > commit 412a724aaf8a532adba71f8e8491c258f651a4e1
> > Author: Jean-Marc Lasgouttes 
> > Date:   Sat Jul 18 00:07:30 2015 +0200
> > 
> > Some fixups to row margins
> > 
> > In breakRow set left and right margin properly for RTL paragraphs.
> > Remove corresponding code from ComputeRowMetrics.
> > 
> > In row painter, check the use of left and right margin depending on
> > context. The problem in the original text is that the various
> > leftMargin() methods actually represent right margin for RTL
> > paragraphs. This should be fixed eventually.
> 
> git bisect suggests that this commit might have changed some behavior
> for me regarding cursor placement.
> 
> Start a new document. Type 'a', then do ctrl + return. The cursor is
> further left than it used to be, and is 1cm to the left of where a
> letter would be placed if you type another letter.

And if done in labeling the cursor does not jump to next line. Cursor position 
will be corrected on any subsequent character.
Looks funny.

> Scott

Kornel

signature.asc
Description: This is a digitally signed message part.


Re: [LyX/master] Some fixups to row margins

2016-03-04 Thread Scott Kostyshak
On Wed, Jul 22, 2015 at 12:01:34AM +0200, Jean-Marc Lasgouttes wrote:
> commit 412a724aaf8a532adba71f8e8491c258f651a4e1
> Author: Jean-Marc Lasgouttes 
> Date:   Sat Jul 18 00:07:30 2015 +0200
> 
> Some fixups to row margins
> 
> In breakRow set left and right margin properly for RTL paragraphs.
> Remove corresponding code from ComputeRowMetrics.
> 
> In row painter, check the use of left and right margin depending on
> context. The problem in the original text is that the various
> leftMargin() methods actually represent right margin for RTL
> paragraphs. This should be fixed eventually.

git bisect suggests that this commit might have changed some behavior
for me regarding cursor placement.

Start a new document. Type 'a', then do ctrl + return. The cursor is
further left than it used to be, and is 1cm to the left of where a
letter would be placed if you type another letter.

Scott


signature.asc
Description: PGP signature


Re: Problem with key shortcuts on 2.2 with qt-5.5

2016-03-04 Thread Scott Kostyshak
On Fri, Mar 04, 2016 at 11:17:42AM +, José Matos wrote:

> What is not to be expected is that when pressing Alt it also underlines 
> letter 
> in the open buffers, as an example I have r for read-review, 2 for 2016 and p 
> for software-projects.

I don't understand, are read-review and 2016 file names? Or they are
just text in a buffer?

Scott


signature.asc
Description: PGP signature


Re: new compiler warning in master

2016-03-04 Thread Jean-Marc Lasgouttes

Le 04/03/2016 01:21, Uwe Stöhr a écrit :

Am 04.03.2016 um 00:26 schrieb Jean-Marc Lasgouttes:


In the meantime I pushed (and then reverted) my patch by mistake.


This fixed the warnings I got.


What about this one?


I can test it but what should I do to verify that it works?


I think it is safe. Just confirm that the warnings are gone.

JMarc



Re: Qt5.6 and MSVC

2016-03-04 Thread Peter Kümmel

Am 04.03.2016 um 01:34 schrieb Uwe Stöhr:

I played a bit with Qt5.6RC and the latest MSVC 2015. I can compile current 
master without errors but finally I still get this linking error:

D:\LyXGit\Master\compile-2015\src\tests\check_ExternalTransforms.vcxproj" 
(standard target) (18) ->
   support.lib(os.obj) : error LNK2019: unresolved external symbol ___wgetmainargs 
referenced in function "void __cdecl lyx::support::os::init(int,char * * 
const)" (?init@os@support@lyx@@YAXHQAPAD@Z) 
[D:\LyXGit\Master\compile-2015\src\tests\check_ExternalTransforms.vcxproj]

D:\LyXGit\Master\compile-2015\bin\Release\check_ExternalTransforms.exe : fatal 
error LNK1120: 1 unresolved externals 
[D:\LyXGit\Master\compile-2015\src\tests\check_ExternalTransforms.vcxproj]

for the

- LyX.exe
- check_Length.exe
- check_ExternalTransforms.exe
- check_ListingsCaption.exe
- check_convert.exe

I still cannot imagine that there is a bug in MSVC because I get the same with 
MSVC 2013 and cannot imagine that users did not yet reported such a linking 
error.

Of course I can switch to MinGW but to me its seems we have a real issue that 
should be corrected.
I read that MinGW 4.9 is not fully capable of the windows handling in Win 10 so 
if possible I would prefer using MSVC for maintainability reasons. Of course if 
the mentioned error is not fixable I won't have another option.

thanks and regards
Uwe



___wgetmainargs was removed in 2015.

Please try this patch

https://github.com/syntheticpp/lyx/commit/2470fb442cb2b04a69b2030f28f1da60221556a7?diff=unified

Peter


Re: CMake issue

2016-03-04 Thread Peter Kümmel



Am 04.03.2016 um 03:58 schrieb Uwe Stöhr:

I have this code in my debug build script:

cmake %LYX_SOURCE% -GNinja -G"Visual Studio 14" -DLYX_ENABLE_CXX11=ON 
-DLYX_USE_QT=QT5 -DLYX_ENABLE_EXPORT_TESTS=0 -DLYX_MERGE_FILES=0 -DLYX_NLS=1 
-DLYX_INSTALL=0 -DLYX_RELEASE=0 -DLYX_CONSOLE=FORCE -DLYX_3RDPARTY_BUILD=1
msbuild lyx.sln /p:Configuration=Debug /t:LyX /t:tex2lyx

and get the error that a target "LyX" does not exist in the lyx.sln file. That is true, 
there is only a project folder named "LyX". From within MSVC I can compile this folder 
but not from outside because the target is missing.

I get the same problem with MSVC2010. In LyX's 2.1.x branch the target is 
there, but not in master.

thanks and regards
Uwe



You've passed -G two times.

Peter


Re: CMake issue

2016-03-04 Thread Kornel Benko
Am Freitag, 4. März 2016 um 03:58:08, schrieb Uwe Stöhr 
> I have this code in my debug build script:
> 
> cmake %LYX_SOURCE% -GNinja -G"Visual Studio 14" -DLYX_ENABLE_CXX11=ON 
> -DLYX_USE_QT=QT5 -DLYX_ENABLE_EXPORT_TESTS=0 -DLYX_MERGE_FILES=0 
> -DLYX_NLS=1 -DLYX_INSTALL=0 -DLYX_RELEASE=0 -DLYX_CONSOLE=FORCE 
> -DLYX_3RDPARTY_BUILD=1
> msbuild lyx.sln /p:Configuration=Debug /t:LyX /t:tex2lyx
> 
> and get the error that a target "LyX" does not exist in the lyx.sln 
> file. That is true, there is only a project folder named "LyX". From 
> within MSVC I can compile this folder but not from outside because the 
> target is missing.

'LyX' is not the same as 'lyx'. What about the target 'lyx'?

> I get the same problem with MSVC2010. In LyX's 2.1.x branch the target 
> is there, but not in master.
> 
> thanks and regards
> Uwe

Kornel

signature.asc
Description: This is a digitally signed message part.


pybtex looks interesting

2016-03-04 Thread Neal Becker
http://pythonhosted.org/pybtex/



Problem with key shortcuts on 2.2 with qt-5.5

2016-03-04 Thread José Matos
Hi,
when working with lyx-2.2beta2 I noticed a strange behaviour:

I am used to change the paragraph layout with Alt+P ... The problem now 
is 
that when I press Alt I see in the menus some letters underlined, F for File, 
E for Edit and so on. That is to be expect and it works as in other 
applications.

What is not to be expected is that when pressing Alt it also underlines letter 
in the open buffers, as an example I have r for read-review, 2 for 2016 and p 
for software-projects. So that every time I try to change the paragraph layout 
I get in that buffer. A work around is to open the buffer in a new window and 
then this problems does not occur.

The obligatory reference is:

LyX Version 2.2.0beta2
(Friday, February 12, 2016)
Built from git commit hash 
Library directory: /usr/share/lyx/
User directory: ~/.lyx/
Qt Version (run-time): 5.5.1
Qt Version (compile-time): 5.5.1

Is this to be expected or a bug?

Regards,
-- 
José Abílio


Re: new compiler warning in master

2016-03-04 Thread Enrico Forestieri
On Fri, Mar 04, 2016 at 12:26:11AM +0100, Jean-Marc Lasgouttes wrote:

> Le 04/03/2016 00:14, Jean-Marc Lasgouttes a écrit :
> >>>Nope.
> >>
> >>The attached patch does the trick for me.
> >
> >THe warnings you see are not the same as Uwe's warnings.
> 
> In the meantime I pushed (and then reverted) my patch by mistake.
> 
> What about this one?

This also works for me, of course.

-- 
Enrico