View Single Post
Old 09-25-2019, 04:28 PM   #94
droopy
Guru
droopy ought to be getting tired of karma fortunes by now.droopy ought to be getting tired of karma fortunes by now.droopy ought to be getting tired of karma fortunes by now.droopy ought to be getting tired of karma fortunes by now.droopy ought to be getting tired of karma fortunes by now.droopy ought to be getting tired of karma fortunes by now.droopy ought to be getting tired of karma fortunes by now.droopy ought to be getting tired of karma fortunes by now.droopy ought to be getting tired of karma fortunes by now.droopy ought to be getting tired of karma fortunes by now.droopy ought to be getting tired of karma fortunes by now.
 
Posts: 834
Karma: 2912460
Join Date: Apr 2009
Device: Kobo Forma
What did I do wrong?


Line 505 is
Code:
    # body, p { line-height: %1 ... }  -->  body,div,p{line-height:%1 ... }
I did not mess around with line 505, though I did make a change with line 106 (I uncommented it).


The original (unedited) code reads
Code:
Force user line spacing in KePubs:
  - Enabled: no
  - Description: |
      This patch will allow the line spacing set by the slider to take effect
      in some problem KePub books with fixed linespacing (including ones where the
      publisher has used <div> instead of <p> for paragraphs, or wrapped each
      paragraph in a <span> and set the line-height on the span.) However it might
      also override some line spacing that would better be left fixed, e.g. it can
      cause problems with the spacing of paragraphs that begin with a raise-cap.
    # body, p { line-height: %1 ... }  -->  body *  { line-height: %1 ... }
  - FindBaseAddressString: "body, p { line-height: %1"
  - ReplaceString: {Find: "body, p {", Replace: "body *  {", MustMatchLength: yes}
    #
    # Option: Comment out the replace_string line above and uncomment the one below
    # for a less forceful version of this patch that has fewer side effects and
    # should still work for most problem books, but not the ones where the publisher
    # has set the line-height at <span> level.
    #
    # body, p { line-height: %1 ... }  -->  body,div,p{line-height:%1 ... }
# - ReplaceString: {Find: "body, p { line-height: %", Replace: "body,div,p{line-height:%"}
I changed it to

Code:
Force user line spacing in KePubs:
  - Enabled: yes
  - Description: |
      This patch will allow the line spacing set by the slider to take effect
      in some problem KePub books with fixed linespacing (including ones where the
      publisher has used <div> instead of <p> for paragraphs, or wrapped each
      paragraph in a <span> and set the line-height on the span.) However it might
      also override some line spacing that would better be left fixed, e.g. it can
      cause problems with the spacing of paragraphs that begin with a raise-cap.
    # body, p { line-height: %1 ... }  -->  body *  { line-height: %1 ... }
  - FindBaseAddressString: "body, p { line-height: %1"
 # - ReplaceString: {Find: "body, p {", Replace: "body *  {", MustMatchLength: yes}
    #
    # Option: Comment out the replace_string line above and uncomment the one below
    # for a less forceful version of this patch that has fewer side effects and
    # should still work for most problem books, but not the ones where the publisher
    # has set the line-height at <span> level.
    #
    # body, p { line-height: %1 ... }  -->  body,div,p{line-height:%1 ... }
 - ReplaceString: {Find: "body, p { line-height: %", Replace: "body,div,p{line-height:%"}
droopy is offline   Reply With Quote