-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adjusting canvas height based GenotypeTrack height #12
base: master
Are you sure you want to change the base?
Conversation
…and Genotype.js constructors to take in objects instead
…atureDataSource-test.js file
…th RemoteRequest file
…and Genotype.js constructors to take in objects instead
fixed no data error for alignments and max rows for PileupTrack
…uest areas already in cache.
Variant fixes
# Conflicts: # src/main/sources/GA4GHDataSource.js
Update Feature Transparency Based on Score
searchNode = ReactDOM.findDOMNode(searchNode).parentNode; | ||
if (searchNode && searchNode.className && searchNode.className.split(' ') && searchNode.className.split(' ')[1]) { | ||
let name = searchNode.className.split(' ')[1]; | ||
if (name == GenotypeTrack.displayName) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did we decide to do anything about the PileupTrack?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you comment this a little more so we know whats going on?
Can one of the admins verify this patch? |
Fixed in the latest PR |
Sorry @snaheth , that was a test PR for some build testing and will not be merged. |
Can one of the admins verify this patch? |
Not using props on this one so we don't have to spread our code out over classes, as discussed