Skip to content

Commit

Permalink
fix(devtools): send full parser result from content script to query p…
Browse files Browse the repository at this point in the history
…anel
  • Loading branch information
smeijer committed Mar 26, 2021
1 parent 04467eb commit 690f071
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 13 deletions.
16 changes: 6 additions & 10 deletions devtools/src/content-script/contentScript.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ import { getAllPossibleQueries } from '../../../src/lib';
import inject from './lib/inject';
import { setup } from '../window/testing-library';
import onDocReady from './lib/onDocReady';
import cssPath from '../../../src/lib/cssPath';

function init() {
inject('../window/testing-library.js');
Expand All @@ -25,15 +24,12 @@ function init() {

const suggestion = Object.values(queries).find(Boolean);

Bridge.sendMessage(
'SELECT_NODE',
{
suggestion,
queries,
cssPath: cssPath(node, true).toString(),
},
'devtools',
);
const result = parser.parse({
rootNode: document.body,
query: suggestion?.snippet || '',
});

Bridge.sendMessage('SELECT_NODE', result, 'devtools');
}

Bridge.onMessage('PARSE_QUERY', function ({ data }) {
Expand Down
2 changes: 1 addition & 1 deletion devtools/src/devtools/pane.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ function Panel() {

useEffect(() => {
Bridge.onMessage('SELECT_NODE', ({ data }) => {
setResult({ elements: [data] });
setResult(data);
});
}, [setResult]);

Expand Down
4 changes: 2 additions & 2 deletions devtools/src/devtools/panel.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,8 @@ function Panel() {

useEffect(() => {
Bridge.onMessage('SELECT_NODE', ({ data }) => {
setResult({ elements: [data] });
editor.current.setValue(data.suggestion?.snippet || '');
setResult(data);
editor.current.setValue(data.query);
});
}, [setResult]);

Expand Down

0 comments on commit 690f071

Please sign in to comment.