? ? ? NPM Resource Changed Pending

User tests: Successful: Unsuccessful:

avatar laoneo
laoneo
31 Aug 2021

Summary of Changes

When searching in the media manager and the page got reloaded, then the search should not be cleared and stored in the localStorage sessionStorage of the browser as we do with other parts in the media manager.

image

Testing Instructions

Insert "joomla" into the search field in the media manager. And reload the page.

Actual result BEFORE applying this Pull Request

The whole list is shown without the search term.

Expected result AFTER applying this Pull Request

The page is loaded with the results filtered with the name joomla.

avatar laoneo laoneo - open - 31 Aug 2021
avatar laoneo laoneo - change - 31 Aug 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 31 Aug 2021
Category Administration com_media NPM Change JavaScript Unit Tests
avatar PhilETaylor
PhilETaylor - comment - 31 Aug 2021

Excuse me for asking questions without testing myself.

  1. Is the search persistent for a certain folder? I.e if I change folder will the search query remain?

  2. Is it a "REAL search" (where it calls back to PHP and trawls the folders beneath this one for files that exist with that name) or just a filter on the current page?

avatar richard67
richard67 - comment - 31 Aug 2021
avatar laoneo
laoneo - comment - 31 Aug 2021

@PhilETaylor

  1. No, but as you get forwarded to the same folder when the page get reloaded you have the same state.
  2. No just filtering.
4174c63 31 Aug 2021 avatar laoneo cs
avatar laoneo laoneo - change - 31 Aug 2021
Labels Added: ? ? NPM Resource Changed
avatar dgrammatiko
dgrammatiko - comment - 31 Aug 2021

@laoneo shouldn't it be sessionStorage instead of localStorage? LocalStorage will persist even if a user logs out and re login

avatar laoneo
laoneo - comment - 31 Aug 2021

Sure sessionStorage, using the same as with the other states.

avatar laoneo laoneo - change - 31 Aug 2021
The description was changed
avatar laoneo laoneo - edited - 31 Aug 2021
avatar dgrammatiko dgrammatiko - test_item - 2 Sep 2021 - Tested successfully
avatar dgrammatiko
dgrammatiko - comment - 2 Sep 2021

I have tested this item successfully on 8146ea7


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

avatar alikon alikon - test_item - 3 Sep 2021 - Tested successfully
avatar alikon
alikon - comment - 3 Sep 2021

I have tested this item successfully on 8146ea7


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

avatar alikon alikon - change - 3 Sep 2021
Status Pending Ready to Commit
avatar alikon
alikon - comment - 3 Sep 2021

RTC


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

avatar wilsonge
wilsonge - comment - 9 Sep 2021

I think we need to be a bit more careful than this. In all likihood navigating to a subfolder should clear the search rather than persisting it. although obviously I agree generally speaking that persisting it would be a good thing.

avatar laoneo
laoneo - comment - 10 Sep 2021

@wilsonge this is not the scope of this pr as you have the same behavior even without the changes here. I can do another one but please lets not mixing things up.

avatar wilsonge
wilsonge - comment - 11 Sep 2021

Sorry so the current behaviour is that the search applies and isn't stored in the input box then?

avatar laoneo
laoneo - comment - 12 Sep 2021

It is stored, but you have no page reload on folder change. Only the content in the list changes, so the search box keeps the value and search is applied on folder change.

avatar laoneo
laoneo - comment - 4 Oct 2021

@wilsonge friendly reminder

avatar laoneo
laoneo - comment - 18 Oct 2021

@wilsonge friendly reminder

avatar wilsonge wilsonge - change - 7 Nov 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-11-07 19:50:22
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 7 Nov 2021
avatar wilsonge wilsonge - merge - 7 Nov 2021
avatar wilsonge
wilsonge - comment - 7 Nov 2021

Thanks!

Add a Comment

Login with GitHub to post a comment