Skip to content

web console/refactor jquery modules - #20

Merged
argshook merged 10 commits into
mainfrom
web-console/refactor-jquery-modules
Jul 22, 2022
Merged

web console/refactor jquery modules#20
argshook merged 10 commits into
mainfrom
web-console/refactor-jquery-modules

Conversation

@argshook

Copy link
Copy Markdown
Contributor

This PR is a cleanup and refactor with the aim to make grid.js and
quick-vis.js easier to reuse in React code

  • remove unused constants
  • refactor globals.js
  • move docsearch css import to index.tsx
  • make quick-vis as module
  • make grid as module
  • add @types/jquery
  • use grid and quickVis modules in Result
  • remove quickVis and grid from console/index.js
  • refactor globals
  • remove console-controller

@argshook argshook self-assigned this Jul 15, 2022
argshook added 4 commits July 16, 2022 02:45
they're now used directly in Result
no longer needed, as grid and quickVis are initiated in Result
Comment thread packages/web-console/src/consts/index.ts
Comment thread packages/web-console/src/js/console/index.js
Comment thread packages/web-console/src/js/console/index.js
@argshook
argshook requested a review from insmac July 18, 2022 07:58
@argshook
argshook merged commit 5b1c857 into main Jul 22, 2022
@argshook
argshook deleted the web-console/refactor-jquery-modules branch July 22, 2022 10:47
Vaibhav91one pushed a commit to Vaibhav91one/ui that referenced this pull request Oct 28, 2025
* remove unused constants

* refactor globals.js

* move docsearch css import to index.tsx

* make quick-vis as module

* make grid as module

* add @types/jquery

* use grid and quickVis modules in Result

* remove quickVis and grid from console/index.js

they're now used directly in Result

* refactor globals

* remove console-controller

no longer needed, as grid and quickVis are initiated in Result
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants