Arch Linux package updates articles are currently being rendered without zoom.
I have tested moving the call to setZoomFactor to later QWebEnginePage signals to find the optimal place to put it.
loadFinished: Renders zoomed, but only after the article is fully loaded.
loadStarted: Renders without zoom.
loadProgress: Renders zoomed.
This patch implements the loadProgress option.
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
You could wrap it in a check if the current zoom actually differs from Settings::zoom, this way the zoom should only get set once and QtWebEngine will hopefully honor it for the rest of the loading/rendering. Briefly checking the QtWebEngine code reveals that any call to setZoomFactor() triggers something inside the engine, so the fewer times we do it the better :)
Implemented. I have seen the call to setZoomFactor be made at 0, 70, and 100% progress. Worst case from a short test was two calls to setZoomFactor.
| src/articleviewer-ng/webengine/articleviewerwebenginepage.cpp | ||
|---|---|---|
| 74 | static_cast<double>(...) | |