-
Notifications
You must be signed in to change notification settings - Fork 4
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
permit non-ASCII characters for rule #6
Comments
I wasn’t the initial author and I’d have done it differently as well. My work with Raku (and my initial beginnings as a programmer in 90’s-era Haskell) would have pushed me towards a more meta-programmatic approach. Not to mention, when you raised the staleness issue with me in email and I first took a look, my immediate thought was that there should be an option for Unicode box drawing glyphs. What is your approach? I fear two complications, though, not difficult, I think, but ones needing awareness:
|
Oh, I didn’t actually answer your question, hah—I was asking your approach out of curiosity; assuming it doesn’t conflict with what I’m doing to get the library up to modern snuff generally (I’ll post an issue and tracking milestones on that later today and tag you) and incorporating the 3 extant PR’s, I say please, go ahead, and thank you for the courtesy. |
Heya, haven't heard back about this. I don't know my approach yet, but probably it's "make an attribute or attributes that store/s the characters used for I am holding off on this until I know what's going ahead, and especially with the three outstanding PRs already. |
I'd like to use the
BOX DRAWINGS
characters to get a nice solid set of rule lines in my terminal. It turns out that this isn't entirely straightforward because of how the existing dash and pipe characters are used both to represent themselves and the abstract "rule".If I do this, it'll likely require a bit of change to how the library represents its own configuration, so I am posting this as a "Shall I proceed?" before proceeding.
The text was updated successfully, but these errors were encountered: