Skip to content
This repository was archived by the owner on Aug 13, 2022. It is now read-only.

FR: Mixed List style #35

Closed
chrisgrieser opened this issue Jan 11, 2022 · 6 comments · Fixed by #44
Closed

FR: Mixed List style #35

chrisgrieser opened this issue Jan 11, 2022 · 6 comments · Fixed by #44
Labels
enhancement New feature or request

Comments

@chrisgrieser
Copy link

Thanks for this plugin! I was about to write a longer dataviewjs query, but this plugin does the job much better anyway :)

I was wondering, is it possible to have an option in the settings / code block for mixed list styles? e.g. H2 numbered, but H3 bullets?

@Aidurber
Copy link
Owner

Interesting, I'm not entirely sure I'll have to look into it, but in the mean time lets work through how it would work @chrisgrieser.

What happens with a structure like this:

H2 - Numbered
H3 - Bullets
H4 - ??
H2 - Numbered

Is it just the highest level going bullets and then numbered?

What would the settings look like for that?

```toc
	style: varied
    outer_style: bullet
```

We'd need a way to:

  1. Differ from the existing styles that control the entire ToC
  2. Specify the "outer" style (bad name)

@Aidurber Aidurber added needs investigation Unsure whether it's a bug or not. Investigate enhancement New feature or request labels Jan 11, 2022
@chrisgrieser
Copy link
Author

I'd say if you select a mixed list style, you have to specify for each heading level whether it is gonna be numbers or bullets.

@Aidurber
Copy link
Owner

I think this would make this project much more complicated, having a different setting for each level. I'll look into doing something just to configure the top most level.

  1. If you set "style" to bullet
  2. And you set "varied_layout" to true
  3. Then the first level will be bullet and the rest will be number and vice versa.

Then if that works, that is our MVP. I can then play around with the implementation to try and find clean way to build this feature.

@Aidurber Aidurber removed the needs investigation Unsure whether it's a bug or not. Investigate label Feb 2, 2022
@chrisgrieser
Copy link
Author

Thx 😄

@Aidurber
Copy link
Owner

Aidurber commented Feb 2, 2022

Released MVP in 0.0.26, let me know if that's all good for you @chrisgrieser

@chrisgrieser
Copy link
Author

works as intended 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants