Skip to content

Suggestion: Bump D3 dependency to a newer version #48

@danielhjacobs

Description

@danielhjacobs

d3.js v7 is out now. There are a number of improvements that have been made. One issue that I encountered with version 5 was because of d3/d3-zoom#217. We disabled the ability to zoom the TidyTree by default and added a toggle to use to re-enable it, so that page scrolling would work even while hovering over the tree. But scrolling the page using a touchscreen still does not work because until d3.js version 6, when a touch event was detected the touch-action: none style was added.

To workaround this issue, I'll have to add a MutationObserver to the svg created by TidyTree's use of d3 that looks for style changes and if touch-action is none after the change I'll have to remove the touch-action style. But I wouldn't need to workaround the issue if TidyTree used modern d3.js.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions