Skip to content
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

Radio History Pageの作成 #313

Merged
merged 15 commits into from
Jul 4, 2018
Merged

Conversation

kobakazu0429
Copy link
Member

@kobakazu0429 kobakazu0429 commented Jun 23, 2018

closed #301

WHY

WHAT

やったこと

Search

2018-06-23 16 48 05

About #298 をやってしまった... 2018-06-24 21 19 28
RadioHistoryFeature

font-weight: normal

2018-06-25 19 43 28

font-weight: bold

2018-06-25 19 43 40

やってないこと

@@ -0,0 +1,50 @@
const path = require("path");
Copy link
Member

Choose a reason for hiding this comment

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

[ESLint]

Error - 構文解析エラー: 'const' は予約語です。

@kobakazu0429 kobakazu0429 changed the base branch from feature/dev to feature/react June 23, 2018 07:51
@euglena1215
Copy link
Member

@kobakazu0429
スクショ貼ってもらえると助かります!

@kobakazu0429
Copy link
Member Author

@euglena1215 @chanyou0311
軽くレビューお願いします!

@kobakazu0429
Copy link
Member Author

サイドバーとコンテンツの幅が微妙な気がする...
もう少しサイドバーにあげてもいいかもしれない

@@ -1,11 +1,13 @@
import * as React from "react";
import styled from "styled-components";

export const RadioCardWrapper = () => <RadioCardWrapperStyle>radio-card-wrapper</RadioCardWrapperStyle>;
const card = "http://localhost:5000/images/radio-card.png";
Copy link
Member

Choose a reason for hiding this comment

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

📝 このままだとproductionに載せられないのでいい感じに変更する

@@ -1,3 +1,7 @@
export const chkColors = {
orange: "#edb600",
blue: "#8bc8d9",
gray: "#7f8287",
shadow: "#14141433",
Copy link
Member

Choose a reason for hiding this comment

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

こんな書き方できるのか...知らんかった...

import * as React from "react";
import styled from "styled-components";

const feature = "http://localhost:5000/images/radio-history-feature.jpg";
Copy link
Member

Choose a reason for hiding this comment

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

📝 いい感じにする

@euglena1215
Copy link
Member

サンクス!マージしちゃいましょう

@euglena1215 euglena1215 merged commit 94cc1d6 into feature/react Jul 4, 2018
@euglena1215 euglena1215 deleted the react/radio-history-page branch July 4, 2018 06:34
@kobakazu0429 kobakazu0429 mentioned this pull request Jul 28, 2018
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants