Re: [Kicad-developers] wxStaticBoxSizer fixes

2018-01-11 Thread jp charras
Le 11/01/2018 à 19:29, Chris Pavlina a écrit :
> Interesting. I wonder if we could implement a wrapper around
> wxStaticBoxSizers to fix the issue rather than switching away from them
> - they look much nicer than the replacement. Which KDE version are you
> using specifically? I'd like to have a look at this.
> 
> (Would be nice if these fixes could also be submitted upstream...)
> 

I am using Kubuntu 14.04 LTS.
This is a old version, but it is a good version to test build issues
(I have a more recent version installed: Ubuntu 16.04 LTS / Unity for other 
tests:
Unity has its own issues)
I am thinking my WM version is KDE 4
Note also the issue is annoying only when using wxNotebooks.

This issue is especially noticeable with wxWidgets built against GTK3 (although 
GTK3 is not usable
because wxDC just does not work for us: it is broken).

About: "look much nicer than the replacement", it is hard to have a strong 
opinion:
Remember, the beauty is in the eye of the guy who is looking at.

For me, I prefer the replacement on KDE and the wxStaticBoxSizer version on 
Windows.

Good luck.


-- 
Jean-Pierre CHARRAS

___
Mailing list: https://launchpad.net/~kicad-developers
Post to : kicad-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp


Re: [Kicad-developers] wxStaticBoxSizer fixes

2018-01-11 Thread Chris Pavlina
Interesting. I wonder if we could implement a wrapper around
wxStaticBoxSizers to fix the issue rather than switching away from them
- they look much nicer than the replacement. Which KDE version are you
using specifically? I'd like to have a look at this.

(Would be nice if these fixes could also be submitted upstream...)

On Thu, Jan 11, 2018 at 07:14:27PM +0100, jp charras wrote:
> Le 11/01/2018 à 18:27, Chris Pavlina a écrit :
> > Hi (@JP mostly),
> > 
> > I see you're removing wxStaticBoxSizers [1] due to issues with KDE. Have
> > you tried the fix I implemented in [2] for what I think is the same
> > issue? You commented on the bug report and said it worked for you, and I
> > didn't have to remove the wxStaticBoxSizer, only change a property of
> > it.
> > 
> > [1] 
> > https://git.launchpad.net/kicad/commit/?id=6f2beebf2c623b9cf85faa6deb8f65d32b8ef7ff
> > [2] 
> > https://git.launchpad.net/kicad/commit/?id=6f900530204b44c0e80464ea3c752877c4ff6fc7
> > 
> > I'm not sure exactly how this is connected to KDE, I'm quite curious
> > about that - but they do seem to be wonky without that setting change.
> > I'll admit I don't fully understand why my own fix works.
> > 
> 
> 
> On KDE, I always had issues with wxStaticBoxSizers. It is not a recent issue.
> But the issue is not the same as [2].
> 
> They are incorrectly vertically sized (incorrect height: a few pixels error).
> 
> And when  wxStaticBoxSizer are used in wxNotebook, when placed in page 2, 
> they are correctly sized
> only after the dialog is resized if the dialog is opened with wxNotebook page 
> 1 selected.
> 
> of course if the dialog is opened with wxNotebook page 2 selected, 
> wxStaticBoxSizers used in page 1
> are incorrectly sized (and correctly sized after a dialog resize).
> 
> I do not have this issue on Unity.
> 
> -- 
> Jean-Pierre CHARRAS
> 
> ___
> Mailing list: https://launchpad.net/~kicad-developers
> Post to : kicad-developers@lists.launchpad.net
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp

___
Mailing list: https://launchpad.net/~kicad-developers
Post to : kicad-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp


Re: [Kicad-developers] wxStaticBoxSizer fixes

2018-01-11 Thread jp charras
Le 11/01/2018 à 18:27, Chris Pavlina a écrit :
> Hi (@JP mostly),
> 
> I see you're removing wxStaticBoxSizers [1] due to issues with KDE. Have
> you tried the fix I implemented in [2] for what I think is the same
> issue? You commented on the bug report and said it worked for you, and I
> didn't have to remove the wxStaticBoxSizer, only change a property of
> it.
> 
> [1] 
> https://git.launchpad.net/kicad/commit/?id=6f2beebf2c623b9cf85faa6deb8f65d32b8ef7ff
> [2] 
> https://git.launchpad.net/kicad/commit/?id=6f900530204b44c0e80464ea3c752877c4ff6fc7
> 
> I'm not sure exactly how this is connected to KDE, I'm quite curious
> about that - but they do seem to be wonky without that setting change.
> I'll admit I don't fully understand why my own fix works.
> 


On KDE, I always had issues with wxStaticBoxSizers. It is not a recent issue.
But the issue is not the same as [2].

They are incorrectly vertically sized (incorrect height: a few pixels error).

And when  wxStaticBoxSizer are used in wxNotebook, when placed in page 2, they 
are correctly sized
only after the dialog is resized if the dialog is opened with wxNotebook page 1 
selected.

of course if the dialog is opened with wxNotebook page 2 selected, 
wxStaticBoxSizers used in page 1
are incorrectly sized (and correctly sized after a dialog resize).

I do not have this issue on Unity.

-- 
Jean-Pierre CHARRAS

___
Mailing list: https://launchpad.net/~kicad-developers
Post to : kicad-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp


[Kicad-developers] wxStaticBoxSizer fixes

2018-01-11 Thread Chris Pavlina
Hi (@JP mostly),

I see you're removing wxStaticBoxSizers [1] due to issues with KDE. Have
you tried the fix I implemented in [2] for what I think is the same
issue? You commented on the bug report and said it worked for you, and I
didn't have to remove the wxStaticBoxSizer, only change a property of
it.

[1] 
https://git.launchpad.net/kicad/commit/?id=6f2beebf2c623b9cf85faa6deb8f65d32b8ef7ff
[2] 
https://git.launchpad.net/kicad/commit/?id=6f900530204b44c0e80464ea3c752877c4ff6fc7

I'm not sure exactly how this is connected to KDE, I'm quite curious
about that - but they do seem to be wonky without that setting change.
I'll admit I don't fully understand why my own fix works.

-- 
Chris

___
Mailing list: https://launchpad.net/~kicad-developers
Post to : kicad-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp