![]() |
#31 | |
null operator (he/him)
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 20,825
Karma: 27620684
Join Date: Mar 2012
Location: Sydney Australia
Device: none
|
Quote:
BR |
|
![]() |
![]() |
![]() |
#32 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 7,990
Karma: 5449552
Join Date: Nov 2009
Device: many
|
QLineEdit's like those used in Find and Replace do not have a setCursorWidth setting at all, so a QStyledProxy is needed to try something like that in things like that.
I did get setCursorWidth to work in a QPlainTextEdit (CodeView) finally. It is undocumented but it must be set every time a new QTextDocument is set. So I can get a double width insert point text cursor working in CodeView but no luck so far in QTreeViews, QComboBoxes, QLineEdits and other major widgets types. I am still investigating. Last edited by KevinH; 04-11-2024 at 06:30 PM. |
![]() |
![]() |
Advert | |
|
![]() |
#33 |
Junior Member
![]() Posts: 2
Karma: 10
Join Date: Apr 2024
Device: none
|
Hello, I noticed that Article 8 has optimized the renaming efficiency when there are many files, and the speed has been greatly improved. Will the efficiency of adding/deleting files in batches when there are many files be optimized in the future? Also, it takes a long time to set up the Cover page when the file volume is large. The problem.
Many thanks |
![]() |
![]() |
![]() |
#34 | |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 7,990
Karma: 5449552
Join Date: Nov 2009
Device: many
|
Quote:
There is no easy way to speed up the "Add Existing" of huge numbers (thousands) of files as that requires creating thousands of internal Sigil resource objects and loading their contents into QTextDocuments for speed of editing, then constantly editing and reediting the opf thousands of times to create the proper opf manifest, not to mention file copying and reading times. You would be better off using a standalone tool or python script to create a rough opf from those thousands of files and using the FolderIn plugin to load it or even use a create a simple epub from those files and start from that. Obviously using a fast ssd based drive and staying far away from slower network drives helps too. Literally epubs with thousands of images or thousands of files is an extreme use case. Exactly which types of epubs have file counts like that? Even a stats/math textbook that could have thousands of equations as separate images (because it does not properly use mathml) still typically only has less than 100 chapter files of text. Last edited by KevinH; 04-12-2024 at 11:13 AM. |
|
![]() |
![]() |
![]() |
#35 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 7,990
Karma: 5449552
Join Date: Nov 2009
Device: many
|
Okay, after some more experimenting, I now have a way to double the insertion point text cursor width from the default 1 pixel wide to 2 pixels wide which helps to make it easier to detect. This should work in all FindReplace and all QLineEdits too.
Remember the insertion point cursor has to fit between adjacent characters. So the wider it gets the worse it looks when fonts with kerning and tight letter spacing are employed. So going more than 2 pixels is not recommended. It is reasonably painless and seems to work requiring only a few small changes in CodeViewEditor.cpp and main.cpp. So I am thinking of creating a Sigil Main UI Preference Appearance to enable or disable it. Changing this setting would require a Sigil restart since it involves changing the main application style on the fly. So something along the lines of: Code:
[] Make text cursor insertion point more visible I have attached two screen shots. Since they are static, no blinking involved. But look at the end of the xhtml comment at the line in yellow to compare the text cursors before and after. Note: these screencaps appear much larger on the screen when I open them here on Mobileread than the actual on my display so both appear reasonably easy to see but that is not quite how it seems to me. Not sure how to show the difference otherwise. Last edited by KevinH; 04-12-2024 at 11:52 AM. |
![]() |
![]() |
Advert | |
|
![]() |
#36 | |
Grand Sorcerer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 27,803
Karma: 198099188
Join Date: Jan 2010
Device: Nexus 7, Kindle Fire HD
|
Quote:
![]() But I worry that vertical space on the Main UI tab of the Appearances Prefs might be getting tight for space. It would mostly affect laptop users with smaller screens, but still.... There's probably a couple of options to get around it. One would be to create a new Accessibility tab that I could put the new Show Widget Focus pref and this proposed larger insertion point cursor on. Another option would be to remove the Tweak Drag Distance preference. I consider it to be a veritable boat-anchor at this point anyway. I could drop it completely, or if someone can convince me it's being used (Qt5), I could replace it with an environment variable setting for the two people in the world who might find it useful. ![]() |
|
![]() |
![]() |
![]() |
#37 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 7,990
Karma: 5449552
Join Date: Nov 2009
Device: many
|
I like the idea of a new Accessibility tab. Maybe even move ui font sizing there, the icon sizing there, the red focus border, and the larger cursor setting. There might be others.
Just a thought to free up space from other already overcrowded preference tabs. |
![]() |
![]() |
![]() |
#38 |
Evangelist
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 413
Karma: 2289864
Join Date: May 2013
Location: Ontario, Canada
Device: Kindle KB, Oasis, Pop_Os!, Jutoh, Kobo Forma
|
Cursor insertion point more visible: I wold vote Yes (as would most of my age group, I'm sure.)
I did this in Writer quite a while ago, and it makes a huge difference, even when using big text on a huge screen. (The Writer solution is a gtk.css file trick, so it probably wouldn't wouldn't help in the Qt world, I got it from here: https://askubuntu.com/questions/1027...ter-appearance) |
![]() |
![]() |
![]() |
#39 |
Guru
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 735
Karma: 2296998
Join Date: Jan 2017
Location: Poland
Device: Various
|
My two cents on the option "Highlight Widgets with Focus" (applies to Windows, possibly others).
The simple QSS for showing focus has a certain unpleasant side-effect, which applies primarily to the two widgets: QSpinBox and QComboBox. Well, the focus overwrites the default 'Fusion' style and ruins the aesthetics of these controls. The problem has been known for years, but I have not found a simple solution. Link I am not in favor of removing this new option if users find it useful, but it is worth adding a note in the documentation that the aesthetics of the UI will suffer with this option enabled. The Accessibility tab is also a good idea. |
![]() |
![]() |
![]() |
#40 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 7,990
Karma: 5449552
Join Date: Nov 2009
Device: many
|
The FindReplace arrow just shrunk a bit to allow the border. But the spinbox arrows change seems really bad.
I will check things on macOS to see what happens there. |
![]() |
![]() |
![]() |
#41 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 7,990
Karma: 5449552
Join Date: Nov 2009
Device: many
|
@BeckyEbook,
Couldn't we restrict that :focus qss to specific classes leaving out QComboBox and QSpinbox since one highlights the text before the up/down buttons, and the other has a blinking cursor or highlight when it has focus? |
![]() |
![]() |
![]() |
#42 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 7,990
Karma: 5449552
Join Date: Nov 2009
Device: many
|
The opposite problem exists in macOS, as the FindReplace pulldowns look horrible and the QSpinbox is just slightly smaller.
Interesting ... |
![]() |
![]() |
![]() |
#43 | |
Guru
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 735
Karma: 2296998
Join Date: Jan 2017
Location: Poland
Device: Various
|
Quote:
Highlighting focus only on the main windows (BookBrowser, CodeView, Validation Results, TOC, Clips, Preview) will only be a substitute. However, I acknowledge that this is not trivial if indeed this option is to be useful for people with disabilities or those using Tab/Shift+Tab to navigate the Sigil interface. I also come across an interesting issue in Windows, when I switch windows via the Tab key, the focus also switches to the toolbar, but only to one – Automation Lists. |
|
![]() |
![]() |
![]() |
#44 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 7,990
Karma: 5449552
Join Date: Nov 2009
Device: many
|
There are ways to use a paintEvent for each Widget to draw the focus box instead of a stylesheet. All the QDockWidgets already override the paint event now. Perhaps handling it that way might work better.
I will play around a bit just to see. |
![]() |
![]() |
![]() |
#45 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 7,990
Karma: 5449552
Join Date: Nov 2009
Device: many
|
We can supposedly set TabOrder on each Widget to limit where Tab goes according to the docs.
I will try playing around with that. |
![]() |
![]() |
![]() |
|
![]() |
||||
Thread | Thread Starter | Forum | Replies | Last Post |
Icon Redesign for future 1.0 Release of Sigil | shorshe | Sigil | 38 | 06-06-2016 11:29 PM |
Kindle Software Preview Release and calibre 'Fetch News' items | tomsem | Devices | 25 | 07-20-2011 08:02 AM |
Will amazon release another cheaper version of DX in the near future | CalvinL | Amazon Kindle | 6 | 07-31-2010 05:01 PM |
Suggestion For Future Release Of Calibre | clambert | Calibre | 5 | 07-10-2010 01:45 PM |
dotReader alpha release next month with future iLiad support possible | Alexander Turcic | News | 7 | 09-28-2006 12:24 PM |