Peter L Jones

Results 563 comments of Peter L Jones

> > Is there an unrelated bug in MacOS, then? > > As it only shows up on this PR, I do think it's related. If it only shows on...

> No. It seems to be the same on the legacy build. Thanks for checking. I know we've had some Qt issues on MacOS (Qt5 and Qt6)... It's strange that...

OK, I've now manually edited the clientsettingsbase.ui file to make the two input fields the same (copying the properties from the City/State to the Alias/Name). Can you take a look...

Hold on... is that just the field highlighting? Tab onto the audio alerts field and do another screenshot, could you?

I'm happy enough to move this to 3.10.0.

Just to "zoom in" on the poor alignment: MacOS ![image](https://user-images.githubusercontent.com/1549463/189956767-d18b5f66-7b1b-4a59-ba07-79d3558f1567.png) ![image](https://user-images.githubusercontent.com/1549463/189958688-f456bc53-0cd5-4903-85b7-e247bd70da14.png) vs Linux (displayed on Windows) ![image](https://user-images.githubusercontent.com/1549463/189957752-e589240a-7fd2-442e-a1be-ec62e53d0ea0.png) ![image](https://user-images.githubusercontent.com/1549463/189958484-de3d1ad5-df2f-416b-a871-12700e502a5f.png) The MacOS fields are failing to follow the direction to fill the...

I've also done a different version here... https://github.com/pljones/jamulus/pull/new/bugfix/%232723-self-label-checkboxes-v2

Based on your Qt Designer screenshots (and the behaviour I've seen in different builds locally)... all I can say is that it's highly unpredictable what the behaviour will be! Even...

> that's 3.9.0dev-96c0e843-dirty (your second approach) and this seems to be ok. The field edge alignment is still poor compared with Linux and Windows - but I guess that's just...

OK, [pljones:bugfix/\#2723-self-label-checkboxes](https://github.com/pljones/jamulus/compare/bugfix/%232723-self-label-checkboxes) has been rebased to "latest" master as of now. - builds: https://github.com/pljones/jamulus/actions/runs/3084620441 The "v2" version has also been rebased to master. - builds: https://github.com/pljones/jamulus/actions/runs/3084621307 There's also now a...