Add location string to tooltip for wiggle and SNPCoverage tracks

[cmdcolin]

Kind of simple but something that I think can help readability

[codecov]

Codecov Report

Merging #1576 (47900e1) into master (4bb285b) will decrease coverage by 0.02%.
The diff coverage is 21.62%.

@@            Coverage Diff             @@
##           master    #1576      +/-   ##
==========================================
- Coverage   59.51%   59.48%   -0.03%     
==========================================
  Files         435      435              
  Lines       19564    19584      +20     
  Branches     4567     4573       +6     
==========================================
+ Hits        11643    11650       +7     
- Misses       7627     7640      +13     
  Partials      294      294              

Impacted Files	Coverage Δ
packages/core/BaseFeatureWidget/index.ts	71.42% <0.00%> (ø)
...ns/alignments/src/AlignmentsFeatureDetail/index.js	71.42% <0.00%> (ø)
...rc/LinearSNPCoverageDisplay/components/Tooltip.tsx	21.87% <0.00%> (-5.05%)	⬇️
...iew/src/BreakpointAlignmentsFeatureDetail/index.js	28.57% <0.00%> (ø)
plugins/variants/src/VariantFeatureWidget/index.js	42.85% <0.00%> (ø)
plugins/wiggle/src/BigWigAdapter/BigWigAdapter.ts	94.44% <ø> (ø)
...gle/src/LinearWiggleDisplay/components/Tooltip.tsx	40.00% <0.00%> (-17.15%)	⬇️
...kages/core/BaseFeatureWidget/BaseFeatureDetail.tsx	69.35% <88.88%> (+1.25%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4bb285b...47900e1. Read the comment docs.

[garrettjstevens]

I like this. Would it be possible to show just one number instead of start..end when the length is 1? For example have the location here:

Match the overview location here:

[cmdcolin]

@garrettjstevens added a change for that, also corrects it to use feature.get('start')+1 for the displayed loc

[cmdcolin]

I made an additional proposal to the feature detail dialogs that it makes model.featureData=undefined by default rather than model.featureData={}

This uses types.frozen() instead of types.frozen({}) in the model definition

If this is not the case, then the base feature details can receive undefined entries for start,end,etc. and then end.toLocaleString is a crash, etc.

[cmdcolin]

Just because there is some precedent for these things possibly being an empty object, to avoid a crash, I check for empty object now