[5.4] Check if page title element exists in table column script#46776
[5.4] Check if page title element exists in table column script#46776muhme merged 2 commits intojoomla:5.4-devfrom
Conversation
|
This PR fixes the console error but i am not sure it is enough. Both with and without this PR the table column selector is not loaded when it should be. This means that if you have used the table column selector to decide which columns are displayed then it is ignored when the tmpl=component is used |
|
Not sure what to do then in the case when no table name is set and no page title is defined. Adding the name attribute to the tables is out of this scope for this pr. |
|
For me the bug is not the console error but the fact that the table column script will not run in this view |
|
Thanks for sharing your opinion. |
|
pretty sure we already have a pr that would address the situation where the page does not have a title |
|
I couldn't find one. |
|
I have tested this item ✅ successfully on 21c8e72 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46776. |
|
I have tested this item ✅ successfully on 21c8e72 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46776. |
|
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46776. |
|
✅ Final test before merge using JBT
|
|
Thank you @laoneo for your contribution. Thank you @brianteeman for support. Thank you @465645 and @dautrich for testing. |
Summary of Changes
When opening a back end list with tmpl=component parameter there is a javascript error for a missing element.
Testing Instructions
Open in the back end the url /administrator/index.php?option=com_content&view=articles&tmpl=component.
Actual result BEFORE applying this Pull Request
Javascript error:
can't access property "textContent", document.querySelector(...) is null
Expected result AFTER applying this Pull Request
No Javascript error.
Link to documentations
Please select:
Documentation link for docs.joomla.org:
No documentation changes for docs.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed