Manageiq-ui-classic: Missing Search/Advanced Search in Ansible Playbooks/Repositories/Credentials page

[hstastna]

Go to _Automation > Ansible > Playbooks/Repositories/Credentials_ page and you can see that in accordion, there's _No filters defined._ (or maybe some filters already?) but you cannot define any filter because of missing _Advanced Search_ button on the right of the page.

Also it is not possible to search through items in the list. _Search_ not present, too:

[hstastna]

@miq-bot add_label bug

[himdel]

I'm not sure there was ever supposed to be any search or advanced search there.

I think what you're seeing is a screen with no search, and a default listnav which just says no filters.

(IMO it doesn't sound like a bug, but once we have support for listnav-less layouts, we may want to use them here.)

[hstastna]

@himdel So from what I understand, the bug here for now is the misleading text in accordion.

[h-kataria]

@hstastna it looks like we should not render listnav partial for these screens.

[h-kataria]

@hstastna having nameonly search makes sense on all list views, but listnav should only be visible if we list can have advanced search, without advanced search there is no way a user can add filters to the screen. So, we should make changes to code to display nameonly search on the screen and hide the listnav when advanced search is not allowed.

[himdel]

Linking with https://github.com/ManageIQ/manageiq-ui-classic/issues/6356

Maybe instead of adding search to the no-listnav layout, we can move simple search to toolbars.
(Although, toolbar search has to be react, so maybe it's faster to just add the existing one to the layout, if doable.)

[hstastna]

I like the idea about moving Search to toolbar but according to the comment it's faster and easily doable to just add the existing Search to the layout, at least for now.

[himdel]

Fixed in https://github.com/ManageIQ/manageiq-ui-classic/pull/6810