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:%"}