Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Paragraph-Style for Bullet-Point-Lists not working #77

Open
georgbachmann opened this issue Dec 11, 2018 · 1 comment
Open

Paragraph-Style for Bullet-Point-Lists not working #77

georgbachmann opened this issue Dec 11, 2018 · 1 comment

Comments

@georgbachmann
Copy link

I seems that it's not properly respecting lineSpacings of Bullet-Point-Lists?!?

We are using non-default lineSpacings and noticed that the lists aren't quite the same. So I dug a bit into the code and noticed in ListAttributedStringLayoutBlockBuilder.swift in function getBulletIndentingAttributesForLevel when creating the paragraphStyle the lineSpacing isn't set. I fixed it by adding the line

paragraphStyle.lineSpacing = listStyling?.neededLineHeight() ?? 0

and now it's working as expected.

I guess it's a bug in marky-mark?!?
Also... shouldn't lineHeight in LineHeightStylingRule be called lineSpacing?

@Basca
Copy link
Contributor

Basca commented Jan 20, 2019

Hey georgbachmann,

There is indeed something going wrong with LineSpacing not being correctly set for Lists.
After some more investigating based on your remarks I found that starting from #63 the lineHeight has been added, but no default implementation has been added.

We will look further into this, to see what is more clear naming wise and how this should be setup.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

2 participants