From 45334076b309319e56fe1a340fe85ebee97bed96 Mon Sep 17 00:00:00 2001
From: Jure Triglav <juretriglav@gmail.com>
Date: Thu, 6 Aug 2020 11:17:16 +0200
Subject: [PATCH] feat: use ActionGroup for actions in ReviewerItem
---
.../src/components/sections/ReviewerItem.js | 89 ++++++++-----------
1 file changed, 36 insertions(+), 53 deletions(-)
diff --git a/app/components/component-dashboard/src/components/sections/ReviewerItem.js b/app/components/component-dashboard/src/components/sections/ReviewerItem.js
index 9b0eb40986..01a8fceafe 100644
--- a/app/components/component-dashboard/src/components/sections/ReviewerItem.js
+++ b/app/components/component-dashboard/src/components/sections/ReviewerItem.js
@@ -1,16 +1,8 @@
import React from 'react'
-import { Button } from '@pubsweet/ui'
+import { Action, ActionGroup } from '@pubsweet/ui'
// import Authorize from 'pubsweet-client/src/helpers/Authorize'
-import {
- Item,
- Divider,
- Links,
- LinkContainer,
- Actions,
- ActionContainer,
-} from '../../style'
+import { Item } from '../../style'
-import JournalLink from '../JournalLink'
import VersionTitle from './VersionTitle'
// TODO: only return links if version id is in reviewer.accepted array
@@ -31,52 +23,43 @@ const ReviewerItem = ({ version, journals, currentUser, reviewerRespond }) => {
<VersionTitle version={version} />
{(status === 'accepted' || status === 'completed') && (
- <Links>
- <LinkContainer>
- <JournalLink id={version.id} page="reviews" version={version}>
- {status === 'completed' ? 'Completed' : 'Do Review'}
- </JournalLink>
- </LinkContainer>
- </Links>
+ <ActionGroup>
+ <Action to={`/journal/versions/${version.id}/reviews`}>
+ {status === 'completed' ? 'Completed' : 'Do Review'}
+ </Action>
+ </ActionGroup>
)}
{status === 'invited' && (
- <Actions>
- <ActionContainer>
- <Button
- data-testid="accept-review"
- onClick={() => {
- reviewerRespond({
- variables: {
- currentUserId: currentUser.id,
- action: 'accepted',
- teamId: team.id,
- },
- })
- }}
- >
- accept
- </Button>
- </ActionContainer>
-
- <Divider separator="|" />
-
- <ActionContainer>
- <Button
- onClick={() => {
- reviewerRespond({
- variables: {
- currentUserId: currentUser.id,
- action: 'rejected',
- teamId: team.id,
- },
- })
- }}
- >
- reject
- </Button>
- </ActionContainer>
- </Actions>
+ <ActionGroup>
+ <Action
+ data-testid="accept-review"
+ onClick={() => {
+ reviewerRespond({
+ variables: {
+ currentUserId: currentUser.id,
+ action: 'accepted',
+ teamId: team.id,
+ },
+ })
+ }}
+ >
+ Accept
+ </Action>
+ <Action
+ onClick={() => {
+ reviewerRespond({
+ variables: {
+ currentUserId: currentUser.id,
+ action: 'rejected',
+ teamId: team.id,
+ },
+ })
+ }}
+ >
+ Reject
+ </Action>
+ </ActionGroup>
)}
{status === 'rejected' && 'rejected'}
</Item>
--
GitLab