View Single Post
Old 09-17-2017, 03:27 PM   #13
DiapDealer
Grand Sorcerer
DiapDealer ought to be getting tired of karma fortunes by now.DiapDealer ought to be getting tired of karma fortunes by now.DiapDealer ought to be getting tired of karma fortunes by now.DiapDealer ought to be getting tired of karma fortunes by now.DiapDealer ought to be getting tired of karma fortunes by now.DiapDealer ought to be getting tired of karma fortunes by now.DiapDealer ought to be getting tired of karma fortunes by now.DiapDealer ought to be getting tired of karma fortunes by now.DiapDealer ought to be getting tired of karma fortunes by now.DiapDealer ought to be getting tired of karma fortunes by now.DiapDealer ought to be getting tired of karma fortunes by now.
 
DiapDealer's Avatar
 
Posts: 28,706
Karma: 205039118
Join Date: Jan 2010
Device: Nexus 7, Kindle Fire HD
Quote:
Originally Posted by KevinH View Post
FYI - just pushed a "fix" for this master. It should be a safe change (hope those were not famous last words!)
It performs as expected for EPUB2, but there's no call to QString.simplified() on the EPUB3 side of things. So there's a line-break in the nav, the ToC Widget entry, and ultimately the ncx (if you generate one from he nav).

I've found a fairly central location in Headings.cpp to call .simplified() that should ensure that whitespace gets collapsed properly in all toc/nav/ncx generation. Should I push?
DiapDealer is offline   Reply With Quote