Hacktoberfest 2026：メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

ER: Remove unused `search` id query selectors

オープン 初心者向け
  • 何か問題がありますか？
#8,803 コメント 3 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 2 日以内に返信

まだ誰も着手していません。

評価

難易度
1/5
見積もり時間
1時間未満
初心者へのやさしさ
88/100
issue の種類
リファクタリング
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
javascript
領域
frontend

調査の方向性

assets/js/current-projects.js を開き、#search id を対象とする 3 つのセレクターを検索します。_includes/current-projects.html と比較して、その id が存在しないことを確認します。document.querySelector('#search') の出現箇所をすべて削除し、その後、JavaScript ファイルにそのようなセレクターが残っていないことを確認します。

索引モデルが issue の本文から書いたものです。

説明

Complexity: Small Draft ER Issue Making: Level 1 P-Feature: Projects page ready for merge team role: back end/devOps size: 0.25pt
Emergent Requirement - Problem

In assets/js/current-projects.js, there are three query selectors that select an id named search. However, in _includes/current-projects.html, there is no id for search.

We must remove any query selectors for search in assets/js/current-projects.js since it is an unused id.

Issue you discovered this emergent requirement in
  • #8466
  • Issue was pointed out in this comment for the PR of the original issue
Date discovered

9/19/2026

Did you have to do something temporarily
  • YES
  • NO
Who was involved

@ldaws003 @castillios

What happens if this is not addressed

Any deprecated IDs may continue to bloat the codebase if left unaddressed.

Resources
Recommended Action Items
  • Make a new issue
  • Discuss with team
  • Let a Team Lead know
Potential solutions [draft]

  • Create a small issue that lists the 3 instances of document.querySelector('#search') and their corresponding line numbers to be removed from assets/js/current-projects.js.

  • In your issue body under the Instructions section, include before and after code snippets as follows:

    Line 105 before:

    const inputEl = document.querySelector('#search-desktop') || document.querySelector('#search-mobile') || document.querySelector('#search');

    Line 105 after removing document.querySelector('#search'):

    const inputEl = document.querySelector('#search-desktop') || document.querySelector('#search-mobile');

    Lines 344-348 before:

    function searchEventHandler(e){
    e.preventDefault();
    const input = document.querySelector('#search-desktop') || document.querySelector('#search-mobile') || document.querySelector('#search');
    let searchTerm = input ? input.value : '';

    Lines 344-348 after removing document.querySelector('#search'):

    function searchEventHandler(e){
    e.preventDefault();
    const input = document.querySelector('#search-desktop') || document.querySelector('#search-mobile');
    let searchTerm = input ? input.value : '';

    Lines 376-380 before:

    function searchCloseEventHandler(e){
    e.preventDefault();
    const input = document.querySelector('#search-desktop') || document.querySelector('#search-mobile') || document.querySelector('#search');
    if (input) input.value="";
}

    Lines 376-380 after removing document.querySelector('#search'):

    function searchCloseEventHandler(e){
    e.preventDefault();
    const input = document.querySelector('#search-desktop') || document.querySelector('#search-mobile');
    if (input) input.value="";
}

  • After listing implementation details, include an action item to test and verify that the changes still work by performing a search in both mobile and desktop views on the Projects and Projects-Check pages

  • Ensure your issue labels match this parent ER (with the exception of Issue Making: Level 1 and ER)

  • Once the overall issue has been drafted, in the sidebar under Relationships, add this ER as a parent to your issue

  • Review and if ready, apply the ready for merge team label

主要言語
JavaScript
スター
364
フォーク
872
平均マージ
4日 14時間
マージ済み PR（30日）
13

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

hackforla/website のほかの issue

hackforla/website の issue をすべて見る

似ている issue

JavaScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。