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

buffer_nicklist_count includes nicklist groups #1506

Closed
trygveaa opened this issue May 13, 2020 · 2 comments
Closed

buffer_nicklist_count includes nicklist groups #1506

trygveaa opened this issue May 13, 2020 · 2 comments
Assignees
Labels
feature New feature request
Milestone

Comments

@trygveaa
Copy link
Member

trygveaa commented May 13, 2020

Question

I'm not sure if this is intentional or not, but I've always found it strange that the buffer_nicklist_count bar item counts both nicks and groups instead of just nicks.

The description for gui_bar_item_buffer_nicklist_count_cb says "Bar item with number of nicks in buffer nicklist.", so from that it seems that it should only count the number of nicks. However, that function uses the property nicklist_visible_count and the description for that in the documentation is "number of nicks/groups displayed", so there's an inconsistency there.

The wee-slack script uses groups to separate nicks into the ones present and the ones away, which means that buffer_nicklist_count is always two more than the number of nicks in the channel.

I would think the information that is useful to show in the status bar is the number of nicks. However, I don't know what groups are used for other places. Is there anytime it's useful to count them?


  • WeeChat version: 2.8
  • OS, distribution and version: Arch Linux
@trygveaa trygveaa added the question General question label May 13, 2020
@flashcode
Copy link
Member

You're right, so currently we have: buffer_nicklist_count == visible groups + nicks.

To stay compatible with existing config, I propose such changes:

  • buffer_nicklist_count (existing item) == visible nicks only
  • buffer_nicklist_count_groups (new item) == visible groups only
  • buffer_nicklist_count_all (new item) == visible groups + nicks

@flashcode flashcode self-assigned this May 16, 2020
@flashcode flashcode added bug Unexpected problem or unintended behavior feature New feature request and removed question General question bug Unexpected problem or unintended behavior labels May 16, 2020
@flashcode flashcode added this to the 2.9 milestone May 16, 2020
@trygveaa
Copy link
Member Author

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature request
Projects
None yet
Development

No branches or pull requests

2 participants