No Code Attached Yet J4 Issue Information Required bug
avatar chmst
chmst
7 Apr 2021

Steps to reproduce the issue

Open a Category list in frontend, hover the headline.

Expected result

The ordering direction of columns is indicated by an arrow as in backend tables.
No popvers.

Actual result

popovers-frontend

System information (as much as possible)

Additional comments

avatar chmst chmst - open - 7 Apr 2021
avatar joomla-cms-bot joomla-cms-bot - change - 7 Apr 2021
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 7 Apr 2021
avatar chmst chmst - change - 7 Apr 2021
Labels Added: ? J4 Issue
avatar chmst chmst - labeled - 7 Apr 2021
avatar chmst chmst - labeled - 7 Apr 2021
avatar chmst chmst - change - 7 Apr 2021
Labels Removed: ?
avatar chmst chmst - unlabeled - 7 Apr 2021
avatar sandramay0905
sandramay0905 - comment - 9 Apr 2021

Confirming the issue.

avatar YatharthVyas
YatharthVyas - comment - 10 Apr 2021

I could narrow down the issue to

. ' class="hasPopover" title="' . htmlspecialchars(Text::_($tip ?: $title)) . '"'

Here, if I remove class="hasPopover" then the popover no longer appears, and the sorting works as intended.

output)

However, this helper (grid.sort) has been used in the following files as well:

  • components\com_contact\tmpl\category\default_items.php
  • components\com_contact\tmpl\featured\default_items.php
  • components\com_content\tmpl\category\default_articles.php
  • components\com_tags\tmpl\tag\list_items.php

So, if the popover is unnecessary for these other tables too then this will be an easy fix. If not, should I look for a different helper to render the category list header?

I would be glad to submit a PR fixing this.

avatar chmst
chmst - comment - 11 Apr 2021

Popover is unneccessay in all views. A PR will be welcome!

avatar chmst chmst - change - 13 Apr 2021
Status New Closed
Closed_Date 0000-00-00 00:00:00 2021-04-13 09:11:38
Closed_By chmst
avatar chmst chmst - close - 13 Apr 2021
avatar chmst
chmst - comment - 13 Apr 2021

Closing as we have a PR

avatar YatharthVyas
YatharthVyas - comment - 15 Apr 2021

Please re-open this issue as the PR has been closed.

avatar Quy Quy - change - 15 Apr 2021
Status Closed New
Closed_Date 2021-04-13 09:11:38
Closed_By chmst
avatar Quy Quy - reopen - 15 Apr 2021
avatar Hackwar Hackwar - change - 22 Feb 2023
Labels Added: No Code Attached Yet bug
Removed: ?
avatar Hackwar Hackwar - labeled - 22 Feb 2023
avatar Quy
Quy - comment - 10 Mar 2023

@chmst Please confirm this is fixed with PR #39643.

avatar Quy Quy - change - 10 Mar 2023
Labels Added: Information Required
avatar Quy Quy - labeled - 10 Mar 2023
avatar Quy Quy - change - 28 Feb 2024
Status New Closed
Closed_Date 0000-00-00 00:00:00 2024-02-28 14:22:16
Closed_By Quy
avatar Quy Quy - close - 28 Feb 2024
avatar Quy
Quy - comment - 28 Feb 2024

Closing as this is no longer an issue.

Add a Comment

Login with GitHub to post a comment