- Issue created by @prashant.c
- Assigned to sourav_paul
- Merge request !494Issue #3452431 by Sourav_Paul: Pagination click should keep the focus on Project browser component → (Open) created by sourav_paul
- 🇮🇳India prashant.c Dharamshala
Thanks for raising the PR.
In case you are still working on this then keep it assigned to yourself otherwise you can unassign and move the issue to the Needs Review state.
- 🇮🇳India sourav_paul Kolkata
Sourav_Paul → changed the visibility of the branch 3452431-pagination-click-should to hidden.
- Issue was unassigned.
- Status changed to Needs review
6 months ago 7:35am 5 June 2024 - 🇮🇳India sourav_paul Kolkata
Sourav_Paul → changed the visibility of the branch 3452431-pagination-click-should-1.0.0beta5 to hidden.
- 🇮🇳India sourav_paul Kolkata
Sourav_Paul → changed the visibility of the branch 3452431-pagination-click-should-1.0.0beta@5 to hidden.
- Status changed to Needs work
6 months ago 12:13pm 6 June 2024 - 🇮🇳India sourav_paul Kolkata
I've solved the issue, but can't able to build the JS file as it requires node version greater than 16, whereas in Linux latest node version isn't available,
Hence if anyone could build the JS,
Pls update the MR.Thanks In advance...
- Status changed to Needs review
6 months ago 2:40pm 6 June 2024 - 🇮🇳India prashant.c Dharamshala
I tried to compile the files using
yarn install
andyarn build
, these are compiling.
Needs to be reviewed.@Sourav_Paul Looks like NPM won't work for this only yarn will work. You can go through this README file https://git.drupalcode.org/project/project_browser/-/blob/1.0.x/README.m... which is part of this module itself.
- Status changed to Needs work
5 months ago 8:00am 12 June 2024 - 🇮🇳India Kanchan Bhogade
I was able to reproduce the issue, But MR !494 is falling and in summary, also in Issue summary MR is an "Open failed Pipeline"
Updating issue status to "Needs work"
- 🇮🇳India gauravvvv Delhi, India
On the
MR !494
, we have hardcodedhref={'#drupalorg_mockapi'}
which breaks when we're on the recipes tab. It takes the focus to the id=drupalorg_mockapi" tab, which should be on the current tab. - Status changed to Needs review
5 months ago 4:03am 14 June 2024 - 🇮🇳India dishakatariya
I have tried to reproduce this issue but still i am getting the same results.
Attaching recordings.
Test Results are:
Click on the pagination displayed at the bottom of the page
It will show the results and take you to the top of the page - Status changed to Needs work
5 months ago 1:35pm 14 June 2024 - 🇺🇸United States chrisfromredfin Portland, Maine
Please re-check this on latest 2.0.x. I think I was able to see the behavior only the very first time I loaded it after an install, but then never again.
Also, I think paging for Recipes is completely broken, actually - we may not be storing pager settings correctly, or not resetting them when switching tabs (for example I can switch from one source to the other and the pager still thinks I'm on page 3 - and changing pages doesn't change which recipes it's showing me).
Can we get some additional testing on the 2.0.x and an update of this issue summary with the needed fixes?
- 🇺🇸United States chrisfromredfin Portland, Maine
OK, after some more digging it seems the pager just flat-out doesn't work correctly for anything except the first tab that's enabled. Depending on how you order it, that one will work. So by default, paging doesn't work on the recipes plugin. But, if you make Recipes the first in the order, it will work (and then won't work for plugins 2 or 3).
- First commit to issue fork.
- 🇮🇳India omkar-pd
omkar-pd → changed the visibility of the branch 3452431-pagination-click-should-1-0-0beta5 to hidden.
- 🇮🇳India omkar-pd
Tested with the latest 2.0.x and can reproduce the issue from #17.
In
EnabledSourceHandler.php
this code overwrite the page parameters in $query.if ($displayed_source && $displayed_source !== $source_name) { // If the source is not the one currently displayed in the UI, request // page 0. $query['page'] = 0; $query['categories'] = implode(", ", $tabwise_categories[$source_name] ?? []); }
Pushed a fix.
- Status changed to RTBC
4 months ago 2:50pm 31 July 2024 - 🇺🇸United States chrisfromredfin Portland, Maine
Manually tested and reviewed code. Works! Waiting to ship this until 📌 Remove or demote mock to test folder Needs review goes in.
-
chrisfromredfin →
committed df34f27e on 2.0.x authored by
omkar-pd →
Issue #3452431: Pagination doesn't work except on the first plugin
-
chrisfromredfin →
committed df34f27e on 2.0.x authored by
omkar-pd →
- Status changed to Fixed
4 months ago 6:10pm 7 August 2024 Automatically closed - issue fixed for 2 weeks with no activity.