Closed (fixed)
Project:
Project Browser
Version:
2.0.x-dev
Component:
User experience
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
4 Jun 2024 at 10:52 UTC
Updated:
22 Aug 2024 at 08:04 UTC
Jump to comment: Most recent, Most recent file

Comments
Comment #2
sourav_paulComment #4
prashant.cThanks 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.
Comment #5
sourav_paulI've fixed the code.
Maintainers pls review it.
Comment #9
sourav_paulI'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...
Comment #10
prashant.cI tried to compile the files using
yarn installandyarn 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.
Comment #11
kanchan bhogade commentedI 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"
Comment #12
gauravvvv commentedOn 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.Comment #13
gauravvvv commentedComment #14
dishakatariya commentedI 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
Comment #15
dishakatariya commentedComment #16
chrisfromredfinPlease 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?
Comment #17
chrisfromredfinOK, 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).
Comment #20
omkar-pd commentedComment #22
omkar-pd commentedTested with the latest 2.0.x and can reproduce the issue from #17.
In
EnabledSourceHandler.phpthis code overwrite the page parameters in $query.Pushed a fix.
Comment #23
chrisfromredfinManually tested and reviewed code. Works! Waiting to ship this until #3464077: Remove or demote mock to test folder goes in.
Comment #24
chrisfromredfinComment #26
chrisfromredfinThanks for this work!
Comment #27
sourav_paulAwesome..