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

Overlaps in dropdown menu of friends or guild members🐛 #13

Closed
2 tasks done
JasonDepp opened this issue Jul 31, 2024 · 4 comments
Closed
2 tasks done

Overlaps in dropdown menu of friends or guild members🐛 #13

JasonDepp opened this issue Jul 31, 2024 · 4 comments
Assignees

Comments

@JasonDepp
Copy link
Collaborator

Is there an existing issue for this?

  • I have searched the existing open and closed issues.

Description

very interesting addon!
found a minor issue, when you right click on a player in friend list or guild member list, or chat window, the menu item is overlapped and not clickable.

found this code in Shitlist.lua:

if not (UnitIsPlayer(unit) or (unit == nil and target == "FRIEND" or target == "COMMUNITIES_GUILD_MEMBER")) then
    return
end

I guess the original idea is to hide "shitlist" when

  1. unit is not a player
  2. unit is nil
  3. target is my friend
  4. target is from my guild/area/chat window

somehow this code statement doesn't block "friend" or "guild"

if what I mentioned above is true, then I suggest to update this into:

if not (UnitIsPlayer(unit)) or (unit == nil and target == "FRIEND" or target == "COMMUNITIES_GUILD_MEMBER") then
    return
end

with a simple ")" movement

ps: I also tried to disable other addons and keep shitlist only, but it doesn't show anything in menu dropdown (as far as I can see)...
I'm using Shitlist-2.0.3
WOW 3.4.3 zhCN(I added a toc myself)

World of Warcraft Flavor

Wrath of the Lich King Classic

Tested with only Shitlist

  • Yes

Lua Error

No response

Reproduction Steps

  1. Go to my guild
  2. right click on a member's name

Screenshots

image

@Limmek
Copy link
Owner

Limmek commented Jul 31, 2024

Great catch i have added the change to the if statement, also created a wrath toc file.
There is a new 2.0.4 release available.

@JasonDepp
Copy link
Collaborator Author

Thank you so much for your quick response!
I would like to contribute to this addon in the future!
Do you develop this addon yourself?
why curseforge removed your addon...
if the reason is related to the shit word...my advise is to rename this addon into something else like "attention list"?
cuz if I'm going to localize it into Chinese Simplified, I would use that name in Chinese...

@Limmek
Copy link
Owner

Limmek commented Aug 1, 2024

Sure mate you are more than welcome to submit a translation for the addon if you want to.
Yeah it's only me, the idea cam up when i was playing classic and was annoyed of geting in groups with bad players so i wanted a simple way to make a note on a player to remind my self in the future.

Check this comment c6d9146#commitcomment-144901359 i did a support ticket and asked what i should change but still waiting on a response from them...
I think i will rename the addon to Player Notes or something and change the description a bit, then make a new request and see if it will be approved again.

@JasonDepp
Copy link
Collaborator Author

yea Player Notes also makes sense,
curseforge mentioned about impacting other players but it's not the fact... it only reminds "myself" for someone that I like/dislike...

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

No branches or pull requests

2 participants