![]() |
#91 | |
Guru
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 846
Karma: 3341026
Join Date: Jan 2017
Location: Poland
Device: Various
|
Quote:
The last commits from the end of December changed something, and then nothing, and it doesn't work right now. Then you moved to a new CSS parser, but those improvements to Code View have been discontinued. Inserting tags (B, I, U etc.) works in bare BODY, but not in paragraphs. For example, try to bold or italicize any word in the code: Code:
<p>This is an example text paragraph.</p> |
|
![]() |
![]() |
![]() |
#92 | |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 8,810
Karma: 6000000
Join Date: Nov 2009
Device: many
|
Yes, lots of rework has been done in CV but I obviously broke something. Thank you for letting me know it was broken! Fixing the whole wrong CSS removal took quite a lot of work and so took me away from rewriting the CV editor.
I am on it. Quote:
|
|
![]() |
![]() |
![]() |
#93 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 8,810
Karma: 6000000
Join Date: Nov 2009
Device: many
|
And please do not be shy about bug reports! Fixing bugs always helps improve Sigil and this bug appears to be a doozy.
|
![]() |
![]() |
![]() |
#94 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 8,810
Karma: 6000000
Join Date: Nov 2009
Device: many
|
Hmm... Those functions call ToggleFormatSelection and that is actually one of the routines in CodeViewEditor I have not touched, so I have no idea why it is broken.
|
![]() |
![]() |
![]() |
#95 |
Guru
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 846
Karma: 3341026
Join Date: Jan 2017
Location: Poland
Device: Various
|
I don't remember when exactly it stopped working, but the very first commit on December 18 made the tagging different.
Certainly, this group of commits is suspicious (image). I did not report it before - after all, the work was "in progress", the changes were significant, and I stopped understanding them. |
![]() |
![]() |
![]() |
#96 | |
Hedge Wizard
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 802
Karma: 19999999
Join Date: May 2011
Location: UK/Philippines
Device: Kobo Touch, Nook Simple
|
Quote:
And thank you even more to you and DiapDealer for all the work you put into Sigil. It is much appreciated. |
|
![]() |
![]() |
![]() |
#97 | |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 8,810
Karma: 6000000
Join Date: Nov 2009
Device: many
|
I fixed that ToggleFormatSelection routine to actually make sense and work when it checks if the selection end was in a tag. The original code had an off by one as the end() points *past* the last character selected (much like python substrings ranges are done).
So with this change, all of the Bold, Italic, etc started working for me. I have NO IDEA how the original code actually worked at all. It was broken from the very beginning. I think my actually fixing the detection of being in a tag has exposed this issue. Please try a pull from master and let me know if you still run into any trouble. Thanks! Quote:
Last edited by KevinH; 01-27-2021 at 08:02 PM. |
|
![]() |
![]() |
![]() |
#98 |
Guru
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 846
Karma: 3341026
Join Date: Jan 2017
Location: Poland
Device: Various
|
The issue could only be seen by personally compiling the development build, not the stable build, so it wasn't such a problem. I rarely put tags with B, I, U… so I waited for further work. On the other hand, you rarely commit something that doesn't work later and I was able to get your attention faster.
Back to the recent changes. It's nice, but on December 18th I noticed that adding tags around a word works ALMOST always well:
I am attaching an animated GIF that shows these cases. And here is the test line of text (I marked in red the places where surrounding by html tag does not work). Code:
<p>micro|<b>computer|</b></p> |
![]() |
![]() |
![]() |
#99 |
Fanatic
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 500
Karma: 3498633
Join Date: May 2011
Location: Surrey, UK
Device: Kobo Aura One, Sony PRS 600/650
|
May I respectfully remind of the problem I suffered where words I was trying to add to the default dictionary were in fact being added to a temporary book specific dictionary, and subsequently were deleted when I removed that dictionary.
|
![]() |
![]() |
![]() |
#100 | |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 8,810
Karma: 6000000
Join Date: Nov 2009
Device: many
|
Auto selecting of a word is only designed to work when the cursor is actually in a word not after a word. Your last case the cursor is after the word and nothing is autoselected. That is how it was designed to work. Highlighting a word and then selecting the format is the normal way this is handled. The auto selection of a word *under* the cursor is Qt not us.
The key is that for a word to be under the cursor the cursor must be in one of the positions indicated by the '|' below, not after the word (one spot for each letter). |w|o|r|d So to me it appears to work as expected. What am I missing? Kevin Quote:
|
|
![]() |
![]() |
![]() |
#101 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 8,810
Karma: 6000000
Join Date: Nov 2009
Device: many
|
That code was untouched from earlier versions and I am not seeing that issue, but perhaps I am not reproducing exactly the sequence of steps you do.
So please provide a step by step set of instructions so that I can see exactly what you are seeing. Is all of this being done from the spellcheck dialog itself or from a menu? Are the words all in one language or many? Do you have multiple dictionaries set (ie both Primary and Secondary dictionaries or only primary). Are you changing from default to a new wordlist in the Sigil Preferences or someplace else? How exactly are you telling it which wordlist to use, in the SpellCheck dialog itself? Thanks, Kevin |
![]() |
![]() |
![]() |
#102 | |
Fanatic
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 500
Karma: 3498633
Join Date: May 2011
Location: Surrey, UK
Device: Kobo Aura One, Sony PRS 600/650
|
Quote:
1. Is all of this being done from the spellcheck dialog itself or from a menu? - from the menu that pops-up when you right-click a word. 2. Are the words all in one language or many? - English only. 3. Do you have multiple dictionaries set (ie both Primary and Secondary dictionaries or only primary) - Primary only (English - Great Britain) 4. Are you changing from default to a new wordlist in the Sigil Preferences or someplace else? - Not sure what you mean. I don't think I am changing from default just adding a new wordlist specific to the book I am working on. (img) 5. How exactly are you telling it which wordlist to use, in the SpellCheck dialog itself? - I am not using the spellcheck dialog, this is when working in code-view and right-clicking on an underlined word. When right-clicking on an underlined (misspelled) word you have the following options: 1 - Add To Default Dictionary 2 - Add to Dictionary > (which lists the dictionaries available( img)) If I select option 1 the word does NOT get added to default, but to (as in this case) Bell. Surely if the options 1 and 2 were correct the word should have been added to default NOT Bell. If I remove 'Bell' any words that I then add to default using option 1 are saved correctly. The 'slang' dictionary I added does not seem to have any effect on the operation of saving to default. Hope that makes it a bit clearer. https://www.mobileread.com/forums/at...1&d=1611847769 |
|
![]() |
![]() |
![]() |
#103 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 8,810
Karma: 6000000
Join Date: Nov 2009
Device: many
|
I rarely use the CV menus for spellchecking and was doing all of my testing in the SpellCheck Dialog itself and that all appears to work as expected. I will try to recreate what you are seeing with using just the CV menus. If I can recreate the issue, I should be able to fix it.
Thanks, Kevin |
![]() |
![]() |
![]() |
#104 |
Fanatic
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 500
Karma: 3498633
Join Date: May 2011
Location: Surrey, UK
Device: Kobo Aura One, Sony PRS 600/650
|
Thanks Kevin.
Here is what I do if anyone else wishes to try it. 1. Open any book 2. Go to Edit/Preferences/Spellcheck Dictionaries 3. Click on Add and create new dictionary for the book. 4. Open any chapter. 5. In code-view right-click any underlined misspelt word. 6. Click on Add To Default Dictionary. 7. Check in code-view that word is no longer underlined. 8. Go to Edit/Preferences/Spellcheck Dictionaries and remove the dictionary created at step 3. 9. Close and return to code-view and check word again. If the word is once again underlined it shows that the word was added to the dictionary created for the book and not the default as per the menu. |
![]() |
![]() |
![]() |
#105 | |
Grand Sorcerer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 28,624
Karma: 204624552
Join Date: Jan 2010
Device: Nexus 7, Kindle Fire HD
|
Quote:
And since "Bell" and your default dictionary are in fact one and the same, it is only natural there is no difference between choosing to add to the default user dictionary, or choosing to specifically add it to the "Bell" user dictionary. Selecting a user dictionary in the preferences and then clicking 'OK' makes that dictionary the default directory (check out the tooltip when mousing over a blank area in the user dictionary list). There is no all-time-default user dictionary--even if it happens to be named "default". Just rename it something other than "default" if you want to avoid confusion. Last edited by DiapDealer; 01-28-2021 at 12:08 PM. |
|
![]() |
![]() |
![]() |
|
![]() |
||||
Thread | Thread Starter | Forum | Replies | Last Post |
Touch HD, Future features or dead features? | ElWorm | PocketBook | 4 | 02-07-2018 11:36 AM |
New Plugin Features in the upcoming Sigil 0.8.900 | KevinH | Plugins | 6 | 09-14-2015 02:48 PM |
FAQ>How do I use ... features of the conversion tools? | chaot | Library Management | 2 | 08-28-2012 07:28 AM |
Writer2ePub, Sigil, and mjBookMaker Features | Ransom | Writer2ePub | 1 | 09-21-2011 09:20 AM |
ePub Creation Tools: Sigil vs Oxygen | twedigteam | ePub | 6 | 12-10-2010 03:41 AM |