-
Notifications
You must be signed in to change notification settings - Fork 28
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
feat: opossumui-2.0 #2570
Merged
Merged
feat: opossumui-2.0 #2570
Conversation
This file contains 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
ec53ea4
to
aa2dcd1
Compare
aa2dcd1
to
c87d865
Compare
ae99e47
to
fbf99e6
Compare
This was referenced Mar 5, 2024
72eb751
to
7c556e9
Compare
f2c7fc3
to
0c6d8c5
Compare
src/Frontend/web-workers/scripts/__tests__/get-filtered-attributions.test.ts
Outdated
Show resolved
Hide resolved
Not sure that TableConfig.tsx is in the right folder. |
alexzurbonsen
approved these changes
Mar 6, 2024
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.
Impressive Effort. Everything I looked at, I can approve. I left a few comments.
3c02b80
to
d26f86d
Compare
- change default font - improve color scheme and add background wave - merge audit and attribution view to reduce duplication and context switching - group packages by either being an attribution or signal - replace resource popup with new panel to display linked resources - replace file search popup with in-component search fields - introduce bulk actions for signals - add filtering for signals - add filter by license name - add filter for different kinds of incompleteness - highlight incomplete attributions and the missing attributes at all times - improve feature to replace attributions by letting the user select the replacement directly in audit view - improved report view (sticky first column, better display of information, improved scrolling behavior) - generate autocomplete suggestions from filtered signals and attributions and deduplicate attribute values - enable top progress bar to jump to the next actionable resource contained in or after selected resource - update user guide to reflect changes closes #2524 Signed-off-by: Maxim Stykow <maxim.stykow@tngtech.com>
d26f86d
to
d0c56c4
Compare
- add methods to quit or relaunch the app - create error boundary that displays error and offers options to relaunch or quit closes #2564 Signed-off-by: Maxim Stykow <maxim.stykow@tngtech.com>
fix: introduce proper error boundary
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary of changes
Context and reason for change
closes #2524
Pre-release: https://github.com/opossum-tool/OpossumUI/releases/tag/OpossumUI-2.0-beta
How can the changes be tested
Literally every feature of the app needs to be regression tested.
Components to review: