![]() |
#1 |
Grand Sorcerer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 28,548
Karma: 204127028
Join Date: Jan 2010
Device: Nexus 7, Kindle Fire HD
|
PageEdit-1.6.0 Released
PageEdit-1.6.0
PageEdit-1.6.0 is a minor update that represents 3 bug fixes and a single new feature. New Features:
Bug Fixes:
--------------------------------- Notes ----------------------------------- The PageEdit user guide has been updated and can always be downloaded from its own repository. Mac users should also check out the wiki entry on the New Release File Format The binary downloads (and source) can be found as assets at the bottom of The PageEdit Github Release page. |
![]() |
![]() |
![]() |
#2 |
Grand Sorcerer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 28,548
Karma: 204127028
Join Date: Jan 2010
Device: Nexus 7, Kindle Fire HD
|
Post reserved for future use.
|
![]() |
![]() |
Advert | |
|
![]() |
#3 |
Chalut o/
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 439
Karma: 145424
Join Date: Dec 2017
Device: Kobo
|
[bug]
When you create a hyperlink to a web address (via the dialog box), PageEdit creates this <a href="../http://www.creepypasta.com/call-me-tomorrow-okay"> Element that may have an influence : The XHTML page is in a subfolder, displayed "12/Section32.xhtml" in the dropdown menu of the loaded elements. |
![]() |
![]() |
![]() |
#4 | |
Grand Sorcerer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 28,548
Karma: 204127028
Join Date: Jan 2010
Device: Nexus 7, Kindle Fire HD
|
Quote:
|
|
![]() |
![]() |
![]() |
#5 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 8,735
Karma: 5703586
Join Date: Nov 2009
Device: many
|
Most of that link code is copy-pasted directly from Sigil source. Does Sigil display the same bug?
|
![]() |
![]() |
Advert | |
|
![]() |
#6 |
Grand Sorcerer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 28,548
Karma: 204127028
Join Date: Jan 2010
Device: Nexus 7, Kindle Fire HD
|
No. It doesn't seem to.
Also note that PageEdit includes the following as an available target in the Insert Link dialog when in dark mode: Code:
Targets in Book | Text #PageEdit_Injected :root { background-color: #2a2a2a; color: #eeeee |
![]() |
![]() |
![]() |
#7 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 8,735
Karma: 5703586
Join Date: Nov 2009
Device: many
|
Hmm ... I am seeing the following:
1. I took and epub in Sigil, fired up PageEdit using the external editor button. 2. Selected some text that will become the link text 3. Hit the chain link icon to create the link 4. Did not select anything in the table and instead in the target I typed: https://www.mobilreread.com and it worked but if I instead type: https://www.mobileread.com/forums I get a dialog message that you can not insert a link at this point. So literally external links are not being properly handled. I will look into it. |
![]() |
![]() |
![]() |
#8 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 8,735
Karma: 5703586
Join Date: Nov 2009
Device: many
|
Yes I see the same thing with the injected root id for dark mode
which we added to make sure we remove anything injected properly and uniquely otherwise it will become permanent when leaving PageEdit. |
![]() |
![]() |
![]() |
#9 |
Grand Sorcerer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 28,548
Karma: 204127028
Join Date: Jan 2010
Device: Nexus 7, Kindle Fire HD
|
Ahh... makes sense.
|
![]() |
![]() |
![]() |
#10 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 8,735
Karma: 5703586
Join Date: Nov 2009
Device: many
|
Okay, I pushed a fix for this to PageEdit master that will now make it handle any and all external links (any non-relative target - ie. has ':' ) as special cases and therefore not try to build a relative link from the source.
Tested it with https://www.mobileread.com/forums and once PageEdit was in preview mode, the link worked just fine. Triple checked with the Inspector and it should now work. I still have to screen out #PageEdit_Injected from the SelectHyperlink dialog building. |
![]() |
![]() |
![]() |
#11 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 8,735
Karma: 5703586
Join Date: Nov 2009
Device: many
|
Okay, modified to SeletHyperlink to skip PageEdit_Injected ids when making the table of possible targets.
Pushed it to master. So we should be good to go here. Thank you for your bug report! |
![]() |
![]() |
![]() |
|
![]() |
||||
Thread | Thread Starter | Forum | Replies | Last Post |
PageEdit-1.2.0 Released | DiapDealer | Sigil | 13 | 04-02-2020 04:16 AM |
PageEdit-0.9.6 Released | DiapDealer | Sigil | 22 | 01-30-2020 09:00 PM |
PageEdit-0.9.5 Released | DiapDealer | Sigil | 46 | 10-26-2019 06:55 PM |
PageEdit-0.8.0 Released | DiapDealer | Sigil | 37 | 08-25-2019 04:14 PM |
PageEdit-0.7.1 Released | DiapDealer | Sigil | 37 | 08-09-2019 02:23 AM |