Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

set zoom factor after page has finished loading #774

Merged
merged 1 commit into from
Feb 25, 2022
Merged

Conversation

juuz0
Copy link
Collaborator

@juuz0 juuz0 commented Jan 24, 2022

Fixes #770

@juuz0 juuz0 requested a review from kelson42 January 24, 2022 16:36
@juuz0
Copy link
Collaborator Author

juuz0 commented Jan 24, 2022

I couldn't get a good reason on why that was happening (it didnt happen, if the search bar article was opened in a different tab) but this looks like the culprit, known bug :
https://bugreports.qt.io/browse/QTBUG-51851

@kelson42 kelson42 requested a review from mgautierfr January 24, 2022 16:40
Copy link
Collaborator

@kelson42 kelson42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The search result page should have the zoom from the "default zoom" settings

@juuz0 juuz0 requested a review from kelson42 January 24, 2022 17:08
@kelson42
Copy link
Collaborator

It seems we can not avoid this zoom changing problem because of https://bugreports.qt.io/browse/QTBUG-51851

Copy link
Collaborator

@kelson42 kelson42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Beside the rezooming bug (upstream) after clicking on a search result, it looks good. @mgautierfr Please make the code review.

@kelson42
Copy link
Collaborator

kelson42 commented Feb 2, 2022

@juuz0 Would you be able please to make a comment in the code with a link to the upstream ticket to explain why you rezzoom after having loaded the content

@juuz0
Copy link
Collaborator Author

juuz0 commented Feb 2, 2022

@kelson42 Added, rebased

@kelson42 kelson42 requested review from veloman-yunkan and removed request for mgautierfr February 8, 2022 05:47
@kelson42
Copy link
Collaborator

kelson42 commented Feb 8, 2022

@veloman-yunkan @mgautierfr Changing reviewer in an attemos to move forward with this PR :)

src/webview.cpp Outdated Show resolved Hide resolved
src/webview.cpp Outdated Show resolved Hide resolved
src/webview.cpp Outdated Show resolved Hide resolved
src/webview.cpp Outdated Show resolved Hide resolved
@kelson42
Copy link
Collaborator

@juuz0 Whats up with this PR?

src/webview.cpp Outdated Show resolved Hide resolved
src/webview.cpp Outdated Show resolved Hide resolved
This fixes the problem of page not getting the correct zoom factor (after accessing an article from searchbar)
Now, the zoom factor is set again after page has completely finished loading
@kelson42 kelson42 merged commit 09a0259 into master Feb 25, 2022
@kelson42 kelson42 deleted the zoomSearchFix branch February 25, 2022 11:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Zooming function is functionally flawed
4 participants