Code review comment for lp:~mzanetti/ubuntu-docviewer-app/fix-layouts-with-icons

Revision history for this message
Michael Zanetti (mzanetti) wrote :

Hi Stefano.

No, I was asked by Alan to have a look at your branch and had to do those changes to make it work with Qt 5.5 and my High-resolution screen, otherwise it would end up in a loop requesting those images over and over again at startup.

While the root cause for that loop is a bug in Qt, fixing the layout usage is a good idea anyways. As I did those changes already I just pushed the branch so the fix isn't lost.

« Back to merge proposal