Labels |
Added:
?
|
Status | New | ⇒ | Discussion |
Category | ⇒ | com_search JavaScript |
Labels |
Added:
J3 Issue
|
The issue was gone before and after scrolling?
I’m not at my computer to confirm. I will check later today.
My test here show that the modal is indeed dark (in fact the page itself is darkened and the modal is beneath) but the indexing is done and then the modal auto-close.
@infograf768 Can you test with beta3? Be sure to scroll down the page until the toolbar sticks at the top and then click the index
button.
The point is that with staging I do not need to scroll at all. The issue happens whatever the size of the window and no scroll.
Will try to get a beta3.
Please read.. #22465 (comment)
beta3 => same issue!!!
Well, it has not always been present. It was working OK in 3.8.12...
And is the issue present if you scroll first then open the modal (as stated in the description)?
In staging, scrolling or not scrolling is unimportant.
As I said above, the modal is hidden behind the backdrop.
The issue is unrelated to scrolling.
Tested also with scrolling in 3.8.12 and it is also unrelated. It works in both cases.
I consider this as a non B/C situation. A 3rd party extension could be displaying a modal this way where informations have to be entered.
Sighs...
Navigate to Components -> Smart Search
Scroll down so toolbar sticks
Click 'Index'
This issue is everything related to scrolling. This issue was only present after scrolling. Open 3.8.12 and follow the above. I repeat...
#22465 is a separate issue and is only present since 3.9 beta4.
In fact, you are right when there are already indexes in the page.
When indexes have been cleared, no issue after scrolling, as you can see below (3.8.12).
I indeed never tested Indexing when the list contains some stuff. I systematically clear Index before indexing...
EDIT: same situation in staging.
EDIT2: The difference in staging seems that it is also broken when NOT scrolling.
When I say fix, rendering the modal anywhere outside of the subhead div should do it (eg. Batch modal in com-content)
For finder we have
$toolbar->appendButton(
'Popup', 'archive', 'COM_FINDER_INDEX', 'index.php?option=com_finder&view=indexer&tmpl=component', 500, 210, 0, 0,
'window.parent.location.reload()', 'COM_FINDER_HEADING_INDEXER'
);
For batch, we have
$toolbar->popupButton('batch')
->text('JTOOLBAR_BATCH')
->selector('collapseModal')
->listCheck(true);
Also, the issue does not only concern Indexed Content but also the Statistics button in this view as in the other finder views (Maps & Search Filters)
$toolbar->appendButton('Popup', 'bars', 'COM_FINDER_STATISTICS', 'index.php?option=com_finder&view=statistics&tmpl=component', 550, 350);
Status | Discussion | ⇒ | Information Required |
@ciar4n Can you confirm after the revert PR that this issue is fixed?
@jwaisner As noted above (#20255 (comment)), the revert only fixed the issue for scenarios when you don't scroll. When scrolling down (so the toolbar becomes sticky), the issue still exists in v3.9.16.
Priority | Medium | ⇒ | Low |
Status | Information Required | ⇒ | Confirmed |
Is this still an issue? I can't replicate this on J4 in Smart Search.
Labels |
Added:
No Code Attached Yet
Information Required
Removed: ? |
Closing as there are no more J3 releases other than security fixes. Not an issue in J4.
Status | Confirmed | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2022-08-15 13:43:27 |
Closed_By | ⇒ | Quy |
PR #22367 produces this issue. It was ok in beta3 but not in beta4.