I have tested this item ✅ successfully on 74b7091

@brianteeman my AFTER condition matches perfectly, my BEFORE condition with 1 small difference:

The error I get says: Unhandled Promise Rejection: TypeError: Load failed (in Safari 18.5) - could the browser used give a slightly different error message?

Based on your previous comments, you have said if it doesn't match 100% you can't consider a good test so that's why I am checking.

so the icon is NOT displayed after applying the pr except when the option is enabled?

and when the option is enabled and you click on the icon you get the error?

Is there anything in the folders administrator\cache\plg_system_debug_administrator or administrator\cache\plg_system_debug_site

so the icon is NOT displayed after applying the pr except when the option is enabled?

and when the option is enabled and you click on the icon you get the error?

Is there anything in the folders administrator\cache\plg_system_debug_administrator or administrator\cache\plg_system_debug_site

Hi Brian,

Correct, the icon is NOT displayed after applying the PR except when the option in enabled (which is what I understood from your description is the correct result... hopefully I got that right.)

and yes, when the option is enabled, and click on the icon I get that error.

yes there are files in the administrator\cache\plg_system_debug_administrator folder about 13 files with random numbers/letters all starting with X ---> such as Xc05be07d6242b206322682c6a6d013f6.php each measuring 38kb - 73kb

Hope that helps.

sorry just re-read. It is the before condition that gave an error but the error is gone afterwards. Safari console probably gives a slightly different message. The important part of this PR is that there was an error before. There is now no icon so you can never get to the page when the files dont exist so the error condition is never met.

the pr could probably be improved to change the js to give a cleaner error but removing the option has the same effect as you can never get to the page producing the error

sorry just re-read. It is the before condition that gave an error but the error is gone afterwards. Safari console probably gives a slightly different message. The important part of this PR is that there was an error before. There is now no icon so you can never get to the page when the files dont exist so the error condition is never met.

the pr could probably be improved to change the js to give a cleaner error but removing the option has the same effect as you can never get to the page producing the error

Yes, it is behaving as you planned it - the only 'difference' was the error message but if you say that Safari probably gives a slightly different message then that appeases my concern. What browser did you test against so if I have a bit of time I can redo the test with that browser and confirm the same error message which would confirm that Safari uses a slightly different error message.

chrome on windows -the error messages have the same meaning - as long as you dont get an error with this pr then its fine

I have tested this item ✅ successfully on 74b7091

I discovered that folder icon recently and found the only way to get rid of the pop-up box was to reload the page. Glad to see it absent.

the only way to get rid of the pop-up box was to reload the page.

@ceford Simply clicking with the mouse to another place of the page outside of the modal will close it. Not sure though how it works with keyboard navigation.

RTC

Thanks