tether icon indicating copy to clipboard operation
tether copied to clipboard

`tether-drop` does not work with v2.0

Open ericyd opened this issue 2 years ago • 1 comments

Summary

tether-drop library does not work with v2.0, but no breaking changes were announced in the release notes.

Technically tether-drop is a separate lib and it's compatibility is not your concern, but I'm opening an issue here for two reasons

  1. tether-drop seems entirely abandoned, whereas this lib has at least a little recent activity
  2. The documentation for Tether references tether-drop liberally, and I imagine a new user of this lib would be a little confused at the incompatibility

Technical

  1. Drop relies on Tether.Utils (reference)
  2. Tether.Utils does not exist in the new version (reference)
  3. Documentation references tether-drop in many places, e.g.

Recommendation

This could probably be somewhat easily resolved by adding back a Tether.Utils declaration similar to the original.

However, please note that tether-drop also relies on extend, which appears to have been removed in v2.0.0.

If you're interested in a solution like this, let me know and I can try to open a PR. I'd probably just copy-paste the extend function from v1.4.7 since it's pretty small and self-contained.

Thanks for your time and energy 🙂

ericyd avatar Oct 25 '23 21:10 ericyd

Hi @ericyd yes, a lot of these HubSpot libraries were all originally based on Tether. Shepherd was significantly refactored to not need it anymore, and perhaps the same should be done with Drop. These libraries used to just tether elements together can probably go away now and we can use things like https://developer.chrome.com/blog/tether-elements-to-each-other-with-css-anchor-positioning/

RobbieTheWagner avatar Nov 01 '23 14:11 RobbieTheWagner