Cleveroad / AdaptiveTableLayout

Library that makes it possible to read, edit and write CSV files
MIT License
1.9k stars 234 forks source link

may onActionUp be optimized? #9

Closed tootwo2 closed 7 years ago

tootwo2 commented 7 years ago

The method onActionUp of AdaptiveTableLayout will be triggered whenever touchUp happens. May the method content be wrapped with 'if(mState.isDragging())'? code line 1383,like

@Override
public boolean onActionUp(MotionEvent e) {
        if(mState.isDragging()){
              // remove shadows from dragging views
             mShadowHelper.removeAllDragAndDropShadows(this);

             // stop smooth scrolling
            if (!mScrollerDragAndDropRunnable.isFinished()) {
                mScrollerDragAndDropRunnable.stop();
            }

              ........
        }

        return true;
}
yarovoiag commented 7 years ago

optimized in new version 1.1.1