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

セッション一覧作成 #102

Merged
merged 8 commits into from
Sep 7, 2018
Merged

Conversation

yasui05821
Copy link
Contributor

Issue #27

@yasui05821
Copy link
Contributor Author

yasui05821 commented Sep 4, 2018

  • ヘッダに`SESSION LIST`追加
  • TOPから遷移できるよう、タイムテーブルセクションにボタン設置 /  アンケート解除
  • スポンサーセッションである事・企業名を明記

@yasui05821
Copy link
Contributor Author

yasui05821 commented Sep 4, 2018

@kamijin-fanta

  • ヘッダに`SESSION LIST`追加
  • TOPから遷移できるよう、タイムテーブルセクションにボタン設置 /  アンケート解除
  • スポンサーセッションである事・企業名を明記

上記の点、修正しました。ご確認よろしくお願いします 🙇
https://deploy-preview-102--scala-kansai2018.netlify.com/

@yasui05821
Copy link
Contributor Author

yasui05821 commented Sep 4, 2018

@kamijin-fanta
こちらのライティングいただけますと助かります。

【セッション一覧】
2018-09-04 23 04 18

【TOPページ(タイムテーブルセクション)】
2018-09-04 23 04 30

@kamijin-fanta
Copy link
Member

@yasui05821 ありがとうございます

  • トップページ: 仮で入れていただいているものをそのまま使って頂きたいです
  • セッション一覧ページ : 特に必要に必要な案内が思いつかないので、無しで大丈夫です

よろしくおねがいします!

@kamijin-fanta
Copy link
Member

@yasui05821 スピーカー情報の入力・↑の文言修正などを直接Pushさせていただきました。手元で作業される場合は、pullしてから行っていただけると助かります。気になった部分として、2点ありました。

  • プロフィール画像が正方形でない場合がある
  • プロフィール等のリンクをクリックして戻ってくると、必ずスクロールが一番上にリセットされる

スクロールに関してはちょっと調べたのですが、nuxtをあまり触ったことが無いので、知見があれば対応いただけるかなと思い書きました。難しそうなら大丈夫です 🙏

@mikakane
Copy link
Contributor

mikakane commented Sep 5, 2018

@kamijin-fanta スクロールイチの維持は難しそうなので target blank 入れるほうが良さそうです。

以下の Issue 参考に Marked をカスタマイズする感じになります。

markedjs/marked#144

@SAMMY7th
Copy link
Contributor

SAMMY7th commented Sep 6, 2018

@yasui05821 @kamijin-fanta
「上級者向け」となっているラベルを「中/上級者向け」としていただいていいでしょうか 🙏

@SAMMY7th SAMMY7th merged commit feaefa0 into scala-kansai:master Sep 7, 2018
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.

4 participants