Skip to content

refactor: convert webview_controller to TypeScript and make it a class

Tomas Vik (OOO back on 2024-07-02) requested to merge 168-only-one-webview into main

We'll need to keep more state in the WebviewController and it makes sense to make it a class so it can contain the state without relying on global module variables. This MR doesn't introduce any changes to the logic or user-facing behaviour.

The refactoring has been done by moving the methods to the class one by one in each commit. I recommend reviewing this MR commit-by-commit and not displaying whitespace changes. That should make the review much simpler.

Context

The webivew_controller is responsible for opening "Overview" tabs for issues and MRs

webview_test_js___272____gitlab-vscode-extension

refactor__convert_we______gitlab-vscode-extension

VS Code Webview official documentation

Related to #168 (closed)

Edited by Tomas Vik (OOO back on 2024-07-02)

Merge request reports