Skip to content

Commit

Permalink
chore: review fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
Psami-wondah committed Jun 7, 2024
1 parent 85079a4 commit ae3b4be
Show file tree
Hide file tree
Showing 6 changed files with 29 additions and 10 deletions.
1 change: 0 additions & 1 deletion src/app/modules/common/RichEditor/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -147,7 +147,6 @@ export const RichEditor = (props: {
BOLD: {
fontFamily: "GothamNarrow-Bold",
fontWeight: "bold",
fontSize: "14px",
lineHeight: "normal",
},
}}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,9 @@ import { DatasetDataTable } from "app/modules/dataset-upload-module/component/da
import { CssSnackbar, ISnackbarState } from "./previewFragment";
import { ReactComponent as FullScreenIcon } from "../assets/full-screen.svg";
import { ReactComponent as CloseFullScreenIcon } from "../assets/close-full-screen.svg";
import { useRecoilState } from "recoil";
import { homeDisplayAtom } from "app/state/recoil/atoms";
import { ArrowBack } from "@material-ui/icons";

interface Props {
data: any[];
Expand All @@ -25,6 +28,8 @@ export default function FinishedFragment(props: Props) {
const history = useHistory();
const location = useLocation();

const [display, setDisplay] = useRecoilState(homeDisplayAtom);

const setDatasetId = useStoreActions(
(actions) => actions.charts.dataset.setValue
);
Expand Down Expand Up @@ -77,6 +82,25 @@ export default function FinishedFragment(props: Props) {

return (
<div css={dataSetsCss}>
<Link
to={(() => {
setDisplay("data");
return location.search.includes("?fromHome=true") ? "/" : "/explore";
})()}
css={`
display: flex;
align-items: center;
font-size: 14px;
color: #231d2c;
text-decoration: none;
margin-top: 16px;
margin-bottom: 16px;
column-gap: 8px;
`}
data-cy="dataset-back-to-library-btn"
>
<ArrowBack fontSize={"small"} /> Back to Data Library
</Link>
<div
css={`
width: 100%;
Expand All @@ -93,7 +117,7 @@ export default function FinishedFragment(props: Props) {
font-size: 16px;
font-family: "GothamNarrow-Bold", sans-serif;
line-height: 19px;
margin-top: 29px;
margin-top: 19px;
`}
>
{props.description}
Expand All @@ -104,7 +128,7 @@ export default function FinishedFragment(props: Props) {
display: flex;
margin-bottom: 12px;
justify-content: space-between;
margin-top: 55px;
margin-top: 34px;
`}
>
<div
Expand Down
1 change: 0 additions & 1 deletion src/app/modules/home-module/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -446,7 +446,6 @@ export default function HomeModule() {
{displayGrid(searchValue as string, sortValue)}
</div>
</Container>
<Box height={500} />
<HomeFooter />
</React.Fragment>
);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -135,7 +135,7 @@ export default function RowstructureDisplay(props: RowStructureDisplayProps) {
width: 100%;
height: 100%;
position: relative;
margin-bottom: ${!viewOnlyMode ? "0px" : "50px"};
margin-bottom: ${!viewOnlyMode ? "0px" : "16px"};
`}
>
<div
Expand Down
3 changes: 0 additions & 3 deletions src/app/modules/report-module/views/create/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -322,9 +322,6 @@ export const PlaceHolder = (props: PlaceholderProps) => {
return false;
};

console.log(isDroppable(), "candrop");
console.log(itemDragIndex, "isItemDragging");

return (
<div
data-cy="report-row-placeholder"
Expand Down
4 changes: 2 additions & 2 deletions src/app/modules/report-module/views/preview/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -188,7 +188,7 @@ export function ReportPreviewView(props: {
<div
key={"divider" + `${index}`}
css={`
margin: 0 0 50px 0;
margin: 0 0 16px 0;
height: 2px;
width: 100%;
background-color: #cfd4da;
Expand Down Expand Up @@ -228,7 +228,7 @@ export function ReportPreviewView(props: {
/>
);
})}
<Box height={45} />
<Box height={16} />
</Container>

{location.search.includes("?fromLanding=true") && !isAuthenticated ? (
Expand Down

0 comments on commit ae3b4be

Please sign in to comment.