Re: [Geeqie-devel] [BestImageViewer/geeqie] Use Formatted Renaming wrong and you'll lose your pictures, PERMANENTLY! (#380)

2016-04-28 Thread Klaus Ethgen 
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA512

Hi,

Am Do den 28. Apr 2016 um 18:27 schrieb Colin Clark:
> > Which makes me believe that it is #377.
> This patch came from a post by Jonathan Boeing on the sourceforge 
> mailing-list.
> 
> I put it into the repository without a github Issue number being raised.
> 
> In the commit comment I should have put a link to the entry in the 
> sourceforge mailing-list, so that there was traceability to the origin 
> of the patch.

Ah, no problem about that. I just learned myself of how to close bugs
with a commit message[0] recently.

As I migrated all bugs to github and made bugs in SF readonly, I think
we should at least close them if possible. There are to many bugs and
nobody have a real overview over them.

Regards
   Klaus

[0] https://help.github.com/articles/closing-issues-via-commit-messages/
Note that you have to add fix (or likely) for every bug. Enlisting
them in one fix line does only close the first bug.
- -- 
Klaus Ethgen  http://www.ethgen.ch/
pub  4096R/4E20AF1C 2011-05-16   Klaus Ethgen 
Fingerprint: 85D4 CA42 952C 949B 1753  62B3 79D0 B06F 4E20 AF1C
-BEGIN PGP SIGNATURE-
Version: GnuPG v1
Comment: Charset: ISO-8859-1

iQGcBAEBCgAGBQJXIl7ZAAoJEKZ8CrGAGfasoK8L/iKzs/+vrv4UQubnAQcs6veQ
XuELsdLRQd6OAKCYfxodJ3p4bizyTk9wnJe+JKZTob/IGQDaP6V5Ek1boqB++iYR
d9H/Lu9vDmfPPb/kGa0wsBTb0FztsQFh4FiHGUI+QwIUmVTM146iFOFu2JRTn14M
xYlwztlHMPoJvtvdA4SDYN70XSeYsZnBa/+zGJpntlttzei/9INt/1Sm0c1ew3h2
VT72SyeSSe9Q04Xbwc+XP5QnSkrgE1O4t3I2Mi3EbJtPO3fyaccLRxNNWpxsnZpF
4hikYd7dx6eBQHA/wfVfsnfkME3DdNd15/NW3O/xoAOsSlWY/+XOfpdTdpH2vYae
eRfxYL8i3N1DzdCZxAIoPkHlRj3jcsGNLwLiD7chHC4doGd2Wy4C/SQGi7+UuD2s
YjvULsHSk86SQFaU6LkqjdlA4wHFnfpoCtab/dbXwbt14sk7MXPx8ue+4H9qQlQy
ExNKFS4AGpGVCAJiR7MhrpjJ9yhu8al6SKmE64Qj2A==
=QRLW
-END PGP SIGNATURE-

--
Find and fix application performance issues faster with Applications Manager
Applications Manager provides deep performance insights into multiple tiers of
your business applications. It resolves application problems quickly and
reduces your MTTR. Get your free trial!
https://ad.doubleclick.net/ddm/clk/302982198;130105516;z
___
Geeqie-devel mailing list
Geeqie-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geeqie-devel

Re: [Geeqie-devel] [patch] Fullscreen fix

2016-04-28 Thread Klaus Ethgen 
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA512

Hi,

Am Do den 28. Apr 2016 um 19:03 schrieb Colin Clark:
> On 21/04/16 20:13, Josef Kufner wrote:
> > Good news everyone,
> >
> > I managed to fix the fullscreen mode. At least it looks fixed for me :)
> > -- Tested on multi-screen setup on KDE5.
> >
> > Here is the pull request:
> > https://github.com/BestImageViewer/geeqie/pull/379
> >
> > What do you say? Does it work for you?
> >
[...]
> Your patch eliminates this problem for me. So, yes, it works for me too.

Yes, this fixes several full-screen problems. Merged and pushed to both,
github and main repo.

Regards
   Klaus
- -- 
Klaus Ethgen  http://www.ethgen.ch/
pub  4096R/4E20AF1C 2011-05-16   Klaus Ethgen 
Fingerprint: 85D4 CA42 952C 949B 1753  62B3 79D0 B06F 4E20 AF1C
-BEGIN PGP SIGNATURE-
Version: GnuPG v1
Comment: Charset: ISO-8859-1

iQGcBAEBCgAGBQJXIlppAAoJEKZ8CrGAGfas7rAL/27q2YzgvrnTYPPAKByZnJT2
SvycbcZxYpP7GjTbRIqeG4jXJ55FoDJ5iXAZOR/RfgxpSCtFMu3z+ArGwY58dMTA
b8fhtjilixnOWWBj0OUvvsszDcJcqfOvD9U8aRV15tDz93G851S0EjgR4RAumYEr
5RPI9Uy00V+ZHlwG24AuSY9oDqSniP3b8o6SiaZB+FVvWiKvH/89Gwx7VczP2MOK
03jotRIRAUdQzKfaA38h0w2wsZzxpsC6xVNZN22bQ0dzzRW+mnFjk8QCQwW0+GSc
j+rRvDJs/sdibDhwYCuYELg65qsYUShhKsC8IyvLRExU9NDjwU5b+wXkDpThUj3D
p0EXaGGLCK4OhMweYbZNchdbvvKMoDkjUI4CTKJWvObfWcXYEIr9N7Z7cnhTJ3cR
72851FzjSvyMt63BPvxng4cUa1WBrsB7FPVX12BzI+YlbOo/ZWnf+OURA4/uW9yP
Xt/NCcbfV9+r5Xjc0G0KUmkoR8lUrg0THVgZe2DmZg==
=DFJk
-END PGP SIGNATURE-

--
Find and fix application performance issues faster with Applications Manager
Applications Manager provides deep performance insights into multiple tiers of
your business applications. It resolves application problems quickly and
reduces your MTTR. Get your free trial!
https://ad.doubleclick.net/ddm/clk/302982198;130105516;z
___
Geeqie-devel mailing list
Geeqie-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geeqie-devel

Re: [Geeqie-devel] [patch] Fullscreen fix

2016-04-28 Thread Colin Clark 
On 21/04/16 20:13, Josef Kufner wrote:
> Good news everyone,
>
> I managed to fix the fullscreen mode. At least it looks fixed for me :)
> -- Tested on multi-screen setup on KDE5.
>
> Here is the pull request:
> https://github.com/BestImageViewer/geeqie/pull/379
>
> What do you say? Does it work for you?
>
The full-screen problem I have encountered is:

switch geeqie to full screen mode
attempt to alt-tab to switch to another application - nothing happens, 
geeqie remains on top
pressing "escape" no longer terminates geeqie's full-screen
the only way out is to log in to another session and kill geeqie


Your patch eliminates this problem for me. So, yes, it works for me too.


Colin Clark



--
Find and fix application performance issues faster with Applications Manager
Applications Manager provides deep performance insights into multiple tiers of
your business applications. It resolves application problems quickly and
reduces your MTTR. Get your free trial!
https://ad.doubleclick.net/ddm/clk/302982198;130105516;z
___
Geeqie-devel mailing list
Geeqie-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geeqie-devel

Re: [Geeqie-devel] [BestImageViewer/geeqie] Use Formatted Renaming wrong and you'll lose your pictures, PERMANENTLY! (#380)

2016-04-28 Thread Colin Clark 
On 26/04/16 19:54, Klaus Ethgen wrote:
> -BEGIN PGP SIGNED MESSAGE-
> Hash: SHA512
>
> Hi Colin,
>
> Am Di den 26. Apr 2016 um 19:23 schrieb Colin Clark:
>>> I think, you refer to Bug #377, not #380...?
>> #377 is "Reorganising bookmarks by mouse drag causes seg fault", which is
>> still open. (Actually, I think it is probably a general DnD fault, not a
>> bookmarks problem.)
> Hmm... But the commit comment you gave is:
>  Fix crash on drag and drop from Xfe
>
>  The issue is that when dragging and dropping from Xfe, paths aren't
>  escaped like they are when dnd'ing from a GTK app.
>
> Which makes me believe that it is #377.
>
>
Hi Klaus

This patch came from a post by Jonathan Boeing on the sourceforge 
mailing-list.

I put it into the repository without a github Issue number being raised.

In the commit comment I should have put a link to the entry in the 
sourceforge mailing-list, so that there was traceability to the origin 
of the patch.


Colin...




--
Find and fix application performance issues faster with Applications Manager
Applications Manager provides deep performance insights into multiple tiers of
your business applications. It resolves application problems quickly and
reduces your MTTR. Get your free trial!
https://ad.doubleclick.net/ddm/clk/302982198;130105516;z
___
Geeqie-devel mailing list
Geeqie-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geeqie-devel