PR-5.0-dev Pending

User tests: Successful: Unsuccessful:

avatar heelc29
heelc29
24 Jul 2023

Summary of Changes

Apply changes from #39537 for new debugging features (#36753)

Testing Instructions

check button still work and are rendered correctly for LTR and RTL

image
image

Actual result BEFORE applying this Pull Request

works

Expected result AFTER applying this Pull Request

works

Link to documentations

Please select:

  • No documentation changes for docs.joomla.org needed
  • No documentation changes for manual.joomla.org needed
avatar joomla-cms-bot joomla-cms-bot - change - 24 Jul 2023
Category Administration com_finder
avatar heelc29 heelc29 - open - 24 Jul 2023
avatar heelc29 heelc29 - change - 24 Jul 2023
Status New Pending
avatar HLeithner
HLeithner - comment - 25 Jul 2023

actually Toolbar::getInstance() is deprecated, so it doesn't makes it better. I'm planing to move the instance to the document but didn't had the time yet to implement it...

avatar heelc29
heelc29 - comment - 25 Jul 2023

so it doesn't makes it better

Ok, but then we have only to replace how we get the toolbar instance like in all the other components ...

avatar HLeithner
HLeithner - comment - 26 Jul 2023

I added the toolbar to the document in #41254 and used your PR as first example for the usage.

avatar heelc29
heelc29 - comment - 26 Jul 2023

Close in favor of #41254

avatar heelc29 heelc29 - change - 26 Jul 2023
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2023-07-26 16:35:55
Closed_By heelc29
Labels Added: PR-5.0-dev
avatar heelc29 heelc29 - close - 26 Jul 2023

Add a Comment

Login with GitHub to post a comment