質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.51%

縦三点リーダ押下時、複数リンクを表示したい

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 93

score 7

前提・実現したいこと

Ruby(RubyonRails)でTwitterクローンアプリを作成中です。
ツイート一覧をグリッドリストで表示しているのですが、縦三点リーダを押下したときに編集と削除というリンクが出てきて、それぞれのアクションを行えるようにしたいと考えております。
ボタン押下時のアクションなので、visibilityがhiddenの場合とvisibleの場合とでリンクを見せたり消したりしたいと考えています。

参考にしていた記事はこちらのボタン2のコードです。

発生している問題・エラーメッセージ

検証ツールのconsoleを開き、縦三点リーダをクリックしたところ、下記のエラーメッセージが出ました。

Uncaught ReferenceError: verticalThreeReader is not defined at HTMLButtonElement.onclick

該当のソースコード

<div class="more-wrapper">
    <button class="project-btn-more" onclick="verticalThreeReader()"> 
       <svg xmlns="http://www.w3.org/2000/svg " width="24 " height="24 " viewBox="0 0 24 24 " fill="none " stroke="currentColor " stroke-width="2 " stroke-linecap="round " stroke-linejoin="round " class="feather feather-more-vertical ">
            <circle cx="12 " cy="12 " r="1 " />
         <circle cx="12 " cy="5 " r="1 " />
         <circle cx="12 " cy="19 " r="1 " />
        </svg>
<!--縦三点リーダ-->
         <div class="vertical-three-reader">
            <ul>
               <li><%= link_to "edit", edit_post_path(post.id) %></li>
               <li><%= link_to "delete", post_path(post.id), method: :delete %></li>
           </ul>
         </div>
    </button>
</div>
document.addEventListener('DOMContentLoaded', function() {

//縦三点リーダの上には別の記述がありますが、本題と関係ないので割愛しております。

document.querySelector(".vertical-three-reader").style.visibility ="hidden";

    function verticalThreeReader(){
        const verticalThreeReader = document.querySelector(".vertical-three-reader");

        if(verticalThreeReader.style.visibility=="visible"){
            // hiddenで非表示
            verticalThreeReader.style.visibility ="hidden";
        }else{
            // visibleで表示
            verticalThreeReader.style.visibility ="visible";
        }
}


});

試したこと

エラー文で検索をしたところ、関数の定義が間違っているのだとはわかりました。
button class="more-wrapper"の部分が関与しているのかと思い、消してみましたがレイアウトが崩れただけで特にエラーの解決には至りませんでした。
<button class="project-btn-more" onclick="verticalThreeReader()"> の部分でエラーが起きていることはわかります。

ご回答いただければ幸いです。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

0

属性でイベントハンドラを設定すると、グローバルスコープにあるものしか参照できないです。
その他、使わない方がいい理由がたくさんありますので、避けた方がいいです。

HTMLのonclick属性(クリックイベント)を使用してはダメな4つの理由 | iwb.jp

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2021/10/12 17:01

    ありがとうございます。勉強になりました。
    addEventListnerを使用した方がいいとのことですが、全て書き換える必要があるということですよね。

    キャンセル

  • 2021/10/12 17:40

    そうですね、といってもそれほど大きな変更ではないです。

    document.querySelector(".project-btn-more").addEventListener('click', verticalThreeReader);

    と書けばいいだけなので。

    キャンセル

  • 2021/10/12 21:26

    タグを取ってくるだけではなく、クリックしたときの動作も付け加えればいいということですね!
    ありがとうございます!

    キャンセル

checkベストアンサー

-1

イメージ説明

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2021/10/12 17:02

    ありがとうございます。
    表示することができました。

    保守的なことを考えれば、下に書いていただいた回答者様の言った通りに記述し直す必要があるのでしょうね。

    キャンセル

  • 2021/10/12 17:09 編集

    .

    キャンセル

  • 2021/10/12 17:11

    なるほどです。
    javascriptの記述がほぼ初めてなので、いい機会だと思って記述し直してみます!

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.51%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る