Skip to content
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

Creation of DraggableNeatBLASTFeatures tracktype #2637

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

childers
Copy link
Collaborator

@childers childers commented Nov 2, 2021

This is an update to the Draggable BLASFeatiures track type to support Neat Features. This update works, but needs to have the className set to container-10px

This is an update to the Draggable BLASFeatiures track type to support Neat Features.  This update works, but needs to have the className set to container-10px
@childers
Copy link
Collaborator Author

childers commented Nov 2, 2021

What's the best way to adjust the className in DraggableNeatBLASTFeatures.js? I can manually set this in the URL, but I want it to be a default parameter.

Otherwise, this update works to add the track type DraggableNeatBLASTFeatures

@childers childers added NAL NAL, WhipWorm, Large Organization problems etc. waiting for feedback labels Nov 12, 2021
@garrettjstevens
Copy link
Contributor

What's the best way to adjust the className in DraggableNeatBLASTFeatures.js? I can manually set this in the URL, but I want it to be a default parameter.

@cmdcolin any ideas here?

@cmdcolin
Copy link
Collaborator

could maybe override the defaultConfig method similar to

make the className just container-10px

possibly something like like

    _defaultConfig: function() {
        return Util.deepUpdate(
            dojo.clone( this.inherited(arguments) ),
            {
                style: {
                    className: "container-10px",
                }
             }
         )
    }

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NAL NAL, WhipWorm, Large Organization problems etc. waiting for feedback
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants