|
|
#61 | |
|
Grand Sorcerer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 29,244
Karma: 211540138
Join Date: Jan 2010
Device: Nexus 7, Kindle Fire HD
|
Quote:
|
|
|
|
|
|
|
#62 |
|
Guru
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 959
Karma: 3501880
Join Date: Jan 2017
Location: Poland
Device: Various
|
|
|
|
|
|
|
#63 |
|
Bibliophagist
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 50,899
Karma: 178402710
Join Date: Jul 2010
Location: Vancouver
Device: Kobo Sage, Libra Colour, Lenovo M8 FHD, Paperwhite 4, Tolino epos
|
I'd vote for enabled by default. If you don't like/don't want it, you can always turn it off.
|
|
|
|
|
|
#64 | |
|
null operator (he/him)
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 22,539
Karma: 32600658
Join Date: Mar 2012
Location: Sydney Australia
Device: none
|
Quote:
BR |
|
|
|
|
|
|
#65 |
|
Well trained by Cats
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 31,639
Karma: 64144480
Join Date: Aug 2009
Location: The Central Coast of California
Device: Kobo Libra2,Kobo Aura2v1, K4NT(Fixed: New Bat.), Galaxy Tab A
|
|
|
|
|
|
|
#66 |
|
Bibliophagist
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 50,899
Karma: 178402710
Join Date: Jul 2010
Location: Vancouver
Device: Kobo Sage, Libra Colour, Lenovo M8 FHD, Paperwhite 4, Tolino epos
|
It seems that the responses have been to enable by default. We seem to have a winner, folks!
|
|
|
|
|
|
#67 |
|
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 9,479
Karma: 6734148
Join Date: Nov 2009
Device: many
|
I am tied up to night, but I hope to push Becky's visual/theme improvements and add the new pref setting tomorrow. If so, I will push it to master and post here. Thanks everyone for your feedback.
|
|
|
|
|
|
#68 |
|
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 9,479
Karma: 6734148
Join Date: Nov 2009
Device: many
|
Okay, I have added the code to make a new Sigil Preference under General Settings -> Basics
See the attached screenshot I have also added all the support code to use it to show and hide the drop zone. I have pushed all of this to master along with a fix to make the ClipEditor Number field non-editable (it is auto generated), and BeckyEbooks improvements to the DropZone. I must say that adding anything to Sigil Preferences is really getting cumbersome. The form files are already overcrowded, each preference dialogs code needs to be changed, the returns from using Preferences must be added, the setting store key and access functions must be added, etc. It is a major pain just to add one preference. I am beginning to wonder if there is a better way to handle Preferences for something as complex as Sigil. So this brings me up to date with New Suggestions/Features that have been accepted so far this cycle. I am still unsure if adding basic image editing is truly needed given we have "open with". I am still open to hear arguments one way or the other but right now I am leaning towards not adding it. Although Qt and the QTransform class makes these things easier than ever but keeping a history of each image change so that it can be properly undone is a bit of a pain and a potential memory hog. Using an external program really helps with all of that as upon close all its memory is freed. |
|
|
|
|
|
#69 |
|
Bibliophagist
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 50,899
Karma: 178402710
Join Date: Jul 2010
Location: Vancouver
Device: Kobo Sage, Libra Colour, Lenovo M8 FHD, Paperwhite 4, Tolino epos
|
If anyone is interested in testing the current changes to Sigil under Windows, my compile of 2.7.6 can be downloaded from Sigil-2.7.6-Windows-x64-Setup.exe.
|
|
|
|
|
|
#70 |
|
Guru
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 959
Karma: 3501880
Join Date: Jan 2017
Location: Poland
Device: Various
|
I’d like to come back to the suggestions from the previous thread. As these are three separate issues, I’ll cover them in three separate posts.
1. The idea of possibly resizing the image to fit the window and moving the image data to the top. I know that @KevinH is partly against this idea, but I’ve experimented with it and have been using this resizing for a month now. I won’t try to force anyone’s hand; I’m simply putting the idea forward for consideration. Dimensions at the top are a nice feature, and I think we agree on that. As for image fitting – that depends on your working style and what you want to see. If you prefer the original size, we stick with the current solution; if you want the whole image, it might be worth testing. The image is displayed in two situations: 1. Double-clicking the image file in the BookBrowser window. 2. Calling Go To Link from the image tag in the Code View window. This could probably be solved using SVG; I’ve simply changed the styling. Spoiler:
Spoiler:
|
|
|
|
|
|
#71 | |||
|
Guru
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 959
Karma: 3501880
Join Date: Jan 2017
Location: Poland
Device: Various
|
2. The second issue concerns the possible adjustment of the active tab in Code View (and, incidentally, in other places).
Question: Quote:
Quote:
Quote:
The situation is therefore as follows. If we were to remove the styling from the close button in Sigil’s classic dark mode, the active tab would get the default red close button, which would certainly stand out more. This is how it looks now: This is how it looks after disabling the close button styling: This is a good solution, but at the same time a bad solution. It’s not just about my personal feelings, but those red and grey buttons do clash a bit with the classic dark theme. If someone really wanted to, they could disable the classic dark theme in the preferences and have those red close buttons for tabs. However, I see another solution here, which I hope will be of interest. Admittedly, styling bookmarks via QSS is a bit of a pain, as almost every modification forces you to restyle numerous elements. I’ve been wrestling with this issue for ages and thought to myself: I’ll share my suggestion, and if anyone finds the topic important, they can modify QSS to suit their preferences. Here’s my suggestion (add to file qt_styles.qss): Code:
QTabWidget::tab-bar:top {
top: 1px;
}
QTabWidget::tab-bar:bottom {
bottom: 1px;
}
QTabBar::tab:top {
border-top: 1px solid #242424;
border-left: 1px solid #242424;
border-right: 1px solid #242424;
border-top-left-radius: 4px;
border-top-right-radius: 4px;
}
QTabBar::tab:bottom {
border-bottom: 1px solid #242424;
border-left: 1px solid #242424;
border-right: 1px solid #242424;
border-bottom-left-radius: 4px;
border-bottom-right-radius: 4px;
}
QTabBar::tab:selected {
background: #242424;
}
QTabBar::tab:!selected {
background: #353535;
}
QTabBar::tab:!selected:hover {
background: #242424;
}
QTabBar::tab:top:!selected {
margin-top: 3px;
}
QTabBar::tab:bottom:!selected {
margin-bottom: 3px;
}
QTabBar::tab:top, QTabBar::tab:bottom {
min-width: 8ex;
margin-right: -1px;
padding: 5px 10px 5px 10px;
}
QTabBar::tab:top:selected {
border-top: 3px solid #308CC6;
border-bottom-color: none;
}
QTabBar::tab:bottom:selected {
border-bottom: 3px solid #308CC6;
border-top-color: none;
}
QTabBar::tab:top:last, QTabBar::tab:bottom:last,
QTabBar::tab:top:only-one, QTabBar::tab:bottom:only-one {
margin-right: 0;
}
Last edited by BeckyEbook; Today at 05:21 PM. |
|||
|
|
|
|
|
#72 |
|
Guru
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 959
Karma: 3501880
Join Date: Jan 2017
Location: Poland
Device: Various
|
3. The third issue is a minor one.
Every time I open the image file details in BookBrowser, that huge multiplication symbol annoys me. Could we change it to the standard × symbol? |
|
|
|
|
|
#73 | |
|
Grand Sorcerer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 29,244
Karma: 211540138
Join Date: Jan 2010
Device: Nexus 7, Kindle Fire HD
|
RE #2
Quote:
TL;DR: why is it so important that this stand out more? Last edited by DiapDealer; Today at 06:32 PM. |
|
|
|
|
|
|
#74 |
|
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 9,479
Karma: 6734148
Join Date: Nov 2009
Device: many
|
Some thoughts:
- I can change the multiply symbol in the Image Tabs and Get Info. Just tell me exactly which Unicode char you want as a replacement. - I just can *not* see any logical reason to move the image details to be above the image, especially if the full image is zoomed to fit. So I really need to hear a good argument that lays out what benefit is derived from such a move. (And matching calibre in this regard is not a good enough reason). - If most people want zoomed to fit images, we can easily do that with simple svg in the Image Tabs and elsewhere. But certainly seeing tiny images will not show you how truly large some images are which are ones an epub developer should be focusing on. That is why I like the current approach. - as for highlighting the current tab on Windowz, I still see that as a windows only issue with Sigil's classic dark mode, that could be handled by changing the stylesheets used by Windows in classic dark mode. So I will do whatever you and DiapDealer agree should be done. Last edited by KevinH; Today at 09:11 PM. |
|
|
|
![]() |
|
Similar Threads
|
||||
| Thread | Thread Starter | Forum | Replies | Last Post |
| Suggestions for Sigil Improvements or New Features | KevinH | Sigil | 241 | 03-02-2026 01:09 PM |
| Suggestions for Sigil Improvements or New Features | KevinH | Sigil | 168 | 11-18-2025 12:23 PM |
| Ideas for New Improvements or Features | KevinH | Sigil | 97 | 04-09-2025 09:45 AM |
| What Features or Tools does Sigil Still Need Yet? | KevinH | Sigil | 175 | 03-06-2021 09:11 PM |
| Improvements/bugs/features | dontcrash | KOReader | 10 | 10-23-2019 10:05 AM |