xles / hyphenator

Automatically exported from code.google.com/p/hyphenator
GNU General Public License v3.0
0 stars 0 forks source link

Clarify the purpose of Hyphenator_Loader.js or remove it #184

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
The distribution contains Hyphenator_Loader.js, a small file with 
“@description Checks if there's CSS-hyphenation available for the given 
languages and loads and runs Hyphenator if there's no CSS-hyphenation”. This 
seems unnecessary now that Hyphenator.js has an argument for specifying that 
CSS hyphenation be used when available.

Thus, Hyphenator_Loader.js seems to be a holdover from previous versions and 
could be removed. Alternative, if there is still some intended use for it, it 
should be at least briefly mentioned in the documentation.

Original issue reported on code.google.com by jukk...@gmail.com on 24 May 2013 at 9:18

GoogleCodeExporter commented 9 years ago
No it's not unnecessary. Hyphenator.js has grown to a quite large script file 
and modern browsers don't need it. So Hyphenator_Loader.js only loads 
Hyphenator.js if the browser doesn't support CSS-hyphenation.

But it needs documentation, I see.
Thanks!

Original comment by mathiasn...@gmail.com on 25 May 2013 at 3:27

GoogleCodeExporter commented 9 years ago
Hyphenator_Loader.js is no integrated in MergeAndPack and documented on the 
wiki-HowTo page

Original comment by mathiasn...@gmail.com on 17 Oct 2014 at 7:36