Skip to content

Conversation

rtucek
Copy link
Owner

@rtucek rtucek commented Jan 19, 2022

As reported by @ucuncuatabek [1] and @janhorubala [2], this is an attempt to fix odd drag'n'drop behaviour with production builds.

As reported by @ucuncuatabek
[[1]](#61 (comment)) and
@janhorubala [[2]](dabernathy89/vue-query-builder#70 (comment)),
this is an attempt to fix odd drag'n'drop behaviour with production builds.
@rtucek rtucek added bug Something isn't working help wanted Extra attention is needed labels Jan 19, 2022
@rtucek rtucek self-assigned this Jan 19, 2022
@github-actions
Copy link
Contributor

Code Coverage Report

Coverage after merging fix-merge-trap-comparison into main

97.73%
Coverage Report
FileBranchesFuncsLinesUncovered Lines
src
   MergeTrap.ts94.44%100%98.28%8, 9
   QueryBuilder.vue100%100%100%
   QueryBuilderChild.vue83.33%100%92.31%..., 55, 59, 67, 71
   QueryBuilderGroup.vue87.50%100%98.08%..., 292, 292, 293, 297
   QueryBuilderRule.vue75%100%94.74%23, 27
   guards.ts100%100%100%
   types.ts100%100%100%

1 similar comment
@github-actions
Copy link
Contributor

Code Coverage Report

Coverage after merging fix-merge-trap-comparison into main

97.73%
Coverage Report
FileBranchesFuncsLinesUncovered Lines
src
   MergeTrap.ts94.44%100%98.28%8, 9
   QueryBuilder.vue100%100%100%
   QueryBuilderChild.vue83.33%100%92.31%..., 55, 59, 67, 71
   QueryBuilderGroup.vue87.50%100%98.08%..., 292, 292, 293, 297
   QueryBuilderRule.vue75%100%94.74%23, 27
   guards.ts100%100%100%
   types.ts100%100%100%

@github-actions
Copy link
Contributor

Code Coverage Report

Coverage after merging fix-merge-trap-comparison into main

97.73%
Coverage Report
FileBranchesFuncsLinesUncovered Lines
src
   MergeTrap.ts94.44%100%98.28%8, 9
   QueryBuilder.vue100%100%100%
   QueryBuilderChild.vue83.33%100%92.31%..., 55, 59, 67, 71
   QueryBuilderGroup.vue88.10%100%98.08%..., 292, 292, 293, 297
   QueryBuilderRule.vue75%100%94.74%23, 27
   guards.ts100%100%100%
   types.ts100%100%100%

@rtucek
Copy link
Owner Author

rtucek commented Jan 19, 2022

@janhorubala and @ucuncuatabek, kindly asking you to verify if this patch fixes the issue you encounter in the mentioned links above.

For some reason I don't understand, I've never been able to reproduce what the bug, but it seems to me, the webpack's terser plugin, used for the production build, mangels the QueryBuilderGroup component's name and thus breaks the drag'n'drop.

Please don't forget to yarn build locally prior testing 🙏

@rtucek rtucek linked an issue Jan 19, 2022 that may be closed by this pull request
@rtucek rtucek merged commit d9aa458 into main Jan 22, 2022
@rtucek rtucek deleted the fix-merge-trap-comparison branch January 22, 2022 22:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working help wanted Extra attention is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Max nested groups

1 participant