ham-cap

Results 25 comments of ham-cap

本番環境で動作確認できたためクローズしました。

@junohm410 お疲れ様です! お手隙の際で結構ですのでレビューをお願いできますでしょうか🙏 ご都合が悪い場合はご遠慮なくお知らせください👍

@junohm410 ありがとうございます🙏 急ぎではありませんのでゆっくりで大丈夫です。 よろしくお願いいたします🙇‍♂️

@junohm410 お疲れ様です! こちらの件、大変お待たせしてしまい申し訳ありません🙏 本業が繁忙期のため対応に時間がかかってしまいました💧 ご指摘いただいた点を修正いたしましたので再度ご確認いただけますでしょうか🙇‍♂️ よろしくお願いいたします。

@junohm410 お疲れ様です! > また、こちらはすごく小さなことなのですが、document.getElementById('bookmarks-in-dashboard')が重複しているので、DRYにできそうかな?と思いました🙏 こちら、ご指摘いただいておりました点について重複をなくして書いてみましたのでご確認ください🙏 また、コンフリクトについても解消済みです。 よろしくお願いいたします🙇‍♂️

@junohm410 コメントいただいた点、おっしゃるとおりですね💦 修正しておきました! こちらこそ丁寧にレビューしていただき大変勉強になりました! ありがとうございました💪 @komagata お疲れ様です! メンバーのレビューがApproveになりましたのでレビューをお願いできますでしょうか🙇‍♂️ よろしくお願いいたします🙏

@junohm410 お疲れ様です🙏 長らく仕事がバタバタしており全く時間が取れておりませんでした。申し訳ありません🙇‍♂️ ご指摘のとおり不要な変更が含まれていたため除外いたしました! すでにApproveをいただいておりますが、念のためご確認いただければ幸いです🙏

@junohm410 早速ご確認いただきありがとうございます🙏 また、長々とお付き合いいただきありがとうございました🙇‍♂️ @komagata 改めまして、こちらのレビューをお願いいたします🙇‍♂️

@machida @komagata @junohm410 こちら町田さんからお知らせいただいた内容に従い、ブックマークの件数が`0`になった際、残った`div`に`hidden`クラスを付与する形にいたしましたのでご確認いただけますでしょうか👀 不必要なマージンは発生せず、意図したとおりの挙動になっているかと思います🙌 町田さん教えていただきありがとうございました! こんなやり方があったとは。勉強になりました👀

@junohm410 ご確認いただきありがとうございます🙏