From b9b2769563aa5e9ddda464f93af88824f8143246 Mon Sep 17 00:00:00 2001 From: Tom Moor Date: Sat, 2 Sep 2017 16:58:24 -0700 Subject: [PATCH 1/3] Fixes: Documents marked as starred incorrectly --- server/presenters/document.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/server/presenters/document.js b/server/presenters/document.js index 4dce75bb3..7d06c3210 100644 --- a/server/presenters/document.js +++ b/server/presenters/document.js @@ -31,7 +31,7 @@ async function present(ctx: Object, document: Document, options: ?Options) { lastViewedAt: undefined, team: document.teamId, collaborators: [], - starred: !!document.starred, + starred: !!document.starred.length, collaboratorCount: undefined, collection: undefined, views: undefined, From 44e635cbc7105a03eb6450fa38bed392c08e8833 Mon Sep 17 00:00:00 2001 From: Tom Moor Date: Sat, 2 Sep 2017 18:43:12 -0700 Subject: [PATCH 2/3] Closes #189 --- .../components/DocumentPreview/DocumentPreview.js | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/frontend/components/DocumentPreview/DocumentPreview.js b/frontend/components/DocumentPreview/DocumentPreview.js index 30cc0f845..6703e896f 100644 --- a/frontend/components/DocumentPreview/DocumentPreview.js +++ b/frontend/components/DocumentPreview/DocumentPreview.js @@ -19,13 +19,9 @@ const StyledStar = styled(StarIcon)` top: 2px; position: relative; margin-left: 4px; - opacity: ${props => (props.solid ? 1 : 0.25)}; + opacity: ${props => (props.solid ? '1 !important' : 0)}; transition: opacity 100ms ease-in-out; - &:hover { - opacity: 1; - } - svg { width: 1em; height: 1em; @@ -48,6 +44,14 @@ const DocumentLink = styled(Link)` background: ${color.smokeLight}; border: 2px solid ${color.smoke}; outline: none; + + ${StyledStar} { + opacity: .25; + + &:hover { + opacity: 1; + } + } } &:focus { From 95182057a62cbaefe4620ad7c22606b5f987f4cd Mon Sep 17 00:00:00 2001 From: Tom Moor Date: Sat, 2 Sep 2017 18:44:41 -0700 Subject: [PATCH 3/3] :green_heart: --- server/presenters/document.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/server/presenters/document.js b/server/presenters/document.js index 7d06c3210..0ca0ea186 100644 --- a/server/presenters/document.js +++ b/server/presenters/document.js @@ -31,7 +31,7 @@ async function present(ctx: Object, document: Document, options: ?Options) { lastViewedAt: undefined, team: document.teamId, collaborators: [], - starred: !!document.starred.length, + starred: !!(document.starred && document.starred.length), collaboratorCount: undefined, collection: undefined, views: undefined,