- Issue created by @gordon
- Status changed to Needs review
about 1 year ago 3:44am 18 October 2023 - last update
about 1 year ago 30,414 pass, 1 fail - ๐ฎ๐ณIndia gauravvvv Delhi, India
I have updated the library overrides. Attached patch for same. please review
The last submitted patch, 2: 3393441-2.patch, failed testing. View results โ
- last update
about 1 year ago 30,529 pass, 1 fail - Status changed to Needs work
about 1 year ago 5:38pm 27 November 2023 I have tested the attached patch from Gauravvvvv, and it worked for me
- ๐ฎ๐ณIndia mithun s Bangalore
Mithun S โ made their first commit to this issueโs fork.
- Merge request !6841Issue-3383441: Update the libraries override for stable9. โ (Closed) created by mithun s
- Status changed to Needs review
10 months ago 4:57am 1 March 2024 - ๐ฎ๐ณIndia mithun s Bangalore
Add a MR for the above issue from the patch #2 as the MR's can be tracked better compared to legacy patch methods.
Thanks! - Status changed to Needs work
10 months ago 2:32pm 1 March 2024 - Status changed to Needs review
6 months ago 8:52am 11 July 2024 - ๐ฎ๐ชIreland markconroy
Setting to Needs review again, I just want to check that we don't have a red herring test failure.
===
Thanks to Code Enigma for sponsoring my time to work on this. - ๐ฎ๐ชIreland markconroy
I think I've come back out of the rabbit hole and tracked down the correct issue here. It's not that stable9 is trying to reference the wrong files, stable9 does not have any CSS files any more for off_canvas, they were all removed by
c2af57548
.So when we try to run the tests, specifically
Stable9LibraryOverrideTest
, it's looking for CSS files instable9/css/core/dialog...
which are no longer there.Let's see if removing that section of the
stable9.info.yml
file and the corresponding library from$librariesToSkip
fixes this issue.===
Thanks to Code Enigma for sponsoring my time to work on this. - Status changed to Needs work
5 months ago 1:07pm 15 July 2024 - ๐บ๐ธUnited States smustgrave
Nice research!
Still seems Stable9LibraryOverrideTest is failing, with you though thought removing 'core/drupal.dialog.off_canvas', from the skipped would be the fix.
- ๐ฎ๐ชIreland markconroy
Hi @smustgrave,
Looks like we need to keep the line to override the off_canvas library in the
$librariesToOverride
array.Tests are passing now.
---
Thanks to Code Enigma for sponsoring my time to work on this. - Status changed to Needs review
5 months ago 12:03pm 19 July 2024 - Status changed to RTBC
5 months ago 2:13pm 19 July 2024 - ๐ฌ๐งUnited Kingdom catch
Committed/pushed to 11.x and cherry-picked back through to 10.3.x, thanks!
- Status changed to Fixed
5 months ago 2:47pm 19 July 2024 Automatically closed - issue fixed for 2 weeks with no activity.