New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[4.0] Fix sorted by: table caption #34487
Conversation
I have tested this item ✅ successfully on f6f6428 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34487. |
I have tested this item ✅ successfully on f6f6428 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34487. |
I have not tested this item. This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34487. |
I have tested this item 🔴 unsuccessfully on f6f6428 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34487. |
@brianteeman @chmst @sandramay0905 Please test once more |
I have tested this item ✅ successfully on 7648da9 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34487. |
I have tested this item ✅ successfully on 7648da9 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34487. |
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34487. |
Thanks! |
Thanks everyone |
Pull Request for Issue #34476.
Summary of Changes
correct ternary operator in $data->direction
Testing Instructions
Visit the articles page in the backend
Sort the table, for example - by Status ascending
Just look at the table caption in the developer tools or just remove class="visually-hidden" from the code
joomla-cms/administrator/components/com_content/tmpl/articles/default.php
Line 110 in 44934f7
Actual result BEFORE applying this Pull Request
Expected result AFTER applying this Pull Request
Documentation Changes Required
No