Skip to content
  • P
    Projects
  • G
    Groups
  • S
    Snippets
  • Help

Yutaka Kobayashi / jsStudy

  • This project
    • Loading...
  • Sign in
Go to a project
  • Project
  • Repository
  • Issues 0
  • Merge Requests 1
  • Pipelines
  • Wiki
  • Snippets
  • Settings
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Open
Merge Request !1 opened Dec 16, 2022 by Yutaka Kobayashi@yutaka.kobayashi

レビュー用マジリク

レビュー用のマジリクです。

×

Check out, review, and merge locally

Step 1. Fetch and check out the branch for this merge request

git fetch origin
git checkout -b develop origin/develop

Step 2. Review the changes locally

Step 3. Merge the branch and fix any conflicts that come up

git checkout review
git merge --no-ff develop

Step 4. Push the result of the merge to GitLab

git push origin review

Note that pushing to GitLab requires write access to this repository.

Tip: You can also checkout merge requests locally by following these guidelines.

  • Discussion 15
  • Commits 37
  • Changes 24
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Yutaka Kobayashi @yutaka.kobayashi

    added 1 commit

    • a3fc6b7b - node_modulesをgitから削除

    Compare with previous version

    Dec 20, 2022

    added 1 commit

    • a3fc6b7b - node_modulesをgitから削除

    Compare with previous version

    added 1 commit * a3fc6b7b - node_modulesをgitから削除 [Compare with previous version](https://gitlab.sonicjam.jp/yutaka.kobayashi/jsStudy/merge_requests/1/diffs?diff_id=3187&start_sha=603277ba37a703354c7b48c096577b68981737ed)
    Toggle commit list
  • Yutaka Kobayashi @yutaka.kobayashi

    added 1 commit

    • 8cb48e44 - Delete home.js

    Compare with previous version

    Dec 20, 2022

    added 1 commit

    • 8cb48e44 - Delete home.js

    Compare with previous version

    added 1 commit * 8cb48e44 - Delete home.js [Compare with previous version](https://gitlab.sonicjam.jp/yutaka.kobayashi/jsStudy/merge_requests/1/diffs?diff_id=3188&start_sha=a3fc6b7b18a6b8f6653adb5b002055b31cf4f696)
    Toggle commit list
  • Yutaka Kobayashi @yutaka.kobayashi

    added 1 commit

    • 991e3602 - Delete index.html

    Compare with previous version

    Dec 20, 2022

    added 1 commit

    • 991e3602 - Delete index.html

    Compare with previous version

    added 1 commit * 991e3602 - Delete index.html [Compare with previous version](https://gitlab.sonicjam.jp/yutaka.kobayashi/jsStudy/merge_requests/1/diffs?diff_id=3189&start_sha=8cb48e44bc472f2dafdb6164dd181e864cc26d64)
    Toggle commit list
  • Yutaka Kobayashi @yutaka.kobayashi

    added 1 commit

    • e078af7e - Delete home.min.css.map

    Compare with previous version

    Dec 20, 2022

    added 1 commit

    • e078af7e - Delete home.min.css.map

    Compare with previous version

    added 1 commit * e078af7e - Delete home.min.css.map [Compare with previous version](https://gitlab.sonicjam.jp/yutaka.kobayashi/jsStudy/merge_requests/1/diffs?diff_id=3190&start_sha=991e360248acdd051543166e3fd797e03cf67ff2)
    Toggle commit list
  • Yutaka Kobayashi @yutaka.kobayashi

    added 1 commit

    • ccad1efa - Delete home.min.css

    Compare with previous version

    Dec 20, 2022

    added 1 commit

    • ccad1efa - Delete home.min.css

    Compare with previous version

    added 1 commit * ccad1efa - Delete home.min.css [Compare with previous version](https://gitlab.sonicjam.jp/yutaka.kobayashi/jsStudy/merge_requests/1/diffs?diff_id=3191&start_sha=e078af7ed05a2d1c76c3142eebfe4c483b8f9bee)
    Toggle commit list
  • Yutaka Kobayashi @yutaka.kobayashi

    added 1 commit

    • dde562ee - Delete modal.min.css.map

    Compare with previous version

    Dec 20, 2022

    added 1 commit

    • dde562ee - Delete modal.min.css.map

    Compare with previous version

    added 1 commit * dde562ee - Delete modal.min.css.map [Compare with previous version](https://gitlab.sonicjam.jp/yutaka.kobayashi/jsStudy/merge_requests/1/diffs?diff_id=3192&start_sha=ccad1efa5336d83aaecbf1ebf6e0819492d16ec7)
    Toggle commit list
  • Yutaka Kobayashi @yutaka.kobayashi

    added 1 commit

    • cfba98c1 - Delete modal.min.css

    Compare with previous version

    Dec 20, 2022

    added 1 commit

    • cfba98c1 - Delete modal.min.css

    Compare with previous version

    added 1 commit * cfba98c1 - Delete modal.min.css [Compare with previous version](https://gitlab.sonicjam.jp/yutaka.kobayashi/jsStudy/merge_requests/1/diffs?diff_id=3193&start_sha=dde562eebc074305887bf674c8b47d793b1a1f03)
    Toggle commit list
  • Yutaka Kobayashi @yutaka.kobayashi

    added 1 commit

    • b9e85749 - Delete magnific-popup.css.map

    Compare with previous version

    Dec 20, 2022

    added 1 commit

    • b9e85749 - Delete magnific-popup.css.map

    Compare with previous version

    added 1 commit * b9e85749 - Delete magnific-popup.css.map [Compare with previous version](https://gitlab.sonicjam.jp/yutaka.kobayashi/jsStudy/merge_requests/1/diffs?diff_id=3194&start_sha=cfba98c177deafee9d9fb058cd92158d7414eb83)
    Toggle commit list
  • Yutaka Kobayashi @yutaka.kobayashi

    added 1 commit

    • e8534425 - Delete magnific-popup.css

    Compare with previous version

    Dec 20, 2022

    added 1 commit

    • e8534425 - Delete magnific-popup.css

    Compare with previous version

    added 1 commit * e8534425 - Delete magnific-popup.css [Compare with previous version](https://gitlab.sonicjam.jp/yutaka.kobayashi/jsStudy/merge_requests/1/diffs?diff_id=3195&start_sha=b9e85749656c1c672e85663d6fede194bbf52534)
    Toggle commit list
  • No avatar
    Tetsuya Maki
    @Tetsuya.Maki started a discussion on an outdated diff Dec 20, 2022
    Resolved by Yutaka Kobayashi Jan 13, 2023
    develop/src/assets/js/home.js 0 → 100644
    1 class Home {
    2 constructor() {
    3 this.modal();
    4 this.fadeIn();
    5 }
    6
    7 modal() {
    8 const body = document.getElementsByTagName("body")[0];
    9 const btn = document.querySelectorAll(".novelList__button");
    10 const modalBack = document.querySelector(".modalBackground");
    11 const modal = document.querySelectorAll(".modal");
    12 const closeBtn = document.querySelector(".modal__button");
    13
    • Tetsuya Maki @Tetsuya.Maki commented Dec 20, 2022
      Master

      要素の取得の方法の使い分けの意図を教えてください。

      • getElementsByTagName
      • querySelectorAll
      • querySelector
      Edited Dec 20, 2022 by Tetsuya Maki
      要素の取得の方法の使い分けの意図を教えてください。 * getElementsByTagName * querySelectorAll * querySelector
    • Yutaka Kobayashi @yutaka.kobayashi commented Dec 20, 2022
      Master

      getElementsByTagName
      body要素を取得したかったのですが、class名が付いていなかったので、タグごと取得しました。
      querySelectorAllとquerySelector
      実装の始めでは、下記のルールで記述していました。
      クラスが複数個あるもの→「querySelectorAll」
      1つしかないもの→「querySelector」

      Edited Dec 20, 2022 by Yutaka Kobayashi
      **getElementsByTagName**<br> body要素を取得したかったのですが、class名が付いていなかったので、タグごと取得しました。<br> **querySelectorAllとquerySelector**<br> 実装の始めでは、下記のルールで記述していました。<br> クラスが複数個あるもの→「querySelectorAll」<br> 1つしかないもの→「querySelector」
    • Tetsuya Maki @Tetsuya.Maki commented Dec 26, 2022
      Master

      @yutaka.kobayashi querySelectorを使用すればbodyも取得できるのでできれば使い分けずにquerySelectorで統一でも良いかなと思いました!

      Edited Jan 13, 2023
      @yutaka.kobayashi querySelectorを使用すればbodyも取得できるのでできれば使い分けずにquerySelectorで統一でも良いかなと思いました!
  • No avatar
    Tetsuya Maki
    @Tetsuya.Maki started a discussion on an outdated diff Dec 20, 2022
    Resolved by Yutaka Kobayashi Dec 20, 2022
    develop/src/assets/js/home.js 0 → 100644
    1 class Home {
    2 constructor() {
    3 this.modal();
    4 this.fadeIn();
    5 }
    6
    7 modal() {
    8 const body = document.getElementsByTagName("body")[0];
    9 const btn = document.querySelectorAll(".novelList__button");
    • Tetsuya Maki @Tetsuya.Maki commented Dec 20, 2022
      Master

      htmlの命名と被るところがありますが、変数名は無理に省略しない方が明確かとおもうのと、btnだと抽象度が高すぎて一見してどの要素が格納されているか分かりづらいので命名を再考してみまよう。

      Edited Dec 20, 2022
      htmlの命名と被るところがありますが、変数名は無理に省略しない方が明確かとおもうのと、btnだと抽象度が高すぎて一見してどの要素が格納されているか分かりづらいので命名を再考してみまよう。
  • Tetsuya Maki @Tetsuya.Maki commented Dec 20, 2022
    Master

    全体的に考え方として下記のようにするとさらに良くなるかと思います。

    今回の場合「モーダルを制御する」機能、「要素がフェードインして掲出する」機能が汎用的に使用されることを考えると、この二つの機能を別ファイルに切り出してコンポーネント化し、該当のページでimportして使用する方がページ数が多い時に別ページで使いたくなった際に汎用的に使えるかと思います。

    全体的に考え方として下記のようにするとさらに良くなるかと思います。 今回の場合「モーダルを制御する」機能、「要素がフェードインして掲出する」機能が汎用的に使用されることを考えると、この二つの機能を別ファイルに切り出してコンポーネント化し、該当のページでimportして使用する方がページ数が多い時に別ページで使いたくなった際に汎用的に使えるかと思います。
  • No avatar
    Tetsuya Maki
    @Tetsuya.Maki started a discussion on an outdated diff Dec 20, 2022
    Resolved by Yutaka Kobayashi Dec 20, 2022
    develop/src/assets/js/home.js 0 → 100644
    30 modal[i].classList.remove("open");
    31 }
    32 });
    33
    34 function menuClick() {
    35 const menus = document.getElementsByClassName("close");
    36 body.classList.remove("open");
    37 modalBack.classList.remove("open");
    38 const hogeElements = document.querySelectorAll(".modal");
    39
    40 hogeElements.forEach((item) => {
    41 item.classList.remove("open");
    42 });
    43 }
    44
    45 const menus = document.getElementsByClassName("close");
    • Tetsuya Maki @Tetsuya.Maki commented Dec 20, 2022
      Master

      ここも変数名と取得する要素の内容がマッチしていないので命名を再考しましょう。

      Edited Dec 20, 2022
      ここも変数名と取得する要素の内容がマッチしていないので命名を再考しましょう。
  • No avatar
    Tetsuya Maki
    @Tetsuya.Maki started a discussion on an outdated diff Dec 20, 2022
    Resolved by Yutaka Kobayashi Dec 21, 2022
    develop/src/assets/js/home.js 0 → 100644
    39
    40 hogeElements.forEach((item) => {
    41 item.classList.remove("open");
    42 });
    43 }
    44
    45 const menus = document.getElementsByClassName("close");
    46 for (let i = 0; i < menus.length; i++) {
    47 menus[i].addEventListener("click", menuClick, false);
    48 }
    49
    50 const menus02 = document.getElementsByClassName("modal__button");
    51 for (let i = 0; i < menus.length; i++) {
    52 menus02[i].addEventListener("click", menuClick, false);
    53 }
    54 }
    • Tetsuya Maki @Tetsuya.Maki commented Dec 20, 2022
      Master

      この一連のイベント付与ですがクラス名が違う要素に対して同じ関数をアタッチしているので、例えばjs用のクラスを用意してそのクラスがあるものに対してイベントを付与する形にするとよいかなと思いますがどうでしょう?

      Edited Dec 21, 2022 by Tetsuya Maki
      この一連のイベント付与ですがクラス名が違う要素に対して同じ関数をアタッチしているので、例えばjs用のクラスを用意してそのクラスがあるものに対してイベントを付与する形にするとよいかなと思いますがどうでしょう?
  • No avatar
    Tetsuya Maki
    @Tetsuya.Maki started a discussion on an outdated diff Dec 20, 2022
    Resolved by Yutaka Kobayashi Dec 21, 2022
    develop/src/assets/js/home.js 0 → 100644
    58 function scroll_effect() {
    59 var element = document.getElementsByClassName("scrollUp");
    60 if (!element) return;
    61
    62 var scrollY = window.pageYOffset;
    63 var windowHeight = window.innerHeight;
    64 var showTiming = 200;
    65 for (var i = 0; i < element.length; i++) {
    66 var elementClientRect = element[i].getBoundingClientRect();
    67 var elementY = scrollY + elementClientRect.top;
    68 if (scrollY > elementY - windowHeight + showTiming) {
    69 element[i].classList.add("is-show");
    70 }
    71 }
    72 }
    73 window.addEventListener("scroll", scroll_effect);
    • Tetsuya Maki @Tetsuya.Maki commented Dec 20, 2022
      Master

      イベントの付与はconstructor関数内で行うとよいでしょう。

      Edited Dec 21, 2022
      イベントの付与はconstructor関数内で行うとよいでしょう。
    • Tetsuya Maki @Tetsuya.Maki commented Dec 20, 2022
      Master

      scroll系のイベントは間引きの処理をいれてあげましょう。

      Edited Dec 21, 2022
      scroll系のイベントは間引きの処理をいれてあげましょう。
  • No avatar
    Tetsuya Maki
    @Tetsuya.Maki started a discussion on an outdated diff Dec 20, 2022
    Resolved by Yutaka Kobayashi Dec 26, 2022
    develop/src/assets/js/home.js 0 → 100644
    43 }
    44
    45 const menus = document.getElementsByClassName("close");
    46 for (let i = 0; i < menus.length; i++) {
    47 menus[i].addEventListener("click", menuClick, false);
    48 }
    49
    50 const menus02 = document.getElementsByClassName("modal__button");
    51 for (let i = 0; i < menus.length; i++) {
    52 menus02[i].addEventListener("click", menuClick, false);
    53 }
    54 }
    55
    56 // フェードイン
    57 fadeIn() {
    58 function scroll_effect() {
    • Tetsuya Maki @Tetsuya.Maki commented Dec 20, 2022
      Master

      この場合内部的にfunctionで別関数を定義している意図はありますか?

      Edited Dec 26, 2022
      この場合内部的にfunctionで別関数を定義している意図はありますか?
    • Yutaka Kobayashi @yutaka.kobayashi commented Dec 21, 2022
      Master

      window.addEventListener("scroll", () => {...で記述すると閉じ括弧までの行数が長く見えづらかったので別関数にしました。

      Edited Dec 26, 2022
      ` window.addEventListener("scroll", () => {...`で記述すると閉じ括弧までの行数が長く見えづらかったので別関数にしました。
  • No avatar
    Tetsuya Maki
    @Tetsuya.Maki started a discussion on an outdated diff Dec 20, 2022
    Resolved by Yutaka Kobayashi Dec 21, 2022
    develop/src/assets/js/home.js 0 → 100644
    20 modalBack.classList.add("open");
    21 modal[i].classList.add("open");
    22 }
    23 }
    24
    25 // モーダルを非表示にする
    26 closeBtn.addEventListener("click", function () {
    27 body.classList.remove("open");
    28 modalBack.classList.remove("open");
    29 for (let i = 0; i < modal.length; i++) {
    30 modal[i].classList.remove("open");
    31 }
    32 });
    33
    34 function menuClick() {
    35 const menus = document.getElementsByClassName("close");
    • Tetsuya Maki @Tetsuya.Maki commented Dec 20, 2022
      Master

      このように同一要素を複数回使用する際には、constructor関数内で取得・定義してあげると使いわしが効くのでよいかと思います。

      Edited Dec 21, 2022
      このように同一要素を複数回使用する際には、constructor関数内で取得・定義してあげると使いわしが効くのでよいかと思います。
  • No avatar
    Tetsuya Maki
    @Tetsuya.Maki started a discussion on an outdated diff Dec 20, 2022
    Resolved by Yutaka Kobayashi Dec 21, 2022
    develop/src/assets/js/home.js 0 → 100644
    11 const modal = document.querySelectorAll(".modal");
    12 const closeBtn = document.querySelector(".modal__button");
    13
    14 // モーダルを表示する
    15 for (let i = 0; i < btn.length; i++) {
    16 btn[i].addEventListener("click", addOpen);
    17 function addOpen(e) {
    18 e.preventDefault();
    19 body.classList.add("open");
    20 modalBack.classList.add("open");
    21 modal[i].classList.add("open");
    22 }
    23 }
    24
    25 // モーダルを非表示にする
    26 closeBtn.addEventListener("click", function () {
    • Tetsuya Maki @Tetsuya.Maki commented Dec 20, 2022
      Master

      thisを動的に決定させる必要がなければ、ES6記法ではアロー関数を使用することが望ましいです。

      ▼ アロー関数と従来の関数の違い

      https://typescriptbook.jp/reference/functions/function-expression-vs-arrow-functions

      Edited Dec 21, 2022 by Tetsuya Maki
      thisを動的に決定させる必要がなければ、ES6記法ではアロー関数を使用することが望ましいです。 ▼ アロー関数と従来の関数の違い https://typescriptbook.jp/reference/functions/function-expression-vs-arrow-functions
  • No avatar
    Tetsuya Maki
    @Tetsuya.Maki started a discussion on an outdated diff Dec 20, 2022
    Resolved by Yutaka Kobayashi Dec 21, 2022
    develop/src/assets/js/home.js 0 → 100644
    23 }
    24
    25 // モーダルを非表示にする
    26 closeBtn.addEventListener("click", function () {
    27 body.classList.remove("open");
    28 modalBack.classList.remove("open");
    29 for (let i = 0; i < modal.length; i++) {
    30 modal[i].classList.remove("open");
    31 }
    32 });
    33
    34 function menuClick() {
    35 const menus = document.getElementsByClassName("close");
    36 body.classList.remove("open");
    37 modalBack.classList.remove("open");
    38 const hogeElements = document.querySelectorAll(".modal");
    • Tetsuya Maki @Tetsuya.Maki commented Dec 20, 2022
      Master

      変数名はちゃんと命名しましょう。

      Edited Dec 21, 2022
      変数名はちゃんと命名しましょう。
  • No avatar
    Tetsuya Maki
    @Tetsuya.Maki started a discussion on an outdated diff Dec 20, 2022
    Resolved by Yutaka Kobayashi Dec 21, 2022
    develop/src/assets/js/home.js 0 → 100644
    32 });
    33
    34 function menuClick() {
    35 const menus = document.getElementsByClassName("close");
    36 body.classList.remove("open");
    37 modalBack.classList.remove("open");
    38 const hogeElements = document.querySelectorAll(".modal");
    39
    40 hogeElements.forEach((item) => {
    41 item.classList.remove("open");
    42 });
    43 }
    44
    45 const menus = document.getElementsByClassName("close");
    46 for (let i = 0; i < menus.length; i++) {
    47 menus[i].addEventListener("click", menuClick, false);
    • Tetsuya Maki @Tetsuya.Maki commented Dec 20, 2022
      Master

      addEventListenerの第三引数がどういった意味を持つのでしょう。 調べてみてください!

      Edited Dec 21, 2022
      addEventListenerの第三引数がどういった意味を持つのでしょう。 調べてみてください!
  • Yutaka Kobayashi @yutaka.kobayashi

    added 3 commits

    • 54321859 - 閉じる用のjs用のクラスを用意してそのクラスがあるものに対してイベントを付与する形に変更
    • 3b78055d - スクロールを間引くように変更
    • f74a4f9f - ✨レビュー対応

    Compare with previous version

    Dec 21, 2022

    added 3 commits

    • 54321859 - 閉じる用のjs用のクラスを用意してそのクラスがあるものに対してイベントを付与する形に変更
    • 3b78055d - スクロールを間引くように変更
    • f74a4f9f - ✨レビュー対応

    Compare with previous version

    added 3 commits * 54321859 - 閉じる用のjs用のクラスを用意してそのクラスがあるものに対してイベントを付与する形に変更 * 3b78055d - スクロールを間引くように変更 * f74a4f9f - ✨レビュー対応 [Compare with previous version](https://gitlab.sonicjam.jp/yutaka.kobayashi/jsStudy/merge_requests/1/diffs?diff_id=3197&start_sha=e8534425ac6f17c6ada4f35e9c508b7c6db8cf64)
    Toggle commit list
  • Yutaka Kobayashi @yutaka.kobayashi

    resolved all discussions

    Dec 26, 2022

    resolved all discussions

    resolved all discussions
    Toggle commit list
  • Yutaka Kobayashi @yutaka.kobayashi

    added 1 commit

    • 0ab81146 - ✨変数名をキャメルケースで統一

    Compare with previous version

    Dec 26, 2022

    added 1 commit

    • 0ab81146 - ✨変数名をキャメルケースで統一

    Compare with previous version

    added 1 commit * 0ab81146 - ✨変数名をキャメルケースで統一 [Compare with previous version](https://gitlab.sonicjam.jp/yutaka.kobayashi/jsStudy/merge_requests/1/diffs?diff_id=3198&start_sha=f74a4f9f17dce72de7b8691e6f63bce11c4294d3)
    Toggle commit list
  • Yutaka Kobayashi @yutaka.kobayashi

    added 1 commit

    • 14ebb36f - ファイルを外部からimport

    Compare with previous version

    Dec 27, 2022

    added 1 commit

    • 14ebb36f - ファイルを外部からimport

    Compare with previous version

    added 1 commit * 14ebb36f - ファイルを外部からimport [Compare with previous version](https://gitlab.sonicjam.jp/yutaka.kobayashi/jsStudy/merge_requests/1/diffs?diff_id=3199&start_sha=0ab81146130a6e10229ec2fe09ff85fe90c28c28)
    Toggle commit list
  • Yutaka Kobayashi @yutaka.kobayashi

    resolved all discussions

    Jan 13, 2023

    resolved all discussions

    resolved all discussions
    Toggle commit list
  • No avatar
    Yutaka Kobayashi
    @yutaka.kobayashi started a discussion on commit 62edc9b6 Jan 13, 2023
    • Yutaka Kobayashi @yutaka.kobayashi

      mentioned in commit c0ec1fd1

      Jan 13, 2023

      mentioned in commit c0ec1fd1

      mentioned in commit c0ec1fd16665e02ee986b63e0b224f3dfc715f1e
      Toggle commit list
  • Yutaka Kobayashi @yutaka.kobayashi

    added 4 commits

    • 93cc70e6 - modalが動いたので一旦保存
    • 8a103ee0 - フェードインを外部ファイルに変更
    • 62edc9b6 - コード全体の手直し
    • c0ec1fd1 - Merge commit '62edc9b6' into develop

    Compare with previous version

    Jan 13, 2023

    added 4 commits

    • 93cc70e6 - modalが動いたので一旦保存
    • 8a103ee0 - フェードインを外部ファイルに変更
    • 62edc9b6 - コード全体の手直し
    • c0ec1fd1 - Merge commit '62edc9b6' into develop

    Compare with previous version

    added 4 commits * 93cc70e6 - modalが動いたので一旦保存 * 8a103ee0 - フェードインを外部ファイルに変更 * 62edc9b6 - コード全体の手直し * c0ec1fd1 - Merge commit &#x27;62edc9b6577a872ce95b85c866ef52ea7890671d&#x27; into develop [Compare with previous version](https://gitlab.sonicjam.jp/yutaka.kobayashi/jsStudy/merge_requests/1/diffs?diff_id=3208&start_sha=14ebb36f88998ffb08a2810fdfebcdec62f72604)
    Toggle commit list
  • Yutaka Kobayashi @yutaka.kobayashi

    added 1 commit

    • 754b7779 - 変更

    Compare with previous version

    Jan 13, 2023

    added 1 commit

    • 754b7779 - 変更

    Compare with previous version

    added 1 commit * 754b7779 - 変更 [Compare with previous version](https://gitlab.sonicjam.jp/yutaka.kobayashi/jsStudy/merge_requests/1/diffs?diff_id=3209&start_sha=c0ec1fd16665e02ee986b63e0b224f3dfc715f1e)
    Toggle commit list
  • Yutaka Kobayashi @yutaka.kobayashi

    added 1 commit

    • 04565349 - home.js修正

    Compare with previous version

    Jan 16, 2023

    added 1 commit

    • 04565349 - home.js修正

    Compare with previous version

    added 1 commit * 04565349 - home.js修正 [Compare with previous version](https://gitlab.sonicjam.jp/yutaka.kobayashi/jsStudy/merge_requests/1/diffs?diff_id=3211&start_sha=754b7779388fbd76a9ce311941fee65007747063)
    Toggle commit list
  • Yutaka Kobayashi @yutaka.kobayashi

    added 1 commit

    • 0315452d - Co-authored-by: McTex <McTex@users.noreply.github.com>

    Compare with previous version

    Jan 17, 2023

    added 1 commit

    • 0315452d - Co-authored-by: McTex <McTex@users.noreply.github.com>

    Compare with previous version

    added 1 commit * 0315452d - Co-authored-by: McTex &lt;McTex@users.noreply.github.com&gt; [Compare with previous version](https://gitlab.sonicjam.jp/yutaka.kobayashi/jsStudy/merge_requests/1/diffs?diff_id=3214&start_sha=045653497d8a26886ff6cb9dbf043bd98ae5149c)
    Toggle commit list
  • Yutaka Kobayashi @yutaka.kobayashi

    added 1 commit

    • ec58005a - 一旦コミット

    Compare with previous version

    Jan 18, 2023

    added 1 commit

    • ec58005a - 一旦コミット

    Compare with previous version

    added 1 commit * ec58005a - 一旦コミット [Compare with previous version](https://gitlab.sonicjam.jp/yutaka.kobayashi/jsStudy/merge_requests/1/diffs?diff_id=3216&start_sha=0315452d5d00a5ef7d50dbbc1406033a49beb57b)
    Toggle commit list
  • Yutaka Kobayashi @yutaka.kobayashi

    added 1 commit

    • ff9e46a7 - モーダルとフェードインを動くように修正

    Compare with previous version

    Jan 19, 2023

    added 1 commit

    • ff9e46a7 - モーダルとフェードインを動くように修正

    Compare with previous version

    added 1 commit * ff9e46a7 - モーダルとフェードインを動くように修正 [Compare with previous version](https://gitlab.sonicjam.jp/yutaka.kobayashi/jsStudy/merge_requests/1/diffs?diff_id=3223&start_sha=ec58005a234fde32e130ae911ad70b73eec679ce)
    Toggle commit list
  • Yutaka Kobayashi @yutaka.kobayashi

    added 1 commit

    • 998da855 - constructor内の名前変更

    Compare with previous version

    Jan 19, 2023

    added 1 commit

    • 998da855 - constructor内の名前変更

    Compare with previous version

    added 1 commit * 998da855 - constructor内の名前変更 [Compare with previous version](https://gitlab.sonicjam.jp/yutaka.kobayashi/jsStudy/merge_requests/1/diffs?diff_id=3224&start_sha=ff9e46a76946c9874802479e84e71761183dbfe5)
    Toggle commit list
  • Yutaka Kobayashi @yutaka.kobayashi

    added 1 commit

    • 1bde9354 - 不要ファイル削除

    Compare with previous version

    Jan 19, 2023

    added 1 commit

    • 1bde9354 - 不要ファイル削除

    Compare with previous version

    added 1 commit * 1bde9354 - 不要ファイル削除 [Compare with previous version](https://gitlab.sonicjam.jp/yutaka.kobayashi/jsStudy/merge_requests/1/diffs?diff_id=3225&start_sha=998da85591a8c2d39ec6f61bb7e8ad8dda1871f0)
    Toggle commit list
  • Yutaka Kobayashi @yutaka.kobayashi

    added 1 commit

    • fb184766 - てんとう虫

    Compare with previous version

    Jan 20, 2023

    added 1 commit

    • fb184766 - てんとう虫

    Compare with previous version

    added 1 commit * fb184766 - てんとう虫 [Compare with previous version](https://gitlab.sonicjam.jp/yutaka.kobayashi/jsStudy/merge_requests/1/diffs?diff_id=3226&start_sha=1bde93546c19843a8739df126018351bb98323b9)
    Toggle commit list
  • Write
  • Preview
Markdown is supported
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
  • Please register or sign in to post a comment
Tetsuya Maki
Assignee
Tetsuya Maki @Tetsuya.Maki
Assign to
None
Milestone
None
Assign milestone
Time tracking
1
Labels
機能改善
Assign labels
  • View labels
2
2 participants
Reference: yutaka.kobayashi/jsStudy!1