theming suggestions #49

New Issue

cycomanic · 2020-04-04T19:24:23+03:00

cycomanic commented

2020-04-04 19:24:23 +03:00

I'm just trying to create a theme that fits my use-case. And I noticed two things that would be nice to have. Not sure how easy they are to implement.

It would be cool if it was possible to have italic text.
One issue I encountered the compact list view is, the ability to theme odd/even is very nice. However if there is a lot of unseen messages in the same view, the odd/even distinction is gone. It would be great if one could only specify a single attribute, i.e. bold font, and it would automatically inherit from what the view was before. So if I for example make the background white for odd and black for even, and then unseen messages I make bold. The backgrounds would all still alternate between black/white even for the unseen messages.

I'm just trying to create a theme that fits my use-case. And I noticed two things that would be nice to have. Not sure how easy they are to implement. 1. It would be cool if it was possible to have italic text. 2. One issue I encountered the compact list view is, the ability to theme odd/even is very nice. However if there is a lot of unseen messages in the same view, the odd/even distinction is gone. It would be great if one could only specify a single attribute, i.e. bold font, and it would automatically inherit from what the view was before. So if I for example make the background white for odd and black for even, and then unseen messages I make bold. The backgrounds would all still alternate between black/white even for the unseen messages.

Manos Pitsidianakis commented

2020-04-05 14:46:39 +03:00

These are good suggestions!

It would be cool if it was possible to have italic text.

While at it, there should also be dimmed, blink and hidden text. These are all in the xterm escape code sequences but may not be supported in all terminals.

o if I for example make the background white for odd and black for even, and then unseen messages I make bold. The backgrounds would all still alternate between black/white even for the unseen messages.

The problem with this is that there's no distinction for a user-set value vs app-default in themes, so there's no way to know if for example the unseen even background should default to even background unless user has set the unseen even background value.

Instead the following is cleaner and easy to implement: Have separate theme keys for even unseen, odd unseen, even highlighted, etc. It's a bit verbose put provides more flexibility for custom themes.

These are good suggestions! > It would be cool if it was possible to have italic text. While at it, there should also be dimmed, blink and hidden text. These are all in the xterm escape code sequences but may not be supported in all terminals. > o if I for example make the background white for odd and black for even, and then unseen messages I make bold. The backgrounds would all still alternate between black/white even for the unseen messages. The problem with this is that there's no distinction for a user-set value vs app-default in themes, so there's no way to know if for example the unseen even background should default to even background unless user has set the unseen even background value. Instead the following is cleaner and easy to implement: Have separate theme keys for `even unseen`, `odd unseen`, `even highlighted`, etc. It's a bit verbose put provides more flexibility for custom themes.

Manos Pitsidianakis added the

enhancement

label 2020-04-05 14:46:54 +03:00

Manos Pitsidianakis referenced this issue from a commit

2020-04-05 14:58:10 +03:00

Add Italics, Blink, Dim and Hidden text attributes Text attributes have been rewritten as bit flags, so for example instead of "BoldUnderline" you'd have to define "Bold | Underline" in your theme settings. Requested in #21

Manos Pitsidianakis referenced this issue from a commit

2020-04-05 14:58:10 +03:00

themes: add {even,odd}_unseen, {even,odd}_selected, {even,odd}_highlighted Suggested in #21

Manos Pitsidianakis referenced this issue from a commit

2020-04-05 15:57:18 +03:00

Add Italics, Blink, Dim and Hidden text attributes Text attributes have been rewritten as bit flags, so for example instead of "BoldUnderline" you'd have to define "Bold | Underline" in your theme settings. Requested in #21

Manos Pitsidianakis referenced this issue from a commit

2020-04-05 15:57:18 +03:00

themes: add {even,odd}_unseen, {even,odd}_selected, {even,odd}_highlighted Suggested in #21

Manos Pitsidianakis commented

2020-04-05 20:22:08 +03:00

Here's a preview

even_odd_unseen_bold_italics.png

52 KiB

cycomanic commented

2020-04-05 22:07:49 +03:00

Poster

The problem with this is that there's no distinction for a user-set value vs app-default in themes, so there's no way to know if for example the unseen even background should default to even background unless user has set the unseen even background value.

Ok I understand. Maybe the option to avoid the verbosity is to have a keyword parent? And then make a bit more hierachy, i.e. have list.odd.unseen , list.even.unseen, list.even.selected ... if one would then set list.odd.unseen = {fg="parent", bg="white"} it takes the foreground value from the parent, i.e. list.odd?

Obviously what you did works as well I'm just thinking about a way to avoid having to repeat a lot of values.

> The problem with this is that there's no distinction for a user-set value vs app-default in themes, so there's no way to know if for example the unseen even background should default to even background unless user has set the unseen even background value. Ok I understand. Maybe the option to avoid the verbosity is to have a keyword parent? And then make a bit more hierachy, i.e. have list.odd.unseen , list.even.unseen, list.even.selected ... if one would then set list.odd.unseen = {fg="parent", bg="white"} it takes the foreground value from the parent, i.e. list.odd? Obviously what you did works as well I'm just thinking about a way to avoid having to repeat a lot of values.

Manos Pitsidianakis commented

2020-04-05 22:35:13 +03:00

I think this is essentially the same thing but with different key names, isn't it?

cycomanic commented

2020-04-06 00:08:54 +03:00

Poster

Actually, you're right it does not make a difference, because it does not really matter if you put in a color or the "parent" keyword. It would be nice to be able to define custom color names though, that way it's easier to reuse colors, without making mistakes.

Manos Pitsidianakis commented

2020-04-06 10:25:03 +03:00

That'd be useful indeed. TOML doesn't allow variables though so this would require some non-trivial rewrite, unless it can become a m4 macro and expand the colour names in preprocess. I'll look into it.

Manos Pitsidianakis closed this issue

2020-04-06 10:25:11 +03:00

Sign in to join this conversation.