Skip to content

Remove unnecessary tooltip positioning patches#165

Merged
KittenApps merged 2 commits into
KittenApps:betafrom
zorgjbe:fix/remove-tooltip-patches
May 9, 2026
Merged

Remove unnecessary tooltip positioning patches#165
KittenApps merged 2 commits into
KittenApps:betafrom
zorgjbe:fix/remove-tooltip-patches

Conversation

@zorgjbe
Copy link
Copy Markdown

@zorgjbe zorgjbe commented Apr 20, 2026

Moved into BC proper: https://gitgud.io/BondageProjects/Bondage-College/-/merge_requests/6256

Note that I've looked around, and unless I missed something I don't see any users of that additional parameter anywhere.

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 20, 2026

Deploy Preview for wce ready!

Name Link
🔨 Latest commit 50d60dd
🔍 Latest deploy log https://app.netlify.com/projects/wce/deploys/69ff3ca811eb330008207d38
😎 Deploy Preview https://deploy-preview-165--wce.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@KittenApps
Copy link
Copy Markdown
Owner

Yea, looks like all instances where this was previously used were already migrated to DOM ui before.

@zorgjbe
Copy link
Copy Markdown
Author

zorgjbe commented Apr 22, 2026

That was just merged in BC proper.

@KittenApps
Copy link
Copy Markdown
Owner

then we could do a conditional removal for R128 I guess

Copy link
Copy Markdown
Owner

@KittenApps KittenApps left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To test this PR you can use the WCE PR loaders:

@KittenApps KittenApps merged commit 9b57562 into KittenApps:beta May 9, 2026
7 checks passed
@zorgjbe
Copy link
Copy Markdown
Author

zorgjbe commented May 9, 2026

Nice! Thanks~

@zorgjbe zorgjbe deleted the fix/remove-tooltip-patches branch May 9, 2026 14:08
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.

2 participants