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

feat: add indicator to title if toast is toggled #2119

Closed
wants to merge 2 commits into from
Closed

feat: add indicator to title if toast is toggled #2119

wants to merge 2 commits into from

Conversation

kopecmaciej
Copy link

Closes issues/2052

Hi. should this be turn off or on by default? Also do I need to update skins with toastIcon? Thanks

Copy link
Owner

@derailed derailed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kopecmaciej Thank you for this PR Maciej! I think we should perhaps be more generic given that icons might not be viable in some terminal/distros and give user an option to display text as an alternate. So perhaps something like toastIndicator might be more generic. Also might be better to append to the title is Pod! or Pod Faults or Pod ☠️ might make more sense?

@kopecmaciej kopecmaciej changed the title feat: add icon to title if toast is toggled feat: add indicator to title if toast is toggled Jun 14, 2023
@kopecmaciej
Copy link
Author

Hey @derailed good idea, the indicator fits much better for this, + I changed the order, currently looks like this:
Screenshot_2023-06-14-18-07-39_4364

@kopecmaciej
Copy link
Author

Hi, @derailed, can this be merge or there is something that need to be fixed?

@kopecmaciej kopecmaciej closed this by deleting the head repository Oct 20, 2023
@derailed derailed mentioned this pull request Nov 8, 2023
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.

Add visual hint that current list is filtered by 'faults'
3 participants