astoff/code-cells.el

Change the default face for cell separators

Closed this issue · 3 comments

This default:

code-cells-header-line '((t :extend t :inherit header-line))

is rather problematic:

  1. It makes difficult to tell windows from cells inside a window.
  2. Flashy visual bells may be applied when errors are reported.
Screen.Recording.2021-10-12.at.14.38.22.mov

What do you think of

code-cells-header-line '((t :underline t :extend t :inherit font-lock-comment-face)

image

Looks pretty cool to me and I believe it's a safer default.

Looks better indeed, but what do you think of an overline instead? To me the comment marker is part of the cell, so it makes sense to put the division above it.

That works for me also.

To me the comment marker is part of the cell, so it makes sense to put the division above it.

This makes sense. I was thinking of the text as a title to underline, but many times there is no text at all, so your suggestion is better.