Project

General

Profile

Actions

Bug #29614

closed

Search in Template editor doesn't work at all

Added by Marcel Kühlhorn almost 4 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Normal
Category:
Templates
Target version:
-
Difficulty:
Triaged:
No
Fixed in Releases:
Found in Releases:

Description

When searching for a variable or parameter in a template with CTRL+F the search only covers the currently visible part of the template, it should search in the whole template

Actions #1

Updated by Bernhard Suttner almost 3 years ago

  • Found in Releases 2.3.0 added

The search functionality doesn't work in foreman 2.3, too. Did you already recognize this?

Actions #2

Updated by Bernhard Suttner almost 3 years ago

  • Assignee set to Avi Sharvit

Currently I see, that the search functionality in the templates editor is completely unusable. No search box is even shown.

Actions #3

Updated by Bernhard Suttner almost 3 years ago

  • Found in Releases 2.2.0, 2.4.0 added
Actions #4

Updated by Bernhard Suttner almost 3 years ago

  • Subject changed from Search in Template editor only works for currently visible part to Search in Template editor doesn't work at all
Actions #5

Updated by Bernhard Suttner almost 3 years ago

I tried:
- to add ace-builds so that "import 'ace-builds/src-min-noconflict/ext-searchbox';" works. Issue "ace.require is not a function" see https://github.com/securingsincity/react-ace/issues/722
- to replace react-ace with https://www.npmjs.com/package/react-ace-builds - search box is still not shown.

Actions #6

Updated by Maria Agaphontzev almost 3 years ago

  • Assignee changed from Avi Sharvit to Maria Agaphontzev

Currently to work around a user can open the ace editor in full screen and then the browser search works there. (using the maximize button)

Actions #7

Updated by Maria Agaphontzev over 2 years ago

  • Bugzilla link set to 1971072

added a related bz, not completely the same bug

Actions #8

Updated by The Foreman Bot over 2 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/8621 added
Actions #9

Updated by The Foreman Bot over 2 years ago

  • Fixed in Releases 3.0.0 added
Actions #10

Updated by Maria Agaphontzev over 2 years ago

  • Status changed from Ready For Testing to Closed
Actions

Also available in: Atom PDF