Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-33559

Job configuration "search" box doesn't properly clear highlights

    • Icon: Bug Bug
    • Resolution: Won't Do
    • Icon: Minor Minor
    • core
    • 2.0-alpha3

      (setting aside my view that this search box is superfluous)

      When you search for something on the page, e.g. "CVS", the view will scroll to the element and highlight the search query on the page.

      When the search field is cleared, the highlight remains, see attached screenshot.

          [JENKINS-33559] Job configuration "search" box doesn't properly clear highlights

          R. Tyler Croy created issue -

          Daniel Beck added a comment -

          The box will get removed soon, it's kind of useless for scroll-spying anyway.

          Daniel Beck added a comment - The box will get removed soon, it's kind of useless for scroll-spying anyway.

          Daniel Beck added a comment -

          Asking tfennelly for confirmation on the above.

          Daniel Beck added a comment - Asking tfennelly for confirmation on the above.
          Daniel Beck made changes -
          Labels Original: 2.0 New: 2.0 2.0-planned
          Daniel Beck made changes -
          Assignee New: Tom FENNELLY [ tfennelly ]

          Tom FENNELLY added a comment -

          Yes, that is a bug and is fixed in https://github.com/jenkinsci/jenkins/pull/2121.

          I also agree that the find feature is not so useful with scrollspy since the user can use cmd+f anyway. Maybe we should juts remove it for scrollspy.

          Tom FENNELLY added a comment - Yes, that is a bug and is fixed in https://github.com/jenkinsci/jenkins/pull/2121 . I also agree that the find feature is not so useful with scrollspy since the user can use cmd+f anyway. Maybe we should juts remove it for scrollspy.

          Is this bug related to JENKINS-33620 (or can they be fixed at the same time?)

          Antonio Muñiz added a comment - Is this bug related to JENKINS-33620 (or can they be fixed at the same time?)

          Well, if the search box is removed then both issue would become out-dated.

          Antonio Muñiz added a comment - Well, if the search box is removed then both issue would become out-dated.

          Sounds like the consensus is that this issue should be closed because of the change made since then.

          Kohsuke Kawaguchi added a comment - Sounds like the consensus is that this issue should be closed because of the change made since then.

          Based on the comments that this feature is soon to be removed.

          Kohsuke Kawaguchi added a comment - Based on the comments that this feature is soon to be removed.
          Kohsuke Kawaguchi made changes -
          Resolution New: Won't Do [ 10001 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]

            tfennelly Tom FENNELLY
            rtyler R. Tyler Croy
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Created:
              Updated:
              Resolved: