search

jmacdonald / amp

A complete text editor for your terminal.
https://amp.rs
Other
3.67k stars 105 forks source link

Allow specifying multiple values for `line_length_guide` in config #280

Closed christoph-heiss closed 4 months ago

christoph-heiss commented 4 months ago

Hi,

long time no see :^)

Anyway, had this branch still laying around from #210, so I thought I'd quickly rebase it before it completely rots away.

Preferences now returns a Vec instead of an Option. As for the configuration itself, it still allows a single value as before, but also an array now. The only change from the previous PR is a change required by the new justify code, but that's pretty trivial.

jmacdonald commented 4 months ago

Hey @christoph-heiss! Thanks for submitting this PR. 🍻

The code changes here look great, but can I bother you to update the associated documentation as part of this change, too?