03-17-2023, 03:13 PM | #16 |
Sigil Developer
Posts: 8,115
Karma: 5450818
Join Date: Nov 2009
Device: many
|
Understood. If we leave the whole silly issue of dealing with dark mode to the user to handle *before* using print, and with full mathml support, I am pretty sure we can get down to one WebEngine Widget to keep the memory footprint low.
Removing all the dark mode cruft while allowing user css and mathml was the issue. |
03-17-2023, 03:15 PM | #17 |
Grand Sorcerer
Posts: 27,990
Karma: 199001268
Join Date: Jan 2010
Device: Nexus 7, Kindle Fire HD
|
I see no horrible delays with the new feature on my underpowered, and aging Windows laptop. But then I refuse to work with epubs that have "large" xhtml files.
|
Advert | |
|
03-17-2023, 11:21 PM | #18 | |
Wizard
Posts: 1,606
Karma: 8291219
Join Date: Mar 2013
Location: Rosario - Santa Fe - Argentina
Device: Kindle 4 NT
|
Quote:
|
|
03-18-2023, 09:03 AM | #19 |
Sigil Developer
Posts: 8,115
Karma: 5450818
Join Date: Nov 2009
Device: many
|
Sigil uses a much higher print quality setting, not just screen dpi.
Last edited by KevinH; 03-18-2023 at 09:06 AM. |
03-18-2023, 09:20 AM | #20 |
Grand Sorcerer
Posts: 27,990
Karma: 199001268
Join Date: Jan 2010
Device: Nexus 7, Kindle Fire HD
|
Having to change out of dark mode just to print something might be a little counterintuitive to many users. But if it makes things work better, I'm not opposed. I never print ebooks (or convert them to pdf) anyway.
|
Advert | |
|
03-18-2023, 09:50 AM | #21 |
Wizard
Posts: 1,606
Karma: 8291219
Join Date: Mar 2013
Location: Rosario - Santa Fe - Argentina
Device: Kindle 4 NT
|
As I said, I use dark mode and custom .css; so in PP I have white text on a dark background. But I only have to do a mouse click on the respective icon in PP to put black text on a white background. It seems that you (both Kevin and Diap) forgot that valuable feature also in PP
|
03-18-2023, 09:53 AM | #22 |
Wizard
Posts: 1,606
Karma: 8291219
Join Date: Mar 2013
Location: Rosario - Santa Fe - Argentina
Device: Kindle 4 NT
|
|
03-18-2023, 10:17 AM | #23 |
Grand Sorcerer
Posts: 27,990
Karma: 199001268
Join Date: Jan 2010
Device: Nexus 7, Kindle Fire HD
|
I think we're going to find that the QPrinter::HighResolution in the Preview dialog is the larger culprit of the overall performance hit, here (as opposed to dark mode diddling). On some only moderately long xhtml files (text and links only), my development machine will lock up waiting on the Print Preview dialog. Remove QPrinter::HighResolution and its display is almost instantaneous.
|
03-18-2023, 10:47 AM | #24 |
Sigil Developer
Posts: 8,115
Karma: 5450818
Join Date: Nov 2009
Device: many
|
We can revert that for the next release if you want.
|
03-18-2023, 11:34 AM | #25 |
Wizard
Posts: 1,606
Karma: 8291219
Join Date: Mar 2013
Location: Rosario - Santa Fe - Argentina
Device: Kindle 4 NT
|
Please, do that. It can't be a .pdf (only text) of just 5 pages with a size of 22MB! Can't be possible to set a medium quality of printing (300/600dpi)?
Last edited by RbnJrg; 03-18-2023 at 11:36 AM. |
03-18-2023, 11:37 AM | #26 |
Sigil Developer
Posts: 8,115
Karma: 5450818
Join Date: Nov 2009
Device: many
|
I was actually asking DiapDealer that question.
|
03-18-2023, 12:03 PM | #27 |
Grand Sorcerer
Posts: 27,990
Karma: 199001268
Join Date: Jan 2010
Device: Nexus 7, Kindle Fire HD
|
Definitely seeing some historical evidence of different behavior of QPrinter::HighResolution on Unix compared to Windows/Mac. Perhaps an os-based ifdef is in order.
Also, I can crash Sigil on Arch (invalid pointer) by clicking the print button again while waiting on the print preview dialog from the first click. Interestingly enough, the same action will produce the same rare Gumbo assertion error you just fixed recently (with the 1.9.30 codebase). After the fix, there is only the invalid pointer when clicking the button twice. Last edited by DiapDealer; 03-18-2023 at 12:09 PM. |
03-18-2023, 12:32 PM | #28 |
Sigil Developer
Posts: 8,115
Karma: 5450818
Join Date: Nov 2009
Device: many
|
Yes, we will need an ifdef. On macOS I need the high dpi setting to get any type of quality pdf generated. I think since Qt on the mac uses pseudo pixels to prevent scaling issues, it produces a very poor 72 dpi instead of something reasonable.
|
03-18-2023, 12:34 PM | #29 |
Sigil Developer
Posts: 8,115
Karma: 5450818
Join Date: Nov 2009
Device: many
|
And if we have to redesign webviewprinter a bit, we may want to have it clean up after itself (delete view widget (if we decide to keep using a second widget) once the printing has concluded. That will reduce the memory footprint instead of waiting until the next print to clear it.
Last edited by KevinH; 03-18-2023 at 12:41 PM. |
03-18-2023, 12:43 PM | #30 |
Sigil Developer
Posts: 8,115
Karma: 5450818
Join Date: Nov 2009
Device: many
|
FWIW, I just tried double clicking fast on the Preview print button but only the first click registered and subsequent clicks were properly ignored on macOS. No issues.
|
|
Similar Threads | ||||
Thread | Thread Starter | Forum | Replies | Last Post |
Sigil-0.9.12 Released | DiapDealer | Sigil | 194 | 03-15-2019 03:51 PM |
Sigil-0.9.8 Released | DiapDealer | Sigil | 58 | 05-07-2017 04:56 PM |
Sigil-0.8.900 released for testing - Wait for Sigil-0.8.901 | KevinH | Sigil | 106 | 10-04-2015 10:41 AM |
Sigil 0.8.1 Released | user_none | Sigil | 33 | 10-16-2014 05:04 PM |
Sigil 0.7.4 Released | user_none | Sigil | 47 | 01-05-2014 02:31 PM |