ES2015 allows iteration of sets which is used in this codebase.

Use named imports from the package instead of direct imports from
source. This allows proper typing by the external types/d3 package.

For improved readability.

The AppDispatch type is already used elsewhere.

Converted most but not all tree-related files. There are many type
errors exposed by conversion. They will be addressed in subsequent
commits.

I removed a few comments that are now self-explanatory with the added
types. I also moved all JSDoc types and descriptions into the function
signature.

Note that there are many optional properties in the newly defined
interfaces. This is not ideal and doesn't capture the fact that many
properties are conditional on things such as layout. But it's a starting
point to be improved over time.

These are not directly used by other TypeScript files, but I've inferred
them from existing JavaScript files.

The conversion done in the previous commits came with hundreds of
violations of this rule. While this could be addressed by better types,
it will take some work and can be deferred. A new issue/PR will be
created after this is merged.

<#1864 (comment)>

To satisfy the TypeScript rule noImplicitOverride.

To satisfy the ESLint rule @typescript-eslint/no-unused-vars.

Define the list of layouts in a single place.

Previous usage of parseInt was to convert float to integer. However,
this isn't the intended usage of parseInt, which is made apparent with
type checking - the parameter is expected to be a string type.

Replace it with Math.trunc which is intended for converting float to
integer.

Thse are typed to return a value that is not undefined.

This is easier for the TypeScript compiler to understand.

D3 expects the empty value to be null, not undefined.

Previously, there was no guarantee that an unknown error would produce a
helpful message.

svgPropsToUpdate is Set<string>, not Set<string[]>.

Only KeyboardEvent has the shiftKey property.

calcTipRadii takes a key name of tipSelectedIdx. Previously, the value
of idx was unused.

This should be a boolean, not whatever value newDistance is when
defined.

This should be a boolean, not whatever value
ReduxNode.parentInfo.original is when defined.

Not [string], [], [[string], [string]], or [string, [string]].

The extra value `true` had no effect.

These were initialized with default values but unused in the code.

Follow-up to "Lift node-selected modal to redux state" (f7e944d).

This is left over from before we shifted the selected node to redux
state.

This value was causing a type error because {} is not a Map. Remove it
entirely since it isn't actually used - the value is created from
scratch in src/middleware/performanceFlags.js.

There is a type error that prevents this from working properly:
includes() takes a single string, but an array of PhyloNodes is given.
It's not used on any existing calls to updateTipRadii, so just remove
it entirely.

This previously worked because d3 was able to properly parse a color
string out of RGBColor. However, it's better to keep things consistent
and simple. Convert so that range is simply an array of hex color
strings.

Centralize the defaults for consistency. Also, this avoids the need to
mark the property as optional on the state.

This is more consistent with other booleans.

React will shallow merge the provided object into the existing state¹
which works for our usage. The type definition of setState does not
allow partial state by default. Use 'never' to configure it to expect no
fields in particular².

¹ <https://react.dev/reference/react/Component#setstate-parameters>
² <https://stackoverflow.com/a/55824499>