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

Toggle displaying major and minor modes in mode-line #199

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ayys
Copy link

@ayys ayys commented Feb 14, 2024

Why do this?

A lot of times, the mode-line gets cluttered with many minor modes, which makes it look messy. Many times (especially on smaller screens), the content of mode-line exceeds screen-width and gets cut-off. This is very annoying.
I don't usually need to see the minor-modes all the time, so adding this customization made sense to me. This way, I can have a function to toggle the minor modes on if I want to view them, otherwise I can keep them turned off.

What does it do?

Adds the ability to toggle whether major-mode and/or minor-modes are displayed in the mode line.

Major mode can be toggled via powerline-display-major-mode custom var.
Minor modes can be toggled via powerline-display-minor-modes custom var.

Adds the ability to toggle whether major-mode and/or minor-modes are displayed
in the mode line.

Major mode can be toggled via `powerline-display-major-mode` custom var
Minor modes can be toggled via `powerline-display-minor-modes` custom var
Copy link

@jcs090218 jcs090218 left a comment

Choose a reason for hiding this comment

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

I don't have a strong opinion on this. 🤔 Can you update the README as well? Thanks!

(if powerline-display-minor-modes
(list (powerline-minor-modes face2 'l)
(powerline-raw " " face2)
(funcall separator-right face2 face1))))

Choose a reason for hiding this comment

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

Why not use when here, too?

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.

None yet

2 participants