LB (Ben Johnston)
LB (Ben Johnston)
no idea why the UI tests are failing from this change - looks like a timeout issue. ``` FAIL client/tests/integration/groups.test.js (7.551 s) ● Groups › axe thrown: "Exceeded timeout of...
Rebased. Note @thibaudcolas we will need this in before we start looking at moving the Tabs to Stimulus controllers + would be good to have our storybook pattern library working...
I'll rebase this and probably just merge in - it's a small change and this is blocking us testing some other storybook items.
Thanks @laymonage - appreciate the quick review. The other context is that Thibaud is getting the storybook functional again + getting it to be set up in the CI so...
@Headon1729 go for it - please start with reproducing the issue in a different browser and confirming you can see the problem. Maybe add another screenshot. That way we can...
Go for it @salty-ivy
Sorry that code was just a rough start/guide - I didn't actually test it.
@salty-ivy ok - do a bit more digging - feel free to propose a better solution. My suggestion above for a fix was only done super quickly without spending too...
If you search through the code for `tab_nav_link` you will find it is used in lots of places via `{% include 'wagtailadmin/shared/tabs/tab_nav_link.html'` But the main one for this issue is...
Sorry - not sure on that.