Skip to content

MellowList#441

Open
cpz-crafts wants to merge 27 commits intoTechnigo:masterfrom
cpz-crafts:master
Open

MellowList#441
cpz-crafts wants to merge 27 commits intoTechnigo:masterfrom
cpz-crafts:master

Conversation

@cpz-crafts
Copy link
Copy Markdown

@cpz-crafts cpz-crafts commented Apr 23, 2023

Copy link
Copy Markdown

@dannebrob dannebrob left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work Carol! It fulfills all requirements of blue level, and beyond. Be proud! I like the design and nice addition with the playlist on top, I liked that alot.

}

const getDueDate = (date) => {
switch (date) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice use case of switch / case


import { createSlice } from '@reduxjs/toolkit';

export const playlist = createSlice({
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Interesting way to store the Spotify link, any reason why you did it like this? Want to add more playlists?

import { createSlice } from '@reduxjs/toolkit';

const getInitialState = () => {
const projectsFromLocalStorage = JSON.parse(localStorage.getItem('projectsList'));
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice implementation of localStorage

<meta property="og:title" content="MellowList" />
<meta property="og:description" content="Get in the zone and conquer your to-do list" />
<meta property="og:url" content="https://peppy-stroopwafel-0a74de.netlify.app/" />
<meta property="og:type" content="website" />
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great that you added meta data :D


return (
<PlaylistContainer>
<span>{name}</span>
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You might want to add css rule to display: none on smaller screen-sizes for a more clean look

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants