? J3 Issue No Code Attached Yet
avatar saraheagle
saraheagle
17 Oct 2017

Steps to reproduce the issue

On a clean new site >> system >> global config >> smart search >> smart search tab
Allow empty search = no

Expected result

on /index.php?option=com_finder&view=search

Button has class="disabled"
When you enter a search term, I expect the disabled class to be removed

Actual result

The disabled class is not removed until after the form has been submitted.
If you have Bootstrap this style makes the button unclickable.
.btn.disabled {cursor:not-allowed; pointer-events:none}

avatar saraheagle saraheagle - open - 17 Oct 2017
avatar joomla-cms-bot joomla-cms-bot - labeled - 17 Oct 2017
avatar brianteeman
brianteeman - comment - 17 Oct 2017

That entire code needs review.
The aim of the option "allow empty search" is for when you have filters selected
So for example if i wanted to find all the articles from a specific author then I could simply select the filter. But if allow empty search is set to no then I cant do that.
I guess that what needs to happen is for the class disabled to be removed as soon as you type in the search box when "allow empty search" is set to no

avatar brianteeman
brianteeman - comment - 18 Oct 2017

it probably needs something like ajaxchosen added to detect the inputs

avatar franz-wohlkoenig franz-wohlkoenig - change - 22 Oct 2017
Status New Information Required
avatar franz-wohlkoenig franz-wohlkoenig - change - 26 Dec 2017
Status Information Required Discussion
avatar brianteeman brianteeman - change - 25 Mar 2018
Labels Added: J3 Issue
avatar brianteeman brianteeman - labeled - 25 Mar 2018
avatar brianteeman brianteeman - change - 25 Mar 2018
The description was changed
avatar brianteeman brianteeman - edited - 25 Mar 2018
avatar brianteeman
brianteeman - comment - 3 Jul 2018

@Hackwar did you see this issue

avatar franz-wohlkoenig franz-wohlkoenig - change - 5 Apr 2019
Labels Added: ?
avatar franz-wohlkoenig franz-wohlkoenig - labeled - 5 Apr 2019
avatar franz-wohlkoenig franz-wohlkoenig - change - 9 Apr 2019
Category Accessibility com_finder com_finder

Add a Comment

Login with GitHub to post a comment