|
|
#61 | |
|
Grand Sorcerer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 29,247
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,906
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,545
Karma: 32600660
Join Date: Mar 2012
Location: Sydney Australia
Device: none
|
Quote:
BR |
|
|
|
|
|
|
#65 |
|
Well trained by Cats
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 31,641
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,906
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,906
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 Link Removed see message #79.
Last edited by DNSB; Today at 12:21 AM. Reason: Removed outdated download link. |
|
|
|
|
|
#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; Yesterday 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,247
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; Yesterday 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. The current version is generated in ResourceObjects/ImageResource.cpp and as far as I can tell it is using the × character which is defined to be: U+00D7. - 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 Windows, 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 inside Sigil. So I will do whatever you and DiapDealer agree should be done. Last edited by KevinH; Yesterday at 09:52 PM. |
|
|
|
|
|
#75 |
|
A Hairy Wizard
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 3,470
Karma: 21000001
Join Date: Dec 2012
Location: Charleston, SC today
Device: iPhone 15/11/X/6/iPad 1,2,Air & Air Pro/Surface Pro/Kindle PW & Fire
|
- I like seeing the image at its full size. Having to scroll all the way to the bottom of the screen to find the image dimensions wears out my mouse wheel… so I suppose you could say there is a monetary incentive to have the info at the top of the image…
isn’t it possible to fit the screen with alt-0 or am I confusing that with photoshop?? Perhaps ctrl-scroll zooms the image???- finding the active tab becomes a little more difficult (for me) in the following scenario: I have several sheets open with multiple tabs across the top. I need to work on adjusting the style for a particular page so I open the css file which becomes the active tab. The preview pane updates as I make css changes which is very convenient. Now I need to go back to the code view of that html sheet…. Which one of the many tabs is it??? I find it by looking at the info header at the top of the preview pane to get the filename then either double click the file in the book browser, or scroll through the open tabs looking for the filename. Not crazy, but would definitely be an improvement if there was an easier way…. Perhaps double clicking the preview pane or something ?? |
|
|
|
![]() |
| Thread Tools | Search this Thread |
|
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 |