Hm... I also use Fira Code in my terminal (on Linux) and do not have any problems with that symbol.

Edit: works in Alacritty, too

from hexyl.

Weird - it does seem to be the font though - this is what I see in the Font Settings control panel (and this also shows I have what seems to be the latest version of the font):

from hexyl.

Does this also appear with other fonts?

from hexyl.

I only tried Consolas and that rendered correctly.

from hexyl.

If there is really a general issue with this character, I'm certainly okay with changing this. However, it seems strange to me to change that character just because your font setup seems to be broken.

from hexyl.

Here are some more data points for the terminal emulators Terminal and iTerm2 on macOS. In all of the tested fonts, the character is displayed. In some fonts its alignment is wrong, in others it is correct. The choice of terminal emulator does not make a difference to whether the character is correctly aligned.

I draw no conclusion from this data on whether the character should be changed.

Fonts in which the character is misaligned

16pt Consolas

Terminal

iTerm2

16pt Fira Code

Terminal

iTerm2

18pt Ubuntu Mono

Terminal

iTerm2

Fonts in which the character is aligned correctly

16pt DejaVu Sans Mono

Terminal

iTerm2

16pt Menlo

Terminal

iTerm2

from hexyl.

@roryokane Thank you!

If somebody wants to suggest a different character (or maybe just remove the additional vertical lines), I'd be ok with changing this.

from hexyl.

Maybe a CLI flag to disable ┊?

Or a --borders={fancy,simple,none} flag, where fancy is the curent default, simple either removes ┊ or replaces it with |, and none disables both horizontal and vertical borders (I personally question the usefulness of the bordering, and would alias hexyl='command hexyl --borders=none immediately ;-)).

from hexyl.

What do others think? Should we just remove the borders completely? Could somebody provide a screenshot of how that would look like?

from hexyl.

Here's a mockup quickly cobbled together:

Note, this only removed horizontal borders and replaced the vertical ones with a space.

from hexyl.

Closed via #54 by @dmke.

from hexyl.