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

[Merged by Bors] - Add ‘knobby’ and ‘dots’ pipe kinds #36

Closed
wants to merge 1 commit into from

Conversation

lunacookies
Copy link
Collaborator

No description provided.

@lunacookies lunacookies requested a review from lhvy March 4, 2021 05:53
@lunacookies
Copy link
Collaborator Author

Ideally dots would have a width of 2, just like emoji, as the horizontal density of dots is currently too high.

Screen Shot 2021-03-04 at 4 54 30 pm

I’ll make that change once this PR is merged.

@lunacookies
Copy link
Collaborator Author

I tried braille characters, but it doesn’t look very good
Screen Shot 2021-03-04 at 5 00 50 pm

@lunacookies
Copy link
Collaborator Author

I thought emoji arrows would be cool, but it seems that they are two code points, meaning they have to be represented as strings instead of individual characters. In a separate PR I’ll convert all the pipe kind code to use strs instead of chars

Screen Shot 2021-03-04 at 5 02 46 pm

Screen Shot 2021-03-04 at 5 03 49 pm

Also, I’m not sure why my terminal isn’t rendering the emoji arrows as emoji.

@lhvy
Copy link
Owner

lhvy commented Mar 4, 2021

Proposed changes for after this PR sound good!

@lhvy
Copy link
Owner

lhvy commented Mar 4, 2021

bors r+

bors bot pushed a commit that referenced this pull request Mar 4, 2021
@bors
Copy link

bors bot commented Mar 4, 2021

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title Add ‘knobby’ and ‘dots’ pipe kinds [Merged by Bors] - Add ‘knobby’ and ‘dots’ pipe kinds Mar 4, 2021
@bors bors bot closed this Mar 4, 2021
@bors bors bot deleted the pipe-kinds branch March 4, 2021 06:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants