? Success

User tests: Successful: Unsuccessful:

avatar andrepereiradasilva
andrepereiradasilva
4 Jan 2016

Description

In move to normalize the search/filter/order/limit bar (aka searchtools) in all administrator list views, and as other PR's have made for other list component/views, this PR adds JLayouts searchtools to com_menu menus view and replacing the old HTML code.
With that it also adds an option for ordering to com_menu menus view.

A PHP notice in hathor was corrected too.

Isis template
Before patch

image

After patch

image

Hathor template
Before patch

image

After patch

image

How to test

  1. Go to com_menu menus view (the one with the menutypes, not menu items view).
  2. Create some menus (at least one of them without linked modules)
  3. Apply the patch
  4. Test the Search Tools (Search field, ordering and limit).
  5. View hathor template to and check the php notice is not there now.

Observations

First time using a part of JLayouts so suggestions and code revisions are welcome.

Tags

administrator, menus, search tools, jlayouts, list views, normalization

avatar andrepereiradasilva andrepereiradasilva - open - 4 Jan 2016
avatar andrepereiradasilva andrepereiradasilva - change - 4 Jan 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 4 Jan 2016
Labels Added: ?
avatar infograf768
infograf768 - comment - 5 Jan 2016

This does not look like working. What do you expect exactly. Please attach a screenshot.
Here, except for the Clear Button in the menus list, I lose the correct search in the menu items list where the Menutype field disapears.

avatar andrepereiradasilva
andrepereiradasilva - comment - 5 Jan 2016

Ups. i will check when i have time, sorry for that. I will also put some screenshots them.
The goal here is to normalize the use of JLayouts searchtools in com_menus (menus view doesn't use it, the other views already use it).

avatar andrepereiradasilva
andrepereiradasilva - comment - 5 Jan 2016

@infograf768 code and test instructions updated.
Screenshots added.
Can you check now?

avatar andrepereiradasilva andrepereiradasilva - change - 5 Jan 2016
The description was changed
avatar andrepereiradasilva andrepereiradasilva - change - 5 Jan 2016
The description was changed
avatar andrepereiradasilva andrepereiradasilva - change - 6 Jan 2016
Title
Implement JLayouts Search Tools in com_menu menus view
Search Tools (from JLayouts) implemented in com_menu menus view also
avatar infograf768 infograf768 - test_item - 7 Jan 2016 - Tested successfully
avatar infograf768
infograf768 - comment - 7 Jan 2016

I have tested this item :white_check_mark: successfully on 98f49de

Works now.


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

avatar andrepereiradasilva andrepereiradasilva - change - 7 Jan 2016
Title
Search Tools (from JLayouts) implemented in com_menu menus view also
Search Tools (from JLayouts) implemented in com_menu (menus view)
avatar NLRoosj NLRoosj - test_item - 10 Jan 2016 - Tested successfully
avatar NLRoosj
NLRoosj - comment - 10 Jan 2016

I have tested this item :white_check_mark: successfully on 98f49de


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

avatar brianteeman brianteeman - change - 12 Jan 2016
Status Pending Ready to Commit
avatar brianteeman
brianteeman - comment - 12 Jan 2016

RTC - thanks


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

avatar joomla-cms-bot joomla-cms-bot - change - 12 Jan 2016
Labels Added: ?
avatar andrepereiradasilva
andrepereiradasilva - comment - 12 Jan 2016

@infograf768 @NLRoosj thanks for testing.

avatar rdeutz rdeutz - change - 13 Jan 2016
Milestone Added:
avatar wilsonge wilsonge - change - 17 Jan 2016
Milestone Added:
avatar wilsonge wilsonge - change - 17 Jan 2016
Milestone Removed:
avatar wilsonge wilsonge - change - 17 Jan 2016
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2016-01-17 00:49:17
Closed_By wilsonge
avatar wilsonge wilsonge - close - 17 Jan 2016
avatar joomla-cms-bot joomla-cms-bot - close - 17 Jan 2016
avatar wilsonge wilsonge - close - 17 Jan 2016
avatar joomla-cms-bot joomla-cms-bot - change - 17 Jan 2016
Labels Removed: ?
avatar andrepereiradasilva andrepereiradasilva - head_ref_deleted - 17 Jan 2016

Add a Comment

Login with GitHub to post a comment