QtVCP reading error message

More
7 years 8 months ago #126191 by phillc54
There are a few other odd little issues I have noticed with the gcode editor.
I will have a go at 'fixing' them over the next few days and get back to you, hopefully I can save you spending too much time on them.
I will probably have some questions for you as I stumble along.

Cheers, Phill.

Please Log in to join the conversation.

More
7 years 8 months ago #126435 by phillc54
Most of the gcode editor oddities were caused when mode switching.
[strike]I noticed the editor didn't have the auto_show_mdi_status that the viewer has so I copied it across.[/strike]
I also stuck setModified(False) in a few places to stop file changed dialog appearing when it shouldn't.
Attached is a git diff file of the changes I made, hopefully this is of some use.

Cheers, Phill

Edit: I have just merged master into my repo and I noticed that you had already fixed the auto_show_mdi_status

Please Log in to join the conversation.

More
7 years 8 months ago #126523 by cmorley
Replied by cmorley on topic QtVCP reading error message
I added your code to my last push of work.
Thank you.

Chris M

Please Log in to join the conversation.

More
7 years 8 months ago #126525 by phillc54
I have also done a bit of work on the MDI history widget, what is the best way to get it to you to have a look at.

Cheers, Phill.

Please Log in to join the conversation.

More
7 years 8 months ago #126533 by cmorley
Replied by cmorley on topic QtVCP reading error message
If you are using Git then a git patch is the nicest - also keeps your name on the file.
Here is a link about git patches if that is new to you:
www.devroom.io/2009/10/26/how-to-create-...ly-a-patch-with-git/

you can create a pull request through Github.
github.com/LinuxCNC/linuxcnc

but a diff or even just the file is fine too.
Thank you for your contributions

Chris M

Please Log in to join the conversation.

More
7 years 8 months ago #126589 by phillc54
Git patch sounds fine, would you prefer I post them to the forum or email to you directly.

Cheers, Phill.

Please Log in to join the conversation.

More
7 years 8 months ago #126593 by cmorley
Replied by cmorley on topic QtVCP reading error message
Either is fine by me.

Chris M

Please Log in to join the conversation.

Time to create page: 0.049 seconds
Powered by Kunena Forum