[#9208] - [com_categories] add searchtools to modal layout and solves bugs
- Fixed in Code Base
- 14 Apr 2016
- Medium
- Build: staging
- # 9208
- Diff
- andrepereiradasilva:com_categories_searchtools
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
User tests: Successful: Unsuccessful:
Description
This PR adds searchtools to com_categories modal layout categories view.
Also, in normal layout:
- correct bug: the modal on mobile xs show a column without title.
- correct bug: ordering the view for Association doesn't add the arrow icon (removed ordering in this column).
- improves responsive visual (mobile/tablet/desktop)
- solves other minor graphical issues.
- minor code style improvements
Before PR (modal)
After PR (modal)
How to test
Normal view:
1. Install latest staging and apply this patch
2. Go to Content -> Categories
3. Check if search, filters, ordering and pagination are working properly
4. Test on mobile and tablet (ex: reduce browser width)
Modal view:
1. Enable language associations.
2. Create some categories in several languages.
3. Edit one of those categories and go to "Associations" tab and click on "Select" on any language.
4. Check if search, filters, ordering and pagination are working properly in modal.
5. Test on mobile and tablet (ex: reduce browser width)
Should be also tested in other extensions that use categories (ex: User Notes, Contacts, Banners and News Feeds)
| Status | New | ⇒ | Pending |
| Labels |
Added:
?
|
||
| Title |
|
||||||
I have tested this item 
successfully on 4ce7fae
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9208.
| Status | Pending | ⇒ | Ready to Commit |
| Category | ⇒ | Administration UI/UX |
RTC - thanks once again
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9208.
| Labels |
Added:
?
|
||
| Milestone |
Added: |
||
| Milestone |
Added: |
||
@andrepereiradasilva could you have a look at the merge conflicts, thanks.
This PR has received new commits.
CC: @brianteeman, @infograf768
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9208.
| Labels |
Added:
?
|
||
This PR has received new commits.
CC: @brianteeman, @infograf768
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9208.
conflicts fixed.
@brianteeman can you remove the Unit tests label again? sorry
| Labels |
Removed:
?
|
||
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-04-14 21:17:48 |
| Closed_By | ⇒ | rdeutz |
| Labels |
Removed:
?
|
||
| Milestone |
Removed: |
||
| Milestone |
Added: |
||
| Milestone |
Added: |
||
| Milestone |
Removed: |
||
I have tested this item
successfully on 4ce7fae
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9208.