<div dir="ltr"><div>Hi Peter,</div><div><br></div><div>I don't see a problem to conditionally disable shadows for Qt <= 5.10, we also have other conditions depending on the Qt version in other parts of the codebase.<br></div><div>Is there a hidden question how to go about testing this in CI? If yes, I'd say the best approach would indeed be to enable tests for Qt 5.14.</div><div><br></div><div>Best regards<br></div><div>Matthias<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Aug 27, 2020 at 1:58 PM Peter Petrik <<a href="mailto:peter.petrik@lutraconsulting.co.uk">peter.petrik@lutraconsulting.co.uk</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr">Hi, <div><br></div><div>CI on travis runs with qt 5.9 and for <a href="https://github.com/qgis/QGIS/pull/38448" target="_blank">https://github.com/qgis/QGIS/pull/38448</a> we need to have qt 5.10+. In 5.9 the 3d shadow rendering does not work correctly and since Windows, Mac and latest Linux distributions run on newer qt's I do not see a reason to spend considerable time to fix the 3d shadows for qt 5.9. I was thinking to disable this feature for builds with qt 5.9.I see the qt5.14 build in the repository, but that one is without tests. </div><div><br></div><div>I am wondering how to best approach this? </div><div><br></div><div>Thanks,</div><div>Peter</div><div></div></div></div>
_______________________________________________<br>
QGIS-Developer mailing list<br>
<a href="mailto:QGIS-Developer@lists.osgeo.org" target="_blank">QGIS-Developer@lists.osgeo.org</a><br>
List info: <a href="https://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br>
Unsubscribe: <a href="https://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/qgis-developer</a></blockquote></div>