Skip to content

Fix Suppress Tooltips behaviour - #6029

Open
bubblobill wants to merge 2 commits into
RPTools:developfrom
bubblobill:suppressTooltipsFix
Open

Fix Suppress Tooltips behaviour#6029
bubblobill wants to merge 2 commits into
RPTools:developfrom
bubblobill:suppressTooltipsFix

Conversation

@bubblobill

@bubblobill bubblobill commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

Identify the Bug or Feature request

fixes #6028

Description of the Change

TooltipView:

  • Rearranged logic and renamed variables to avoid double negatives and make things clearer
  • Added decision fork to catch "lib:" uris containig "/macro/".
  • When suppressing anti-cheat tooltip, or not a macro link the hyperlink "title" attribute is thte first choice as tooltip

i18n.properties

  • Reworded the appalling tooltip text

Possible Drawbacks

I unwrapped the double negatives poorly.
People using span tags for tooltips that also have a title attribute on their links will get weird results.

Documentation Notes

n/a

Release Notes

n/a


This change is Reviewable

- Rearranged logic
- Added return value for "lib:" uris.
Reworded the appalling i18n text
Added extra check on "lib:" uris for presence of "macro"
@github-actions github-actions Bot added the bug label Aug 7, 2026
@bubblobill
bubblobill requested a review from cwisniew August 7, 2026 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Suppress ToolTips for MacroLinks is broken

1 participant