Skip to content

Conversation

@proycon
Copy link

@proycon proycon commented Dec 19, 2022

This is a continuation on top of #218 (and includes it) , I'm adding support for colours to indicate toot types, but to maintain readability I needed to find an alternative selection method than setting a background colour, so I implemented one (opt-in) that makes the selected item bold and also underlines when on the focussed pane.

This still needs some fine-tuning (code cleanup etc). Screenshot of how it can look:

tut_timeline_colors

This gives a better visual indication of what a message might be interesting to you and helps filter out noise from signal

To make this work I made the space for symbols a bit bigger (configurable), allowing multiple symbols
@proycon proycon changed the title WIP: Adding colours to the feed to indicate type Adding colours to the feed to indicate type Dec 20, 2022
@proycon proycon marked this pull request as ready for review December 20, 2022 10:44
@proycon
Copy link
Author

proycon commented Dec 20, 2022

By default the colours are mostly unused and you have the regular behaviour, so to use this you have to opt-in with for example the following in your style section (or theme):

timeline-name-text=#808080                                                                                                                     
boost-text=#446672                                                                                                                             
media-text=#c630ce                                                                                                                             
card-text=#3a46a1 

# Alternative color-preserving highlighter that take effect instead of the background color                                                    
# default=0                                                                                                                                    
list-selected-boldunderline=1       

@RasmusLindroth
Copy link
Owner

Looking good! I'll look more at both your PRs and merge them later so you know :)

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.

2 participants