-
Notifications
You must be signed in to change notification settings - Fork 10
Analyze plugin #147
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Analyze plugin #147
Changes from all commits
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
642a059
feat: add analyze plugin
FilipVrubel 37a78c9
fix: show new app form
FilipVrubel 87420e0
fix: collapse Analyze dropdown
FilipVrubel 3e8db89
style: refactor analyze plugin
FilipVrubel cf6a1d7
fix: mounting of analyze plugin
FilipVrubel 0cfe3ad
feat: create ui for creating job
FilipVrubel 68e8652
feat: update ui for manageing job inputs
FilipVrubel e849e33
style: replace alert with dialog
FilipVrubel 96b80ea
fix: resolve console errors
FilipVrubel 3d76ec3
refactor: make changes based on review in PR #147
FilipVrubel df94bf7
refactor: replace sidePanel with dropdown
FilipVrubel 58229da
feat(analyze): add _captureAnnotation and _waitForAnnotationId helpers
FilipVrubel 6920085
fix(analyze): use strict undefined check for prevModeId restore in _c…
FilipVrubel 4b2c433
feat(analyze): thread fw reference to _createAppCard
FilipVrubel d017f04
feat(analyze): wire onCapture callback from _createAppCard to _buildI…
FilipVrubel 79f04b7
refactor(analyze): remove ROI fetch from _buildInputsForm
FilipVrubel 0b18418
feat(analyze): replace ROI dropdown with annotation capture button
FilipVrubel b4a7dc3
fix(analyze): fix fw visibility during id polling, wsi omission, and …
FilipVrubel 0da5949
merge: sync with origin/release/v3
FilipVrubel 33fdc0f
chore: rename analyze plugin to analyze-dev as it is still in develop…
FilipVrubel b1a0730
Merge branch 'release/v3' into dev-vrubel
FilipVrubel fee30b4
fix: use global.UI instead of relative paths
FilipVrubel File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,25 @@ | ||
|
|
||
| > **IN DEVELOPMENT — NOT YET FUNCTIONAL.** This plugin is under active development and is not ready for use in production or testing environments. Features may be incomplete, broken, or subject to change without notice. | ||
|
|
||
| Analyze plugin | ||
| ======================== | ||
|
|
||
| Purpose | ||
| ------- | ||
| Adds an "Analyze" tab to the AppBar with two actions: a "Run Recent →" anchor that opens a right-side recent-jobs panel, and "Create New App" which creates floating window with new app form. | ||
|
|
||
| Files | ||
| ----- | ||
| - `analyzeDropdown.mjs` - registers the tab and wires dropdown items. | ||
| - `newAppForm.mjs` - the floating form used by "Create New App". | ||
|
|
||
| How to use | ||
| ---------- | ||
| - Provide recent jobs by passing `params.recentJobs` or saving `recentJobs` via plugin options. | ||
| - Handle job clicks by implementing `onJobClick({ index, label })` on the plugin instance. | ||
| - Provide `params.onCreate` to receive form submission data from `NewAppForm`. | ||
|
|
||
| Implementation notes | ||
| -------------------- | ||
| - UI behaviors (menu, positioning, hover) are implemented in `SidePanel` (`setMenu`, `showNear`, `scheduleHide`, `cancelHide`) — reuse it for other flyouts. | ||
| - `SidePanel.hide()` currently removes the element; consider switching to `display:none` if you need faster show/hide cycles. | ||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Documentation? Awesome!