? ? ? Pending

User tests: Successful: Unsuccessful:

avatar sanderpotjer
sanderpotjer
15 Feb 2017

Summary of Changes

Adding input-medium class for input field to get the same output as for mod_finder. We have had this issue for the joomla.org templates for example, where the output is different if using mod_search instead of mod_finder.

Testing Instructions

Before patch the input-medium class is not in the output for mod_search, after applying patch the output should be changed and input-medium is one of the classes.

avatar sanderpotjer sanderpotjer - open - 15 Feb 2017
avatar sanderpotjer sanderpotjer - change - 15 Feb 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 15 Feb 2017
Category Modules Front End
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 16 Feb 2017

I have tested this item successfully on 10a5eb0

After Patch com_search (Modules: Search) have a input-medium similar to com_finder (Modules: Smart Search).


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14099.

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 16 Feb 2017 - Tested successfully
avatar joomdonation
joomdonation - comment - 19 Feb 2017

Do we have to worry about adding input-medium class will change the display of search module on existing websites?

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 22 Feb 2017

@C-Lodder do we have worry about?

avatar C-Lodder
C-Lodder - comment - 22 Feb 2017

This is fine. The input-medium width will only kick in, if the Box Width parameter for mod_search has no value.

avatar C-Lodder C-Lodder - test_item - 22 Feb 2017 - Tested successfully
avatar C-Lodder
C-Lodder - comment - 22 Feb 2017

I have tested this item successfully on 10a5eb0


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14099.

avatar dgt41 dgt41 - change - 22 Feb 2017
Status Pending Ready to Commit
avatar dgt41
dgt41 - comment - 22 Feb 2017

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14099.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 22 Feb 2017

Thanks @C-Lodder @dgt41

avatar rdeutz
rdeutz - comment - 22 Feb 2017

unit test must be fixed before we can merge this @sanderpotjer ca you do that?

avatar wilsonge wilsonge - change - 23 Feb 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 23 Feb 2017
Category Modules Front End Modules Front End Unit Tests
avatar wilsonge
wilsonge - comment - 23 Feb 2017

I have fixed el unit test

avatar wilsonge wilsonge - change - 23 Feb 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-02-23 00:12:47
Closed_By wilsonge
Labels Added: ? ?
avatar wilsonge wilsonge - close - 23 Feb 2017
avatar wilsonge wilsonge - merge - 23 Feb 2017

Add a Comment

Login with GitHub to post a comment