-
Notifications
You must be signed in to change notification settings - Fork 37
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
Docviewer stlye revamp #658
Conversation
|
Codecov Report
@@ Coverage Diff @@
## develop #658 +/- ##
===========================================
+ Coverage 91.92% 92.02% +0.10%
===========================================
Files 212 212
Lines 5647 5646 -1
Branches 1415 1415
===========================================
+ Hits 5191 5196 +5
+ Misses 456 450 -6
Continue to review full report at Codecov.
|
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.
One small thing. 🍜
packages/sn-document-viewer-react/src/components/document-widgets/ToggleThumbnails.tsx
Outdated
Show resolved
Hide resolved
…ets/ToggleThumbnails.tsx Co-authored-by: Zoltán Bedi <zoltan.bedi@gmail.com>
When opening a document the command palette's icon in the header becomes black
The docviewer's header should be the same color as the sidebar with no shadow
The icons color should be the same as the sidebar icons, when active the color should be #26A69A
The comment icon is too far away from the close button
When the comments are opened the add comment link should be under the title
The cancel button should be outlined Screen Shot 2020-04-23 at 14.48.14.png -> should be white/black without outlined style
Question: Is there a need for multiple rotate functions (there are two separate right now) and multiple resizing tools?
Yes. One is for the page and one is for the document
+1: Approve dialog button style change