? ? Pending

User tests: Successful: Unsuccessful:

avatar rjharishabh
rjharishabh
24 Jun 2021

Summary of Changes

Add !$this->isEmptyState when adding toolbar->delete()

Testing Instructions

  1. Add a contact (if you don't have any)
  2. Trash the contact
  3. Delete the contact
cont.mp4

Actual result BEFORE applying this Pull Request

Empty Trash button in Emptystate
trash-before

Expected result AFTER applying this Pull Request

No Empty Trash button in Emptystate
trash-after

Documentation Changes Required

No

avatar rjharishabh rjharishabh - open - 24 Jun 2021
avatar rjharishabh rjharishabh - change - 24 Jun 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 24 Jun 2021
Category Administration com_contact
avatar brianteeman
brianteeman - comment - 24 Jun 2021

You only have the button because the last action you performed (deleting) used a filter and the filter doesn't get reset on use.

This behaviour most likely exists everywhere so if you're going to address it here then you should do so everywhere.

avatar rjharishabh
rjharishabh - comment - 24 Jun 2021

You only have the button because the last action you performed (deleting) used a filter and the filter doesn't get reset on use.

This behaviour most likely exists everywhere so if you're going to address it here then you should do so everywhere.

yes, I've tested in banners and categories, it's not there but there are lots which have to correct

avatar rjharishabh rjharishabh - change - 26 Jun 2021
Title
[4.0] Remove Empty Trash in empty state in com_contact
[4.0] Remove Empty Trash button in empty state in com_contact
avatar rjharishabh rjharishabh - edited - 26 Jun 2021
avatar Kostelano Kostelano - test_item - 29 Jun 2021 - Tested successfully
avatar Kostelano
Kostelano - comment - 29 Jun 2021

I have tested this item successfully on c7a6fe0


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

avatar Quy Quy - test_item - 29 Jun 2021 - Tested successfully
avatar Quy
Quy - comment - 29 Jun 2021

I have tested this item successfully on c7a6fe0


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

avatar Quy Quy - change - 29 Jun 2021
Status Pending Ready to Commit
Labels Added: ?
avatar Quy
Quy - comment - 29 Jun 2021

RTC


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

avatar bembelimen bembelimen - close - 2 Jul 2021
avatar bembelimen bembelimen - merge - 2 Jul 2021
avatar bembelimen bembelimen - change - 2 Jul 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-07-02 13:12:14
Closed_By bembelimen
Labels Added: ?
avatar bembelimen
bembelimen - comment - 2 Jul 2021

Thanks

Add a Comment

Login with GitHub to post a comment