wxGUI: create parallel wx.frame for Single Window layout development

[lindakarlovska]

This PR serves for preparing a coding environment for building a separate Single Window application. It creates a copy of GMFrame which is ready for adapting and developing a new Single Window Layout solution through Google Summer of Code.

[lindakarlovska]

This PR serves for preparing a coding environment for building a separate Single Window application. It creates a copy of GMFrame called SingleWindowFrame which is ready for adapting and developing a new Single Window Layout solution through Google Summer of Code.

@petrasovaa I would merge it to master (if everything is ok) and then every change against multiple layout will be well visible. I need all dependencies to see anyway so I think it is better to start from something which works.
I could make an entirely blank file and start really from the scratch but this approach makes more sense, at least for me, what's your opinion?

[petrasovaa]

This PR serves for preparing a coding environment for building a separate Single Window application. It creates a copy of GMFrame called SingleWindowFrame which is ready for adapting and developing a new Single Window Layout solution through Google Summer of Code.

@petrasovaa I would merge it to master (if everything is ok) and then every change against multiple layout will be well visible. I need all dependencies to see anyway so I think it is better to start from something which works.
I could make an entirely blank file and start really from the scratch but this approach makes more sense, at least for me, what's your opinion?

That makes sense, please fix first the flake8 issues and make a new directory, let's say 'main_window' and put the file there and this file could be named frame.py.

[lindakarlovska]

This PR serves for preparing a coding environment for building a separate Single Window application. It creates a copy of GMFrame called SingleWindowFrame which is ready for adapting and developing a new Single Window Layout solution through Google Summer of Code.

@petrasovaa I would merge it to master (if everything is ok) and then every change against multiple layout will be well visible. I need all dependencies to see anyway so I think it is better to start from something which works.
I could make an entirely blank file and start really from the scratch but this approach makes more sense, at least for me, what's your opinion?

That makes sense, please fix first the flake8 issues and make a new directory, let's say 'main_window' and put the file there and this file could be named frame.py.

@wenzeslaus What if flake and black do not match, which should take precedence? ..

[petrasovaa]

@wenzeslaus What if flake and black do not match, which should take precedence? ..

Which problem are you referring to specifically, I thought the problem with " : " but E203 is already ignored in gui/wxpython/.flake8
See also this.

[wenzeslaus]

I thought the problem with " : " but E203 is already ignored in gui/wxpython/.flake8

You need to run flake8 from the gui/wxpython/ dir in order for the config file to be picked up automatically. When in doubt about the local setup, see results in the CI which is (supposed to be) set the right way.

What if flake and black do not match, which should take precedence?

Generally, Black, but you should not be dealing with that unless you need to run flake8 in some special way.

[lindakarlovska]

I thought the problem with " : " but E203 is already ignored in gui/wxpython/.flake8

You need to run flake8 from the gui/wxpython/ dir in order for the config file to be picked up automatically. When in doubt about the local setup, see results in the CI which is (supposed to be) set the right way.

What if flake and black do not match, which should take precedence?

Generally, Black, but you should not be dealing with that unless you need to run flake8 in some special way.

Vashek, thank you for your explanation! Now it is clear. I added a new directory to the .flake8 file.

[petrasovaa]

Could you please fix these instead of ignoring them?

[lindakarlovska]

Fixed. :-)