-
Notifications
You must be signed in to change notification settings - Fork 5
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
Improve GUI #165
Improve GUI #165
Conversation
65166b7
to
b5fd612
Compare
GUI follows legacy AB3 mockup Improvements made to suit AssetBook needs
b5fd612
to
c8f763a
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #165 +/- ##
============================================
- Coverage 78.98% 78.83% -0.16%
Complexity 509 509
============================================
Files 73 73
Lines 1542 1545 +3
Branches 136 136
============================================
Hits 1218 1218
- Misses 298 301 +3
Partials 26 26 ☔ View full report in Codecov by Sentry. |
Window size small currently, increased to improve feel ok application Text area does not wrap text, leading to sideway navigation Window size increased to feel more comfortable, and wrapping enabled to stop side scrolling
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.
LGTM, just some minor nits.
b0916c6
to
683aa99
Compare
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.
LGTM. Nice work on the icons!
GUI follows legacy AB3 mockup
Improvements made to suit AssetBook needs
Closes #22, #118 and #163