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(options): added max_messages option #127

Merged
merged 1 commit into from
Jun 10, 2023

Conversation

pyrho
Copy link

@pyrho pyrho commented May 5, 2023

Description

Some LSP server are very chatty, and when they send too many messages in a short burst the area taken by fidget.nvim becomes so large that it covers up some of the editing window area.

The new option (defaulted to 2) limits the number of messages shown at any given time.

Current behavior
CleanShot 2023-05-04 at 19 57 41

New behavior
CleanShot 2023-05-04 at 19 59 30

Fixes #122

Some LSP server are very chatty, and when they send too many messages in
a short burst the area taken by fidget.nvim becomes so large that it
covers up some of the editing window area.

The new option (defaulted to `2`) limits the number of messages shown at
any given time.

Fixes j-hui#122
@pyrho pyrho mentioned this pull request May 5, 2023
@j-hui
Copy link
Owner

j-hui commented Jun 10, 2023

Thank you for this PR! I really should have merged this in sooner, but I am going to rewrite fidget.nvim from scratch. I'm going to change the base branch to legacy, where I'm keeping a snapshot of the original implementation.

@j-hui j-hui changed the base branch from main to legacy June 10, 2023 06:26
@j-hui j-hui merged commit 90c22e4 into j-hui:legacy Jun 10, 2023
@dbalatero
Copy link

Thank you!

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.

spammy Diagnosing messages
3 participants