Page 1 of 2

macOS room dialog's save button

Posted: Thu May 17, 2018 4:25 pm
by Vadi
As mentioned here, there's an issue with the room exits dialog on mac OS.

SlySven took a stab at fixing it, does https://transfer.sh/rodjR/Mudlet-3.9.0- ... d407d9.dmg work better?

Re: macOS room dialog's save button

Posted: Thu May 17, 2018 6:48 pm
by Jor'Mox
No apparent change with this version, sorry.

Re: macOS room dialog's save button

Posted: Fri May 18, 2018 7:48 pm
by SlySven
It is blooming weird - but it only affects macOs at the moment! ;)

One thing I did note was that the Travis CI build - the source of what I imaging you are using is a Release build made using Qt 5.10 - which is later than the Windows or Linux CI builds. If there is anyone who can build Mudlet from sources on the macOs it would be very helpful if they could investigate what is happening in (void) dlgRoomExits::slot_checkModified() in a debug build (so they can inspect the variables) when the doors of an exit is changed - is it detecting a change and reaching the end of the method with (bool) isModified set to true... :ugeek:

Re: macOS room dialog's save button

Posted: Fri May 18, 2018 8:45 pm
by Jor'Mox
Just for reference, I checked back with different versions, and in version 3.6.1 things work properly here, but they don't work in 3.7. I figured that might help pin down the source of the problem.

Re: macOS room dialog's save button

Posted: Sat May 19, 2018 1:05 am
by SlySven
Guess what - there is no change in the Mudlet source code in the dlgRoomExits class between 3.6.1 and 3.7.1!!! Indeed since 3.6.1 the only change up to 3.9.0 was a missing check for the out exit door changing being added in (and what was added cannot have caused this issue) and some spelling errors fixed in some texts.

Re: macOS room dialog's save button

Posted: Sat May 19, 2018 5:24 am
by Vadi
Thanks for doing that research, Jor'Mox. 3.6.1 was built using Qt 5.9.3 and 3.7.1 was built using 5.10.0 - perhaps the issue is indeed in the Qt version difference?

We'll see if we can build using 5.9.3 (or 5.9.5, I think that is the latest in 5.9 LTS series) to check for the issue.

Re: macOS room dialog's save button

Posted: Sat May 19, 2018 12:31 pm
by keneanung
Building the latest with Qt 5.9 might be hard as we'd have to install Qt through other means than homebrew. We can do another test more easily: check 3.6.1 built with 5.10 for the issue. I'll add a link later as this can't be done through travis either :(

Re: macOS room dialog's save button

Posted: Sat May 19, 2018 1:51 pm
by Jor'Mox
While I get that it is a different issue, the loss of window size permanence for Macs also arose as we moved from 3.6.1 to 3.7, so it would be interesting to see if both problems can be attributed to the version of Qt used, or if there is some other change that potentially caused them both.

Re: macOS room dialog's save button

Posted: Sat May 19, 2018 4:22 pm
by keneanung
Okay, after some issues with backwards compatibility of our installer creation scripts, here's the promised link: https://transfer.sh/Ngs1u/Mudlet-3.6.1.dmg

Re: macOS room dialog's save button

Posted: Sat May 19, 2018 4:47 pm
by Jor'Mox
I can confirm that both issues, the window size impermanence and the save button not visibly enabling when only changing door status of rooms appear in the version of 3.6.1 that you linked here. But since both things should definitely be within the capabilities of Qt, regardless of version, hopefully a way can be found to change the existing code for those two behaviors to get Qt to do what it is supposed to do.