Skip to content

Commit

Permalink
Replace react-dragula with dnd-kit (#1451)
Browse files Browse the repository at this point in the history
* Replace react-dragula package with dnd-kit

* Refactor to function component

* Remove @types/dragula as well

* Move

* Initial implementation

* Ditch DragOverlay

* Fix function name

* Fix handling drag

* Fix scrolling issue

* Create gorgeous-meals-glow.md

* Fix app test

* Fix extension test

* Fix styling
  • Loading branch information
Methuselah96 authored Aug 18, 2023
1 parent 176a6cc commit 14a7957
Show file tree
Hide file tree
Showing 7 changed files with 260 additions and 230 deletions.
5 changes: 5 additions & 0 deletions .changeset/gorgeous-meals-glow.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'@redux-devtools/inspector-monitor': patch
---

Replace react-dragula with dnd-kit
4 changes: 1 addition & 3 deletions extension/test/app/containers/App.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -37,8 +37,6 @@ describe('App container', () => {
</Provider>,
);
const actionList = screen.getByTestId('actionList');
expect(
within(actionList).getByTestId('actionListRows'),
).toBeEmptyDOMElement();
expect(within(actionList).queryByRole('button')).not.toBeInTheDocument();
});
});
4 changes: 1 addition & 3 deletions packages/redux-devtools-app/test/app.spec.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -45,9 +45,7 @@ describe('App container', () => {
</Provider>,
);
const actionList = screen.getByTestId('actionList');
expect(
within(actionList).getByTestId('actionListRows'),
).toBeEmptyDOMElement();
expect(within(actionList).queryByRole('button')).not.toBeInTheDocument();
});
});

Expand Down
7 changes: 4 additions & 3 deletions packages/redux-devtools-inspector-monitor/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,10 @@
},
"dependencies": {
"@babel/runtime": "^7.22.6",
"@types/dragula": "^3.7.1",
"@dnd-kit/core": "^6.0.8",
"@dnd-kit/modifiers": "^6.0.1",
"@dnd-kit/sortable": "^7.0.2",
"@dnd-kit/utilities": "^3.2.1",
"@types/lodash": "^4.14.196",
"@types/prop-types": "^15.7.5",
"@types/redux-devtools-themes": "^1.0.0",
Expand All @@ -50,7 +53,6 @@
"lodash.debounce": "^4.0.8",
"prop-types": "^15.8.1",
"react-base16-styling": "^0.9.1",
"react-dragula": "^1.1.17",
"react-json-tree": "^0.18.0",
"redux-devtools-themes": "^1.0.0"
},
Expand All @@ -68,7 +70,6 @@
"@types/history": "^4.7.11",
"@types/lodash.debounce": "^4.0.7",
"@types/react": "^18.2.18",
"@types/react-dragula": "^1.1.0",
"@typescript-eslint/eslint-plugin": "^5.62.0",
"@typescript-eslint/parser": "^5.62.0",
"eslint": "^8.46.0",
Expand Down
Loading

0 comments on commit 14a7957

Please sign in to comment.