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

Report button inconsistency #409

Open
2 tasks done
emiljensen2 opened this issue Jan 18, 2024 · 6 comments
Open
2 tasks done

Report button inconsistency #409

emiljensen2 opened this issue Jan 18, 2024 · 6 comments
Labels
enhancement Improvement to an already existing notifier

Comments

@emiljensen2
Copy link

Checklist

Describe your issue

When dink takes a screenshot of events it doesn't always show the date despite it being enabled in the base screenshot plugin.
I'm not quite sure what causes it
Collection log events seem to always have the date show if popup is enabled, I see some cases of date not showing and they don't have the popup enabled.
CA events also seem to capture date consistently (Though this is without the popup, so unsure if it's always true)
Levels, loot, kill count, quests is inconsistent from what I've seen.

Sorry for yet another report

Screenshots

image
image

Runelite Logs

shouldn't be relevant

Runelite Version

1.10.19

@pajlada
Copy link
Member

pajlada commented Jan 18, 2024

Could you compare these with the screenshots that were saved on disk (and maybe post them too)?

@emiljensen2
Copy link
Author

emiljensen2 commented Jan 18, 2024

I've taken some levels screenshots since my rl doesn't do loot itself.
image
Runecraft(84) 2024-01-16_22-24-18
image
Mining(91) 2024-01-10_19-21-35
The mining screenshot is interesting, they appear to be taken a little time apart as my idle notification is visible on the dink screenshot, but not the local screenshot.
Though ig it's also visibly different for the zeah screenshot.

@pajlada
Copy link
Member

pajlada commented Jan 18, 2024

We delay certain notifiers, sometimes to make sure we get all level ups that happened, sometimes to give time for the loot to appear in the chat or on the ground for the screenshot.

I suspect this is just a matter of us not using the screenshot plugin, but just happen to be taking screenshots at the same time as the screenshot plugin for certain events.

I guess it boils down to more of a feature request: Should we replicate the screenshot plugin's "Report button" functionality?

@emiljensen2
Copy link
Author

Ah that makes sense, I thought at first you were using the screenshot plugin's functionality so to speak. but that makes sense.
So it's inconsistent because it sometimes captures on the same frame/tick(?), but it is not intended.

Yeah I guess it'd be a feature request then.

@iProdigy
Copy link
Member

@iProdigy iProdigy added enhancement Improvement to an already existing notifier and removed issue-report labels Jan 18, 2024
@Felanbird
Copy link
Member

The current upstream implementation of the report button does not respect where the report button actually is, if the chat box is somewhere else, it will just float in it's normal spot. Keep that in mind whether or not we want to fix it ourselves, fix it for them, or wait for someone to fix it.

Discussed in the discord last week but I don't believe anyone bothered to open an issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improvement to an already existing notifier
Projects
None yet
Development

No branches or pull requests

4 participants