Skip to content

Creation of DraggableNeatBLASTFeatures tracktype#2637

Open
childers wants to merge 1 commit intodevelopfrom
childers-nbf
Open

Creation of DraggableNeatBLASTFeatures tracktype#2637
childers wants to merge 1 commit intodevelopfrom
childers-nbf

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 childers requested a review from cmdcolin November 2, 2021 15:56
@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 requested a review from nathandunn November 12, 2021 15:18
@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