? Success

User tests: Successful: Unsuccessful:

avatar pe7er
pe7er
17 Aug 2015

This PR moves Filters from left sidebar to Search Tools.

Testing instructions

Before this PR

Users > User Notes
1. Left sidebar has 2 filter options (Select: Status, Category) that cannot be reset easily to original state with one button.
2. On the top right there are two duplicate ordering fields: "Sort Table By" & "Select the ordering" (for the direction of the ordering).

usernotes-before

After this PR

Users > User Notes
1. the Filters from sidebar should be moved to the middle under "Search Tools" option. All filters can be reset with the "Clear" button.
2. the "Sort Table By" & "Select the ordering" have been removed. The ordering field + ordering direction can be set by clicking on a column header (fieldname).

usernotes-after

Votes

# of Users Experiencing Issue
1/1
Average Importance Score
1.00

avatar pe7er pe7er - open - 17 Aug 2015
avatar pe7er pe7er - change - 17 Aug 2015
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 17 Aug 2015
Labels Added: ?
avatar zero-24 zero-24 - change - 17 Aug 2015
Easy No Yes
avatar zero-24 zero-24 - change - 17 Aug 2015
Category Administration UI/UX
avatar pe7er pe7er - reference | f6665aa - 17 Aug 15
avatar watchfulli-dev
watchfulli-dev - comment - 20 Aug 2015

@test after applying the patch the top right ordering option is removed, and the filters from the left side are moved inside "Search tools" on top, but now seams the filters are not working, for example if I try to filter them by "Trashed" items, the published user note is still displayed after applying the filter


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

avatar watchfulli-dev watchfulli-dev - test_item - 20 Aug 2015 - Tested unsuccessfully
avatar hitchblade hitchblade - test_item - 20 Aug 2015 - Tested successfully
avatar kesh52
kesh52 - comment - 20 Aug 2015

I have an issue like by watchfulli-dev.
If I filter by status and need only "Unpublished" items, all items are displayed


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

avatar kesh52 kesh52 - test_item - 20 Aug 2015 - Tested unsuccessfully
avatar pe7er
pe7er - comment - 20 Aug 2015

Thank you for testing @watchfulli-dev & @kesh52 !
I was able to reproduce the issue you both reported & corrected it in my patch.
Now the Status filter should work correctly...

avatar bullet64
bullet64 - comment - 29 Aug 2015

Hello, i have installed the patch but i have issues.

I have created 4 user-notes.

  • published
  • unpublished
  • trashed
  • archived

Filter all - works
Filter published - shows published & unpublished
Filter trashed - ok
Filter archived - ok

avatar YouriiM26
YouriiM26 - comment - 7 Oct 2015

I have the same as @bullet64. Published and unpublished shows the same thing.


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

avatar joomla-cms-bot
joomla-cms-bot - comment - 11 Oct 2015

This PR has received new commits.

CC: @hitchblade, @kesh52, @watchfulli-dev


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

avatar pe7er
pe7er - comment - 11 Oct 2015

Thank you @bullet64 @YouriiM26 for testing & reporting the problem.
I've fixed it with a new patch.

avatar YouriiM26 YouriiM26 - test_item - 12 Oct 2015 - Tested successfully
avatar YouriiM26
YouriiM26 - comment - 12 Oct 2015

I have tested this item :white_check_mark: successfully on a9bce89

The new patch worked for me. The published and unpublished aren't the same anymore. Thanks!


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

avatar RemcoJanssen RemcoJanssen - test_item - 12 Oct 2015 - Tested successfully
avatar RemcoJanssen
RemcoJanssen - comment - 12 Oct 2015

I have tested this item :white_check_mark: successfully on a9bce89


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

avatar zero-24 zero-24 - change - 12 Oct 2015
Status Pending Ready to Commit
avatar zero-24
zero-24 - comment - 12 Oct 2015

RTC for 3.5


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

avatar joomla-cms-bot joomla-cms-bot - change - 12 Oct 2015
Labels Added: ?
avatar zero-24 zero-24 - change - 12 Oct 2015
Milestone Added:
avatar Kubik-Rubik Kubik-Rubik - change - 4 Nov 2015
Labels Removed: ?
avatar joomla-cms-bot joomla-cms-bot - change - 4 Nov 2015
Labels Added: ?
avatar Kubik-Rubik Kubik-Rubik - change - 4 Nov 2015
Status Ready to Commit Needs Review
Labels
avatar Kubik-Rubik
Kubik-Rubik - comment - 4 Nov 2015

@pe7er Thank you for the contribution. The move from the sidebar is good but we decided in the PLT to not remove the ordering fields. Could you please remove this part from your PR and we can add this PR to the 3.5 version.

This needs a review to be merged!

avatar joomla-cms-bot joomla-cms-bot - change - 4 Nov 2015
Labels Removed: ?
avatar pe7er
pe7er - comment - 4 Nov 2015

@Kubik-Rubik Thanks! I know about not removing the ordering fields.
I corrected that in some other PRs I did list night.
I'll correct it in this patch too...

avatar joomla-cms-bot
joomla-cms-bot - comment - 4 Nov 2015

This PR has received new commits.

CC: @hitchblade, @kesh52, @RemcoJanssen, @watchfulli-dev, @YouriiM26


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

avatar pe7er
pe7er - comment - 4 Nov 2015

I've corrected the PR to add the ordering fields back (via the filter_notes.xml that is used by Search Tools)

com_users-notes-afterv2

avatar Kubik-Rubik Kubik-Rubik - test_item - 4 Nov 2015 - Tested successfully
avatar Kubik-Rubik
Kubik-Rubik - comment - 4 Nov 2015

I have tested this item :white_check_mark: successfully on 78f1f4f

Works as described. Thank you @pe7er!


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

avatar dgt41 dgt41 - test_item - 4 Nov 2015 - Tested successfully
avatar dgt41 dgt41 - test_item - 4 Nov 2015 - Tested successfully
avatar dgt41
dgt41 - comment - 4 Nov 2015

I have tested this item :white_check_mark: successfully on 78f1f4f

Small note here: there is no ordering option so no drag and drop to reorder. maybe in another PR


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

avatar zero-24
zero-24 - comment - 4 Nov 2015

@Kubik-Rubik this gets tested twice please do the final review and move to RTC if ok ;)


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

avatar Kubik-Rubik Kubik-Rubik - change - 4 Nov 2015
Status Needs Review Ready to Commit
avatar Kubik-Rubik Kubik-Rubik - change - 4 Nov 2015
Labels Added: ?
avatar joomla-cms-bot
joomla-cms-bot - comment - 4 Nov 2015

This PR has received new commits.

CC: @dgt41, @hitchblade, @kesh52, @Kubik-Rubik, @RemcoJanssen, @watchfulli-dev, @YouriiM26


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

avatar pe7er pe7er - change - 4 Nov 2015
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2015-11-04 22:52:51
Closed_By pe7er
avatar pe7er pe7er - close - 4 Nov 2015
avatar joomla-cms-bot joomla-cms-bot - close - 4 Nov 2015
avatar pe7er pe7er - close - 4 Nov 2015
avatar joomla-cms-bot joomla-cms-bot - change - 4 Nov 2015
Labels Removed: ?
avatar pe7er pe7er - head_ref_deleted - 5 Nov 2015
avatar zero-24 zero-24 - change - 5 Nov 2015
Milestone Removed:

Add a Comment

Login with GitHub to post a comment