-
Notifications
You must be signed in to change notification settings - Fork 32
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
FeatureViewer React Component #31
Comments
In general: would you consider making this package more react friendly? Another consideration in this regard is that there's a heavy reliance on Bootstrap to render the component correctly, but needing such a big amount of extra CSS just to render this one component is a bit of an overkill. I'm working on my fork to make the component react friendly, and I would appreciate any kind of help on that :) |
works for me within react.js without any trouble. Main code goes into the |
We are not using react, do you still have conflicts or Kohze way of doing it does fix it ? |
@Kohze 's way doesn't work if you are using different styling than bootstrap. Recently, react started allowing scss, aka. it's possible to apply component-wide (isolated) styling. The code for how this looks like with FeatureViewer can be found here: https://github.com/sacdallago/embed.protein.properties/tree/master/src/components/FeatureViewer You can use the code in that repo to make an example (I suggest so! there are so many react users out there...) edit: notice the |
Hi,
I have recently found Feature Viewer and really like it I finally found a nice d3-based genomics visualization tool.
Since my framework I am developing is based on TypeScript + React (+Webpack + Bootstrap v4), I created a compatible version of your FeatureViewer (if you are interested, I will push it to my fork as soon as I got it working without below error).
It seems to work for most parts (I am only using the visualization part, not the NeXtProt part), but there's one error I cannot get away:
TypeError: d3_selection__WEBPACK_IMPORTED_MODULE_3__.event.target.__data__ is undefined
Which seems to originate from
started brush.js:296 contextListener/< on.js:27 FeatureViewer.prototype.forcePropagation/< contextListener/<
which is the forcePropagation function, which I needed to rewrite a bit to make it work with TypeScript:
`
forcePropagation(item) {
var self=this;
`
Now, you know the Feature Viewer better than me, do you have any idea what could go wrong there and result such an error? I also wonder what this mousedown-event should be doing?
Thanks for you help!
The text was updated successfully, but these errors were encountered: