Trojitá plaintext emails are displayed letter by letter

Bug #1799026 reported by Simon Quigley
18
This bug affects 2 people
Affects Status Importance Assigned to Milestone
qtwebkit-opensource-src (Ubuntu)
Fix Released
High
Simon Quigley
Cosmic
Fix Released
High
Simon Quigley

Bug Description

[Impact]

Without this fix, when viewing plaintext emails in Trojitá, each letter is on its own line, making it difficult to read messages at best.

Two fixes from upstream QtWebkit can fix this problem.

[Test Case]

Open any email when signed into an IMAP server with Trojitá, and if the email is an HTML email, select the "Plaintext" tab if it isn't selected already.

It should be displayed normally, with a regular sized font.

[Regression Potential]

Little to none. If another QtWebkit-using package depends on the way this has already worked in an unsafe way, it could break, but this is unlikely.

I could also see how it could potential to break if the email is very, very, very large, because it has to refresh the view, but at most you would expect some slowness.

Revision history for this message
Simon Quigley (tsimonq2) wrote :

Please note that I had to cherry-pick two patches which are already uploaded to Debian that, without them, result in the package FTBFS on [!ppc64el !s390x].

These patches, while unrelated, are trivial enough that the functionality of the package should not be affected.

Changed in qtwebkit-opensource-src (Ubuntu Cosmic):
status: New → In Progress
importance: Undecided → High
assignee: nobody → Simon Quigley (tsimonq2)
Revision history for this message
Brian Murray (brian-murray) wrote : Please test proposed package

Hello Simon, or anyone else affected,

Accepted qtwebkit-opensource-src into cosmic-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/qtwebkit-opensource-src/5.212.0~alpha2-12ubuntu1.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-cosmic to verification-done-cosmic. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-cosmic. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in qtwebkit-opensource-src (Ubuntu Cosmic):
status: In Progress → Fix Committed
tags: added: verification-needed verification-needed-cosmic
Revision history for this message
Simon Quigley (tsimonq2) wrote :

This is quite confusing; even after I uploaded the fix to a PPA, iterated on it before uploading, and confirmed it passed on all architectures, s390x is FTBFS.

I'm subscribing Dmitry to get his thoughts on it, but for some odd reason the current upload isn't good as-is.

Apologies.

Revision history for this message
Dmitry Shachnev (mitya57) wrote :

> c++: fatal error: Killed signal terminated program cc1plus

I have retried it.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package qtwebkit-opensource-src - 5.212.0~alpha2-12ubuntu1.1

---------------
qtwebkit-opensource-src (5.212.0~alpha2-12ubuntu1.1) cosmic; urgency=medium

  * Fix resizing the FrameView, which fixes plaintext emails in Trojitá
    (LP: #1799026).
  * Build with -fpermissive to (hopefully) solve the i386 FTBFS.
  * Update symbols from build logs.

 -- Simon Quigley <email address hidden> Sun, 21 Oct 2018 01:11:18 -0500

Changed in qtwebkit-opensource-src (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
Dan Simmons (kc2bez) wrote :

I have tested the qtwebkit patch in cosmic-proposed. Plaintext emails are now displayed properly.

Revision history for this message
Simon Quigley (tsimonq2) wrote :

I can confirm that this fixes the issue with 5.212.0~alpha2-12ubuntu1.1 in Cosmic as well. Marking the bug tags.

tags: added: verification-done verification-done-cosmic
removed: verification-needed verification-needed-cosmic
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package qtwebkit-opensource-src - 5.212.0~alpha2-12ubuntu1.1

---------------
qtwebkit-opensource-src (5.212.0~alpha2-12ubuntu1.1) cosmic; urgency=medium

  * Fix resizing the FrameView, which fixes plaintext emails in Trojitá
    (LP: #1799026).
  * Build with -fpermissive to (hopefully) solve the i386 FTBFS.
  * Update symbols from build logs.

 -- Simon Quigley <email address hidden> Sun, 21 Oct 2018 01:11:18 -0500

Changed in qtwebkit-opensource-src (Ubuntu Cosmic):
status: Fix Committed → Fix Released
Revision history for this message
Brian Murray (brian-murray) wrote : Update Released

The verification of the Stable Release Update for qtwebkit-opensource-src has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.