?
avatar richard67
richard67
27 Apr 2021

Steps to reproduce the issue

The following describes the issue for com_content, but we have it elsewhere, too.

It is like that on J3 since long time and so some kind of expected behaviour, i.e. not been introduced by the new empty states in J4.

But I think it's confusing and annoying the user, so we should take a chance to change that with J4.

  1. Have some articles so the articles list view in backend shows items.
  2. Trash all articles.
  3. Filter for trashed items.
  4. Select all trashed articles and empty trash.
    Result: The empty state layout is shown. No filter tool bar is shown, but there is still the filter for trashed items active, as you will see in the next step-
  5. Use the "New" button to create a new article.
  6. See that the status of the new article is "Trashed".
  7. Change it to "Published".
  8. Use the "Save & Close" button.
    Result: The article list view is shown, The filter tool bar is visible, the filter for trashed items is set, and so the new article is not shown. That's confusing.

Alternatively you could skip step 6, then the article would be visible but have "Trashed" status, that's the same annoying.

Expected result

When the last item is deleted so that the empty state layout is show, all active filters are cleared.

Actual result

See above steps to reproduce.

System information (as much as possible)

Current 4.0-dev.

Additional comments

This issue has not been introduced by the empty states like I first thought because I didn't remember that this issue was confusing me long time ago in J3.

avatar richard67 richard67 - open - 27 Apr 2021
avatar joomla-cms-bot joomla-cms-bot - change - 27 Apr 2021
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 27 Apr 2021
avatar richard67
richard67 - comment - 27 Apr 2021

P.S. I am not 100% sure, but I think I remember it was working right before the empty state formerly called blank state layouts were introduced.

avatar PhilETaylor
PhilETaylor - comment - 27 Apr 2021

This is exactly how it works currently - you are right. The Empty State code makes no changes to the filter state at all. So if your last filter selection was Trashed, then you would remain filtered with Trashed when you clicked the add new button and then Save & Close would take you back to a Trashed filtered view.

This is how its been for years.

This is no different to how it was before.

Im not saying this is how it should be, it can of course be improved, but this has not been broken or changed by Empty State Design.

avatar richard67
richard67 - comment - 27 Apr 2021

I'm not sure if it was like that before.

avatar PhilETaylor
PhilETaylor - comment - 27 Apr 2021

I am off to bed, I will let you test and see that it has been this way forever :-)

There simply is no code in any of the new empty state changes to make any changes to the filter state, we just leave it alone.

avatar brianteeman
brianteeman - comment - 27 Apr 2021

nothing to do with empty state

avatar richard67
richard67 - comment - 27 Apr 2021

This is no different to how it was before.

nothing to do with empty state

Confirmed. Was like that also on Beta 7, I've just tested.

@brianteeman Do you know if we have already an issue for that?

avatar richard67 richard67 - close - 27 Apr 2021
avatar richard67
richard67 - comment - 27 Apr 2021

Closing for now. Sorry for the confusion.

avatar richard67 richard67 - change - 27 Apr 2021
Status New Closed
Closed_Date 0000-00-00 00:00:00 2021-04-27 21:19:33
Closed_By richard67
avatar brianteeman
brianteeman - comment - 27 Apr 2021

@richard67 its this way by design and has always been this way. Watch this

modal

avatar richard67
richard67 - comment - 27 Apr 2021

I know meanwhile.

avatar richard67
richard67 - comment - 1 May 2021

Re-opening the issue since I still think it is weird behaviour confusing the user, even if it is like that on J3 since ages.

I remember now it had confused me in past, and then I had forgotten about it, it seems.

Will change description and maybe title, too, so it's clear it was not introduced by the empty states.

avatar richard67 richard67 - change - 1 May 2021
Status Closed New
Closed_Date 2021-04-27 21:19:33
Closed_By richard67
avatar richard67 richard67 - reopen - 1 May 2021
avatar richard67 richard67 - change - 1 May 2021
The description was changed
avatar richard67 richard67 - edited - 1 May 2021
avatar richard67
richard67 - comment - 1 May 2021

I think when empty state is detected, filters should be reset/unset first before hiding the filter and any other unneeded buttons.

avatar PhilETaylor
PhilETaylor - comment - 1 May 2021

I think when empty state is detected, filters should be reset/unset first before hiding the filter and any other unneeded buttons.

Go for it.

avatar richard67
richard67 - comment - 1 May 2021

I think when empty state is detected, filters should be reset/unset first before hiding the filter and any other unneeded buttons.

Go for it.

@PhilETaylor I don't really know how to do that. Would have to investigate it first. Maybe you can do it faster?

avatar PhilETaylor
PhilETaylor - comment - 1 May 2021

I think when empty state is detected, filters should be reset/unset first before hiding the filter and any other unneeded buttons.

Added in #33471

avatar richard67 richard67 - change - 1 May 2021
Status New Closed
Closed_Date 0000-00-00 00:00:00 2021-05-01 18:40:38
Closed_By richard67
avatar richard67 richard67 - close - 1 May 2021
avatar richard67
richard67 - comment - 1 May 2021

Closing as having a pull request.

Add a Comment

Login with GitHub to post a comment