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

Fix webview scroll #973

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Fix webview scroll #973

wants to merge 4 commits into from

Conversation

jeremypw
Copy link
Contributor

@jeremypw jeremypw commented Nov 10, 2023

Fixes #950

The currently used version of WebView does not interact properly with Gtk's size allocation when buried inside a scrolled window inside a revealer inside a listbox ...

The only solution I could find was to not show the webview within the list box but to show it in a separate pane of of a Gtk.Paned so that the native scrolling ability of WebView is used. Clearly, this changes the UI significantly so may not be acceptable but at least the message contents of long messages are always visible.

Screenshot from 2024-01-22 12 21 19

Image shows a thread with 4 messages. The most recent message header is shown expanded, the rest are below it in a scrolled window (i.e. reversed order compared to master). This is because of problems controlling where the divider of the vertical pane is placed automatically.

@jeremypw
Copy link
Contributor Author

@danirabbit Is this a viable approach? Or as elementary 8 is not too far off (hopefully) can this bug remain until we see if it resolved by upstream updates?

@jeremypw jeremypw marked this pull request as ready for review January 22, 2024 12:34
@TomiOhl
Copy link
Contributor

TomiOhl commented Feb 27, 2024

Up: Mail is practically unusable since September, a weird temporary fix is still better than a broken app, isn't it? Would be happy if you considered releasing this fix

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.

Many mails flash to white after opening
2 participants