[Plugin] Access-Aide - help improve epub accessibility - Page 5

BeckyEbook · 02-03-2021, 06:47 PM

IMHO, because on Linux the bookpath is immediately compatible and the problem does not occur.

https://www.geeksforgeeks.org/python...rmcase-method/

DiapDealer · 02-03-2021, 06:49 PM

os.path.normcase makes no changes whatsoever on platforms other than Windows... where it converts everything to lowercase and converts forward slashes to backslashes.

KevinH · 02-03-2021, 06:53 PM

So exactly what I do not want ... so can I just remove them completely on Windows as the url itself is case sensitive. Since I am rebuilding a url (a book root relative url) then normcase is not the correct thing since I am not looking for a file path.

So I only need it when looking for a file on the filesystem, correct?

KevinH · 02-03-2021, 07:06 PM

Just to verify, in plugin.py there are 5 lines that use normcase. Each operates on itself and stores the result back in itself. So if you just delete all 5 lines and try again on Windows, does it all work?

On Windows, if a file in an epub has a name "Chapter.xhtml" on the disk itself, is the file stored as "chapter.xhtml" or "CHAPTER.XHTML".

On macOs there are both case sensitive and case insensitive filesystems. I typically always use a case-sensitive file system. But on macOS, even the case-insensitive file system is case-preserving, so the last access case becomes the full name (at least that the user sees).

DiapDealer · 02-03-2021, 07:09 PM

Quote:

Originally Posted by KevinH

So I only need it when looking for a file on the filesystem, correct?

That would be my take, yes.

DiapDealer · 02-03-2021, 07:34 PM

Quote:

Originally Posted by KevinH

Just to verify, in plugin.py there are 5 lines that use normcase. Each operates on itself and stores the result back in itself. So if you just delete all 5 lines and try again on Windows, does it all work?

It seems to, yes. I still get the spaces added after "<?xml" in the xml header, though. Was that narrowed down to a Windows-only issue?

Quote:

Originally Posted by KevinH

On Windows, if a file in an epub has a name "Chapter.xhtml" on the disk itself, is the file stored as "chapter.xhtml" or "CHAPTER.XHTML".

Not sure I know the answer to that. Do you mean in the zip archive on the disk in general, or when extracted in Sigil's scratch area specifically? Filesystem internals are not at all my forte.

KevinH · 02-03-2021, 10:38 PM

The xml header space is a serialization bug in quickparser.py that is part of the plugin_launcher code. I pushed a fix for that earlier today.

I will remove those five lines and post an official 0.5.2 version of AccessAide.

Thanks for everyone's help!

KevinH · 02-03-2021, 11:19 PM

New version of Access-Aide now available in post 1 of this thread.

Changes include:

v052
- no longer consider whitespace alone a valid xhtml title
- properly look for and handle self-closed title tags in head when looking
to replace titles
- work around for quickparser.py serialization bug in Sigil v1.4.3 and earlier
- no longer strip region off dc:language code
- fix Windows specific bug when epub files use mixed case

Special thank you to BeckyEbook, DiapDealer, Doitsu, and DNSB for tracking this one down and helping out with testing. Of course thanks also go to thiago.eec for reporting the bug in the first place and providing the test case.

thiago.eec · 02-04-2021, 05:27 AM

Thank you all! It's working fine now on Windows.

DiapDealer · 02-04-2021, 06:31 AM

I love a happy ending.

KevinH · 08-20-2021, 02:20 PM

New version 0.5.3 released: See first post of this thread for the plugin zip or my github site.

Changes:

- scaling wide but very thin height images by the gui could cause new scaled height to truncate to 0 pixels incorrectly. Now minimum scaled height is 1 pixel.

Prevent PIL module from throwing an error.

KevinH · 09-17-2021, 11:24 AM

New version 0.5.4 released: See first post of this thread for the plugin zip or my github site.

Changes:

- add support for schema:accessibilityHazard none

The schema:accessibilityHazard set to none indicates you are using only *static* images with no Video and no animated gifs. You need to verify that or update that metadata as appropriate.

genossin · 11-15-2021, 08:44 AM

Dear Kevin Hendrick,
i´ve installed the latest version 054 from github to my sigil installation 1.8.0 on mac os x big sur.
The Plugin is shown under the plugin-section (used the reccomanded zip file). I´ve also checked the box "integrated python-interpreter" . If i start the plugin (which is also shown with the correct icon on the toolbar) it dont work, here the failure-message:

Status: failed

Traceback (most recent call last):
File "/Applications/Sigil.app/Contents/plugin_launchers/python/launcher.py", line 141, in launch
target_script = import(script_module)
File "/Users/anke/Library/Application Support/sigil-ebook/sigil/plugins/Access-Aide/plugin.py", line 24, in
from accessgui import GUIUpdateFromList
File "/Users/anke/Library/Application Support/sigil-ebook/sigil/plugins/Access-Aide/accessgui.py", line 16, in
from PIL import Image, ImageTk
File "site-packages/PIL/ImageTk.py", line 28, in
File "tkinter/init.py", line 36, in
ImportError: dlopen(/Applications/Sigil.app/Contents/Frameworks/Python.framework/Versions/3.8/lib/python3.8/lib-dynload/_tkinter.cpython-38-darwin.so, 2): Library not loaded: /Library/Frameworks/Tcl.framework/Versions/8.6/Tcl
Referenced from: /Applications/Sigil.app/Contents/Frameworks/Python.framework/Versions/3.8/lib/python3.8/lib-dynload/_tkinter.cpython-38-darwin.so
Reason: image not found
Error: dlopen(/Applications/Sigil.app/Contents/Frameworks/Python.framework/Versions/3.8/lib/python3.8/lib-dynload/_tkinter.cpython-38-darwin.so, 2): Library not loaded: /Library/Frameworks/Tcl.framework/Versions/8.6/Tcl
Referenced from: /Applications/Sigil.app/Contents/Frameworks/Python.framework/Versions/3.8/lib/python3.8/lib-dynload/_tkinter.cpython-38-darwin.so
Reason: image not found

maybe its a known issue and you could help? am i doing wrong?
python 3 is existing also: /usr/bin/python3.
p.s.: im using the german version of sigil.

greetings, anke

Doitsu · 11-15-2021, 08:52 AM

macOS users will need to manually install the TclTk library.

Try installing the TclTk_8.6.12.dmg package from the official download website.
If it doesn't work, please post about it in the MacOS ActiveState TCL 8.6.x and Sigil issue thread.

KevinH · 11-15-2021, 10:30 AM

Yes ... the TclTk_8.6.12.dmg has now been updated to work on macOS Catalina and later. The version posted previously only worked on Mojave and earlier. The new version should work on all. If you downloaded TclTk_8.6.12.dmg recently, you should grab the current version posted there now.

02-03-2021, 07:06 PM	#64
KevinH Sigil Developer Posts: 7,654 Karma: 5433388 Join Date: Nov 2009 Device: many	Just to verify, in plugin.py there are 5 lines that use normcase. Each operates on itself and stores the result back in itself. So if you just delete all 5 lines and try again on Windows, does it all work? On Windows, if a file in an epub has a name "Chapter.xhtml" on the disk itself, is the file stored as "chapter.xhtml" or "CHAPTER.XHTML". On macOs there are both case sensitive and case insensitive filesystems. I typically always use a case-sensitive file system. But on macOS, even the case-insensitive file system is case-preserving, so the last access case becomes the full name (at least that the user sees). Last edited by KevinH; 02-03-2021 at 07:11 PM.

09-17-2021, 11:24 AM	#72
KevinH Sigil Developer Posts: 7,654 Karma: 5433388 Join Date: Nov 2009 Device: many	New version 0.5.4 released: See first post of this thread for the plugin zip or my github site. Changes: - add support for schema:accessibilityHazard none The schema:accessibilityHazard set to none indicates you are using only static images with no Video and no animated gifs. You need to verify that or update that metadata as appropriate. Last edited by KevinH; 09-17-2021 at 01:16 PM.

Similar Threads
Thread	Thread Starter	Forum	Replies	Last Post
[Plugin] ACE - DAISY EPUB Accessibility Checker wrapper	Doitsu	Plugins	36	01-26-2024 12:24 PM
Easier access to plugin prefs	slowsmile	Plugins	14	10-22-2017 01:42 PM
Help me improve my EPUB converter	epubler	Workshop	2	07-20-2011 09:15 PM
Epub Revision - accessibility support	Nate the great	ePub	1	02-23-2011 03:47 AM
Kindle 2 Accessibility Access Mode suggestions	scotty1024	Amazon Kindle	4	02-28-2009 06:06 PM

02-03-2021, 06:47 PM	#61
BeckyEbook Guru Posts: 692 Karma: 2180740 Join Date: Jan 2017 Location: Poland Device: Misc	IMHO, because on Linux the bookpath is immediately compatible and the problem does not occur. https://www.geeksforgeeks.org/python...rmcase-method/

02-03-2021, 06:49 PM	#62
DiapDealer Grand Sorcerer Posts: 27,553 Karma: 193191846 Join Date: Jan 2010 Device: Nexus 7, Kindle Fire HD	os.path.normcase makes no changes whatsoever on platforms other than Windows... where it converts everything to lowercase and converts forward slashes to backslashes.

02-03-2021, 06:53 PM	#63
KevinH Sigil Developer Posts: 7,654 Karma: 5433388 Join Date: Nov 2009 Device: many	So exactly what I do not want ... so can I just remove them completely on Windows as the url itself is case sensitive. Since I am rebuilding a url (a book root relative url) then normcase is not the correct thing since I am not looking for a file path. So I only need it when looking for a file on the filesystem, correct?

02-03-2021, 10:38 PM	#67
KevinH Sigil Developer Posts: 7,654 Karma: 5433388 Join Date: Nov 2009 Device: many	The xml header space is a serialization bug in quickparser.py that is part of the plugin_launcher code. I pushed a fix for that earlier today. I will remove those five lines and post an official 0.5.2 version of AccessAide. Thanks for everyone's help!

02-03-2021, 11:19 PM	#68
KevinH Sigil Developer Posts: 7,654 Karma: 5433388 Join Date: Nov 2009 Device: many	New version of Access-Aide now available in post 1 of this thread. Changes include: v052 - no longer consider whitespace alone a valid xhtml title - properly look for and handle self-closed title tags in head when looking to replace titles - work around for quickparser.py serialization bug in Sigil v1.4.3 and earlier - no longer strip region off dc:language code - fix Windows specific bug when epub files use mixed case Special thank you to BeckyEbook, DiapDealer, Doitsu, and DNSB for tracking this one down and helping out with testing. Of course thanks also go to thiago.eec for reporting the bug in the first place and providing the test case.

02-04-2021, 05:27 AM	#69
thiago.eec Guru Posts: 930 Karma: 1177583 Join Date: Dec 2016 Location: Goiânia - Brazil Device: iPad, Kindle Paperwhite	Thank you all! It's working fine now on Windows.

02-04-2021, 06:31 AM	#70
DiapDealer Grand Sorcerer Posts: 27,553 Karma: 193191846 Join Date: Jan 2010 Device: Nexus 7, Kindle Fire HD	I love a happy ending.

08-20-2021, 02:20 PM	#71
KevinH Sigil Developer Posts: 7,654 Karma: 5433388 Join Date: Nov 2009 Device: many	New version 0.5.3 released: See first post of this thread for the plugin zip or my github site. Changes: - scaling wide but very thin height images by the gui could cause new scaled height to truncate to 0 pixels incorrectly. Now minimum scaled height is 1 pixel. Prevent PIL module from throwing an error.

11-15-2021, 08:44 AM	#73
genossin Junior Member Posts: 3 Karma: 10 Join Date: Nov 2021 Device: none	Dear Kevin Hendrick, i´ve installed the latest version 054 from github to my sigil installation 1.8.0 on mac os x big sur. The Plugin is shown under the plugin-section (used the reccomanded zip file). I´ve also checked the box "integrated python-interpreter" . If i start the plugin (which is also shown with the correct icon on the toolbar) it dont work, here the failure-message: Status: failed Traceback (most recent call last): File "/Applications/Sigil.app/Contents/plugin_launchers/python/launcher.py", line 141, in launch target_script = import(script_module) File "/Users/anke/Library/Application Support/sigil-ebook/sigil/plugins/Access-Aide/plugin.py", line 24, in from accessgui import GUIUpdateFromList File "/Users/anke/Library/Application Support/sigil-ebook/sigil/plugins/Access-Aide/accessgui.py", line 16, in from PIL import Image, ImageTk File "site-packages/PIL/ImageTk.py", line 28, in File "tkinter/init.py", line 36, in ImportError: dlopen(/Applications/Sigil.app/Contents/Frameworks/Python.framework/Versions/3.8/lib/python3.8/lib-dynload/_tkinter.cpython-38-darwin.so, 2): Library not loaded: /Library/Frameworks/Tcl.framework/Versions/8.6/Tcl Referenced from: /Applications/Sigil.app/Contents/Frameworks/Python.framework/Versions/3.8/lib/python3.8/lib-dynload/_tkinter.cpython-38-darwin.so Reason: image not found Error: dlopen(/Applications/Sigil.app/Contents/Frameworks/Python.framework/Versions/3.8/lib/python3.8/lib-dynload/_tkinter.cpython-38-darwin.so, 2): Library not loaded: /Library/Frameworks/Tcl.framework/Versions/8.6/Tcl Referenced from: /Applications/Sigil.app/Contents/Frameworks/Python.framework/Versions/3.8/lib/python3.8/lib-dynload/_tkinter.cpython-38-darwin.so Reason: image not found maybe its a known issue and you could help? am i doing wrong? python 3 is existing also: /usr/bin/python3. p.s.: im using the german version of sigil. greetings, anke

11-15-2021, 08:52 AM	#74
Doitsu Grand Sorcerer Posts: 5,584 Karma: 22735033 Join Date: Dec 2010 Device: Kindle PW2	macOS users will need to manually install the TclTk library. Try installing the TclTk_8.6.12.dmg package from the official download website. If it doesn't work, please post about it in the MacOS ActiveState TCL 8.6.x and Sigil issue thread.

Advert

Advert

11-15-2021, 10:30 AM	#75
KevinH Sigil Developer Posts: 7,654 Karma: 5433388 Join Date: Nov 2009 Device: many	Yes ... the TclTk_8.6.12.dmg has now been updated to work on macOS Catalina and later. The version posted previously only worked on Mojave and earlier. The new version should work on all. If you downloaded TclTk_8.6.12.dmg recently, you should grab the current version posted there now.