Skip to content

Tweak heatmap ui#999

Merged
timotheeguerin merged 11 commits intomasterfrom
feature/heatmap-tweak
Jan 8, 2018
Merged

Tweak heatmap ui#999
timotheeguerin merged 11 commits intomasterfrom
feature/heatmap-tweak

Conversation

@timotheeguerin
Copy link
Member

image

Reduce max size for a tile to be 100px from 300px
Moved legend to be on the right
Internal; Move the heatmap code to be more structured

@codecov
Copy link

codecov bot commented Jan 8, 2018

Codecov Report

Merging #999 into master will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #999      +/-   ##
==========================================
+ Coverage   54.67%   54.69%   +0.01%     
==========================================
  Files         758      760       +2     
  Lines       17911    17914       +3     
  Branches     2034     2034              
==========================================
+ Hits         9793     9798       +5     
+ Misses       8118     8116       -2
Impacted Files Coverage Δ
...pp/components/pool/graphs/heatmap/state-counter.ts 95% <ø> (ø)
...pp/components/pool/graphs/heatmap/heatmap-color.ts 95.23% <ø> (ø)
app/components/pool/graphs/heatmap/legend/index.ts 100% <100%> (ø)
...pp/components/pool/graphs/pool-graphs.component.ts 27.73% <100%> (ø) ⬆️
app/components/pool/graphs/index.ts 100% <100%> (ø) ⬆️
...nts/pool/graphs/heatmap/nodes-heatmap.component.ts 87.69% <100%> (ø)
app/components/pool/graphs/pool-graphs.module.ts 100% <100%> (ø) ⬆️
...s/heatmap/legend/nodes-heatmap-legend.component.ts 70.73% <100%> (ø)
app/components/pool/graphs/heatmap/index.ts 100% <100%> (ø)
... and 2 more

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 68b14b8...4dc6c24. Read the comment docs.

@timotheeguerin timotheeguerin merged commit d5d02f2 into master Jan 8, 2018
@timotheeguerin timotheeguerin deleted the feature/heatmap-tweak branch January 8, 2018 18:09
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.

1 participant