Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Nightingale upgrade #820

Merged
merged 192 commits into from
Dec 17, 2024
Merged
Show file tree
Hide file tree
Changes from 177 commits
Commits
Show all changes
192 commits
Select commit Hold shift + click to select a range
e000160
Add @floating-ui/dom package.
dlrice May 29, 2024
eb9b9ff
Add initial addTooltip function.
dlrice May 29, 2024
bb3dec0
Add tooltip style.
dlrice May 29, 2024
a8a3023
Use tooltip in SubCellViz.
dlrice May 29, 2024
d0184c3
Remove tooltip on target click.
dlrice May 29, 2024
dbd5dc7
Add useEffect for tooltip display in useColumns.
dlrice May 31, 2024
19b9bf8
Add div and tooltipOnHoverRef.
dlrice May 31, 2024
c49273e
Tooltip wip.
dlrice May 31, 2024
70c7190
Add data-column-name property to column label.
dlrice May 31, 2024
e582b62
FeatureViewer tooltip wip.
dlrice May 31, 2024
1ad3d02
Wip.
dlrice May 31, 2024
656e186
Don't ignore pointer events.
dlrice Jun 10, 2024
d809500
Refine showTooltip2 onClick logic.
dlrice Jun 10, 2024
d9b9a6a
Extract getUpdate and getTooltip fns.
dlrice Jun 10, 2024
410c1e1
Return cleanup function from addTooltip.
dlrice Jun 10, 2024
2de42f1
Remove tippy.js from package.json and webpack config.
dlrice Jun 11, 2024
d72672a
Use timer to make sure tooltip sticks around. Various cleaning up.
dlrice Jun 11, 2024
5a7b795
Style tweak.
dlrice Jun 11, 2024
c741313
Fix isInTooltip selector.
dlrice Jun 11, 2024
8a6b393
Type fixes.
dlrice Jun 11, 2024
bc9d4bd
Remove protvista-tooltip.d.ts
dlrice Jun 11, 2024
e71c7d4
Update snapshots.
dlrice Jun 11, 2024
abcba58
Wait a bit before listening to click events in showTooltipAtCoordinates.
dlrice Jun 11, 2024
5ebc187
Remove old protvista tooltips from AlignmentView.
dlrice Jun 11, 2024
d0be626
More cleaning up on unmount.
dlrice Jun 11, 2024
cf8bd24
Update snapshots.
dlrice Jun 11, 2024
1b8523b
Import order fix.
dlrice Jun 11, 2024
51c1184
Add tooltip titles.
dlrice Jun 11, 2024
1bd77c6
Revert franklin version for now - will need updating.
dlrice Jun 11, 2024
b67bd84
Use useCapture rather than waiting for a frame.
dlrice Jun 12, 2024
3eea477
Also need to specify useCapture in removeEventListener.
dlrice Jun 12, 2024
10c3cf1
Pin exact version for @floating-ui/dom.
dlrice Jun 12, 2024
2f5e515
Forgot to add yarn.lock
dlrice Jun 14, 2024
9e3b357
Use useCallback for onProtvistaUniprotChange.
dlrice Jun 14, 2024
812a410
Add nightingale-track packge.
dlrice Jul 29, 2024
b903424
Add NightingaleTrack custom-element.
dlrice Jul 29, 2024
f335271
Install @nightingale-elements/nightingale-navigation and create custo…
dlrice Jul 31, 2024
100c85d
Install @nightingale-elements/nightingale-sequence and create custom-…
dlrice Jul 31, 2024
b7d725a
Install @nightingale-elements/nightingale-manager and create custom-e…
dlrice Jul 31, 2024
6af3512
Initial upgrade of Nightingale in VisualFeaturesView.
dlrice Jul 31, 2024
c4667a9
Attempt to use nightingale's Feature type; remove protvistaFeatureId …
dlrice Aug 1, 2024
a29c5e7
More type fixes.
dlrice Aug 2, 2024
41bbb1d
More type and test fixes.
dlrice Aug 2, 2024
bfd9ede
Use NightingaleTrackComponent in AlignmentOverview.
dlrice Aug 2, 2024
28522bf
Replace protvista in Overview component.
dlrice Aug 2, 2024
ae72d49
Replace protvista in Wrapped component.
dlrice Aug 2, 2024
57a54b2
Replace protvista in BlastResultTable.
dlrice Aug 2, 2024
c0f3cb1
Adjust stylesheets for nightingale.
dlrice Aug 2, 2024
71cb2e1
Update getColorByType import.
dlrice Aug 2, 2024
e7b3c07
Install @nightingale-elements/nightingale-variation and create custom…
dlrice Aug 2, 2024
f8b2750
Fix Nightingale Manager reflected-attributes to be comma separated. R…
dlrice Aug 6, 2024
48bb364
Replace protvista with nightingale in VisualVariationView.
dlrice Aug 6, 2024
f040426
Update import.
dlrice Aug 6, 2024
775a109
Merge remote-tracking branch 'origin/floating-ui-tooltip' into nighti…
dlrice Aug 6, 2024
fcfe072
Regenerate yarn.lock file.
dlrice Aug 6, 2024
0c69049
Import formatTooltip from protvista.
dlrice Aug 6, 2024
c3e4912
Update imports in AlignmentView.
dlrice Aug 7, 2024
20c46b4
Update package.json
dlrice Aug 7, 2024
5a38955
Remove several protvista packages.
dlrice Aug 7, 2024
13db4a8
Remove several protvista type files; add TODO in nightingale-zoom-too…
dlrice Aug 7, 2024
e3eb9d5
Initial and messy UniProtKBFeaturesView table refactor.
dlrice Aug 7, 2024
d7aff9b
Wip UniProtKBFeaturesView refactor.
dlrice Aug 7, 2024
b4de847
UniProtKBFeaturesView table updates.
dlrice Aug 8, 2024
7f5d1a8
Move NightingaleManagerComponent from FeaturesView to VisualFeaturesV…
dlrice Aug 12, 2024
f68af2a
Initial UniProtKBFeatureColumnConfiguration
dlrice Aug 12, 2024
7d15401
Add types to UniProtKBFeatureColumnConfiguration.
dlrice Aug 13, 2024
4a5ced0
Initial TableFromData WIP.
dlrice Aug 13, 2024
90359d9
Fix position column.
dlrice Aug 14, 2024
27e159d
Get extraContent working.
dlrice Aug 14, 2024
8cb75ed
Extract and use TableFromData.
dlrice Aug 21, 2024
fa072c7
Small fixes.
dlrice Aug 21, 2024
0769170
Convert uniProtKBFeatureColumnConfiguration to just an array of objec…
dlrice Aug 21, 2024
1c7f513
Rely only on data to render each column.
dlrice Aug 21, 2024
a17ecc9
Basic filtering.
dlrice Aug 22, 2024
4fcc77d
Get highlight row working.
dlrice Aug 23, 2024
a722248
Reflect highlight in nightingale track.
dlrice Aug 23, 2024
1f23665
Wire up nightingale coordinate changes.
dlrice Aug 27, 2024
0173241
CSS tweak.
dlrice Aug 27, 2024
e0726be
Fix ID column rendering.
dlrice Aug 27, 2024
4ee7c42
Be more restrictive in +/- button query selector. Restrict width of e…
dlrice Aug 27, 2024
9e08d52
Style tweaking.
dlrice Aug 27, 2024
e14926a
Add Aurel's comments to explain ↔.
dlrice Aug 27, 2024
0b1f981
Don't bother with destructuring feature table args.
dlrice Aug 27, 2024
fd18c0d
Merge stylesheets; simplify logic.
dlrice Aug 27, 2024
17ac7c5
Replace useReducer with useState; more cleanup.
dlrice Aug 27, 2024
3092c0f
Type fixes in UniProtKBFeatureColumnConfiguration.
dlrice Aug 27, 2024
56a91a4
Adding types, assorted fixes and TODOs.
dlrice Aug 28, 2024
f9fad83
More type fixes and tidying.
dlrice Aug 28, 2024
43d31c9
Attempting to fix custom event types.
dlrice Aug 28, 2024
392aef8
Try to solve type issues with generics.
dlrice Aug 28, 2024
74572e4
Copy logic for expandable/collapse logic to Table.
dlrice Aug 29, 2024
fc76912
Create and use useExpandTable custom hook.
dlrice Aug 29, 2024
6cb9790
Show message when no filters selected.
dlrice Aug 29, 2024
9280216
Tooltip type fixes.
dlrice Aug 30, 2024
3848b58
Refactor formatTooltip.
dlrice Aug 30, 2024
6f6ba2c
Make TableFromData more generic.
dlrice Sep 2, 2024
bcac382
Attempting type fixes.
dlrice Sep 2, 2024
4cc9960
Attempting type fixes.
dlrice Sep 2, 2024
deb93a4
Cast feature type as string; make toggleAll conditional upon rowExtra…
dlrice Sep 2, 2024
ae157a7
Initial TableForData in interaction section.
dlrice Sep 2, 2024
607c2d4
Add filtering to interaction section TableForData.
dlrice Sep 2, 2024
b948f4e
Add initial Variation TableFromData.
dlrice Sep 2, 2024
058f84c
Remove TableFromData out of NightingaleManagerComponent.
dlrice Sep 2, 2024
be0f09c
Wire up row/viz highlighting in VariationViewer.
dlrice Sep 2, 2024
b4b58c0
Allow for noTranslateBody in TableFromData.
dlrice Sep 2, 2024
c1a74d1
Tidy up VariationViewer.
dlrice Sep 2, 2024
ca46fa0
Fix extra content logic.
dlrice Sep 2, 2024
44af617
Use Table in DiseaseInvolvementView.
dlrice Sep 2, 2024
823a3a8
Use Table for EMBLView.
dlrice Sep 2, 2024
3f3664d
Use Table in EMBLView.
dlrice Sep 2, 2024
cb6e7ea
Adapt DatatableWrapper tests for Table.
dlrice Sep 2, 2024
5e3f774
Remove DatatableWrapper tsx and scss.
dlrice Sep 2, 2024
555b13d
Update NightingaleZoomTool and use within VisualFeaturesView.
dlrice Sep 3, 2024
6186e5a
Remove NightingaleZoomTool from FeatureViewer for now.
dlrice Sep 3, 2024
6957ba9
Update NightingaleZoomStyles and use in VisualVariationView.
dlrice Sep 3, 2024
41abc7a
Replace protvista-table with Table and adjust space between tables.
dlrice Sep 3, 2024
86b4f2a
Adapt TableFromData for tables within search result pages.
dlrice Sep 3, 2024
0356af6
Remove protvista-datatable references in various locations.
dlrice Sep 3, 2024
93880c4
Remove protvista-datatable css variables (presently undefined in any …
dlrice Sep 3, 2024
67c836c
Remove protvista-datatable and protvista-feature-adapter.
dlrice Sep 3, 2024
254c566
Regenerate lock file.
dlrice Sep 3, 2024
48355c3
Fix processFeaturesData args.
dlrice Sep 3, 2024
7a9dcce
TableFromData in UniParcFeaturesView.
dlrice Sep 3, 2024
fc97d48
Fix getRowId.
dlrice Sep 3, 2024
3868cf1
Make primaryAccession optional in ProcessedFeature type.
dlrice Sep 3, 2024
03efd7f
TableFromData: condition upon number of rows before showing expand ta…
dlrice Sep 3, 2024
c2a51e8
Update nightingale and protvista-uniprot.
dlrice Oct 8, 2024
ea6ca8b
Ignore case for nightingale svg loader.
dlrice Oct 9, 2024
44831cb
Move VARIANT_COUNT_LIMIT into shared limits config file. Update toolt…
dlrice Oct 10, 2024
0ad040e
Just redefine customElements.define to avoid duplicate define being c…
dlrice Oct 10, 2024
2ce1100
feature.begin --> feature.start
dlrice Oct 10, 2024
f26acd0
Add NightingaleFilter with lit/react; adjust filter logic for new nig…
dlrice Oct 10, 2024
1682724
eventtype --> eventType
dlrice Oct 11, 2024
356acaf
Use formatTooltip from protvista-feature-adapter again.
dlrice Oct 11, 2024
2b43723
Move mark border and background fns to nightingale utils file.
dlrice Oct 11, 2024
8d67bf0
Add mark border and background to uniparc features view.
dlrice Oct 11, 2024
13e25ae
Fixes for alignment overview and variation viewer
dlrice Oct 16, 2024
55b0104
Don't set the height of the track.
dlrice Oct 16, 2024
93ac5fa
More attempts at fixing align overview.
dlrice Oct 16, 2024
1943f56
Remove unused imports.
dlrice Oct 17, 2024
7a95d7f
Move featureShapesMocks.json to ts.
dlrice Oct 17, 2024
5ce2f30
Type fixes.
dlrice Oct 17, 2024
3c347ff
Type fixes in sequences utils.
dlrice Oct 17, 2024
a1f32e8
More type fixes.
dlrice Oct 17, 2024
e1de257
Make FeaturesView generic.
dlrice Oct 17, 2024
1c1ddba
Merge remote-tracking branch 'origin/main' into nightingale
dlrice Oct 17, 2024
0cb9dac
Snapshot updates.
dlrice Oct 18, 2024
babb05c
Get more tests to pass.
dlrice Oct 18, 2024
5240ecf
Mock @nightingale-elements/nightingale-navigation
dlrice Oct 18, 2024
c147579
Mock more nightingale elements in ProteinProcessingSection tests.
dlrice Oct 18, 2024
58a1f2f
Code format change.
dlrice Oct 18, 2024
a90d9f8
Add protvista-uniprot to transformIgnorePatterns.
dlrice Oct 18, 2024
da165fe
Upgrade protvista-uniprot.
dlrice Oct 18, 2024
af1309d
Remove yalc references.
dlrice Oct 18, 2024
6ddb391
Tidy up.
dlrice Oct 21, 2024
7d07579
Make table header sticky
Swaathik Nov 7, 2024
6d1ba50
replace sass variables with native css variable
Swaathik Nov 7, 2024
3f3cc0b
Replace no-scroll with use-ctrl-to-zoom like protvista-uniprot
Swaathik Nov 7, 2024
da7a0b4
Remove filter-scroll.
dlrice Nov 7, 2024
82162ef
Upgrade to v5.3.0 of nightingale.
dlrice Nov 8, 2024
7eda581
Update protvista-uniprot imports.
dlrice Nov 11, 2024
7124e74
Use global custom properties for colors.
dlrice Nov 11, 2024
d12e502
Adapt NightingaleZoomTool for protvista-uniprot.
dlrice Nov 11, 2024
26f6690
Upgrade protvista-uniprot.
dlrice Nov 11, 2024
013479a
Remove all protvista packages; update import of getFeatureTooltip.
dlrice Nov 11, 2024
03987cb
Update types for protvista-uniprot.
dlrice Nov 12, 2024
4e42b3e
Upgrade protvista-uniprot.
dlrice Nov 12, 2024
f6433e9
Update ResizeObserver mock.
dlrice Nov 12, 2024
af54efb
Update snapshots.
dlrice Nov 12, 2024
f809783
Upgrade nightingale.
dlrice Nov 12, 2024
5330d20
Upgrade franklin.
dlrice Nov 12, 2024
87c78f8
Update snapshots.
dlrice Nov 12, 2024
424061c
Make border-collapse: collapse so border highlight works.
dlrice Nov 12, 2024
1e87a0d
Add separator in the tooltip and improve styles
Swaathik Dec 3, 2024
a4324d1
update protvista-uniprot
Swaathik Dec 3, 2024
785547a
Update franklin for sequence font size fix
Swaathik Dec 9, 2024
6157378
Initial scrolling behavior on feature click.
dlrice Dec 12, 2024
3ca8577
Make tooltip scrollable
Swaathik Dec 12, 2024
fdc68af
Fix header border.
dlrice Dec 12, 2024
3413af4
Refine scrolling to accomodate sticky header.
dlrice Dec 12, 2024
6006afa
Extract scrolling logic into own hook.
dlrice Dec 12, 2024
9950d68
Use useNightingaleFeatureTableScroll in VariantsViewer.
dlrice Dec 12, 2024
35bf40c
Simplify logic.
dlrice Dec 12, 2024
781cde5
Merge remote-tracking branch 'origin/nightingale' into nightingale
dlrice Dec 12, 2024
aac03c2
Use uuid for uniparc features.
dlrice Dec 12, 2024
8fa3ba4
Update snapshots.
dlrice Dec 12, 2024
fcc212c
address comments
Swaathik Dec 13, 2024
ef7a75e
rename tooltip handler function
Swaathik Dec 16, 2024
822b3d5
Merge remote-tracking branch 'origin/main' into nightingale
dlrice Dec 16, 2024
6021639
Merge remote-tracking branch 'origin/nightingale' into nightingale
dlrice Dec 16, 2024
ac2ed8a
Mock Math.random in VariationViewer unit tests.
dlrice Dec 16, 2024
36211d6
Update react to latest 18 and adjust tsconfig to resolve type issues.
dlrice Dec 16, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
27 changes: 27 additions & 0 deletions jest.setup.ts
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,8 @@ jest.mock('interaction-viewer', () => null);

jest.mock('/shared/custom-elements/NightingaleMSA', () => jest.fn());

jest.mock('@nightingale-elements/nightingale-navigation', () => jest.fn());

global.beforeEach(() => {
resetUuidV1();
});
Expand Down Expand Up @@ -50,6 +52,31 @@ jest.mock('react', () => ({
useId: mockedUseId,
}));

// eslint-disable-next-line @typescript-eslint/no-explicit-any
(global as any).ResizeObserver = class ResizeObserver {
constructor(callback: ResizeObserverCallback) {
this.callback = callback;
}

observe(target: Element) {
this.callback([{ target } as ResizeObserverEntry], this);
}

// eslint-disable-next-line class-methods-use-this, @typescript-eslint/no-unused-vars
unobserve(target: Element) {
// Mock implementation
}

// eslint-disable-next-line class-methods-use-this
disconnect() {
// Mock implementation
}

private callback: ResizeObserverCallback;
};

global.ResizeObserver = ResizeObserver;

/* "Fail on console error" util */
// Uncomment to have jest stop when a console error is shown in order to fix it
// Recommended to use with Jest's "--bail" option
Expand Down
28 changes: 13 additions & 15 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@
"jsx"
],
"transformIgnorePatterns": [
"<rootDir>/node_modules/(?!franklin-sites|p-map|aggregate-error|clean-stack|@nightingale|@lit/react|url-join).+\\.js"
"<rootDir>/node_modules/(?!franklin-sites|p-map|aggregate-error|clean-stack|protvista-uniprot|@nightingale|@lit/react|url-join).+\\.js"
],
"testPathIgnorePatterns": [
"/__mocks__/",
Expand Down Expand Up @@ -98,10 +98,19 @@
"string-width": "4.1.0"
},
"dependencies": {
"@floating-ui/dom": "1.6.5",
"@hello-pangea/dnd": "17.0.0",
"@lit/react": "1.0.5",
"@nightingale-elements/nightingale-msa": "5.0.0",
"@nightingale-elements/nightingale-filter": "5.3.2",
"@nightingale-elements/nightingale-manager": "5.3.2",
"@nightingale-elements/nightingale-msa": "5.3.2",
"@nightingale-elements/nightingale-navigation": "5.3.2",
"@nightingale-elements/nightingale-new-core": "5.3.2",
"@nightingale-elements/nightingale-sequence": "5.3.2",
"@nightingale-elements/nightingale-track": "5.3.2",
"@nightingale-elements/nightingale-variation": "5.3.2",
"@sentry/react": "8.31.0",
"@sentry/tracing": "7.27.0",
"@sentry/types": "8.31.0",
"@swissprot/rhea-reaction-visualizer": "0.0.21",
"@swissprot/swissbiopics-visualizer": "0.0.28",
Expand All @@ -111,7 +120,7 @@
"core-js": "3.38.1",
"d3": "5.16.0",
"deep-freeze": "0.0.1",
"franklin-sites": "0.0.246",
"franklin-sites": "0.0.249",
"front-matter": "4.0.2",
"history": "4.10.1",
"idb": "8.0.0",
Expand All @@ -121,18 +130,7 @@
"marked": "14.1.2",
"marked-gfm-heading-id": "4.1.0",
"p-map": "7.0.2",
"protvista-datatable": "3.8.10",
"protvista-feature-adapter": "3.8.15",
"protvista-filter": "3.8.4",
"protvista-manager": "3.8.4",
"protvista-navigation": "3.8.10",
"protvista-sequence": "3.8.13",
"protvista-tooltip": "3.8.4",
"protvista-track": "3.8.13",
"protvista-uniprot": "2.12.3",
"protvista-variation": "3.8.13",
"protvista-variation-adapter": "3.8.13",
"protvista-zoom-tool": "3.8.4",
"protvista-uniprot": "3.0.8",
"react": "18.3.1",
"react-diff-viewer-continued": "3.4.0",
"react-dom": "18.3.1",
Expand Down
14 changes: 14 additions & 0 deletions src/app/components/App.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,20 @@ import pkg from '../../../package.json';
import 'franklin-sites/franklin.css';
import './styles/app.scss';

// This is hackery is to prevent define being repeatedly called for the same
dlrice marked this conversation as resolved.
Show resolved Hide resolved
// name. This has been observed in Variant viewer and Feature viewer tabs.
const originalDefine = customElements.define;
function newDefine(
this: typeof customElements.define,
name: string,
constructor: CustomElementConstructor
) {
if (!customElements.get(name)) {
originalDefine.call(this, name, constructor);
}
}
customElements.define = newDefine;

if (process.env.NODE_ENV !== 'development') {
SentryInit({
// Key
Expand Down
4 changes: 3 additions & 1 deletion src/app/components/home-page/styles/non-critical.module.scss
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,9 @@
& > * {
position: relative;

text-shadow: -1px -1px 0 white, 1px 1px 0 white;
text-shadow:
-1px -1px 0 white,
1px 1px 0 white;
}

& img {
Expand Down
12 changes: 0 additions & 12 deletions src/app/components/styles/app.scss
Original file line number Diff line number Diff line change
Expand Up @@ -47,18 +47,6 @@
}
}

protvista-datatable:not(:defined) > table {
width: 100%;
overflow-x: auto;
table-layout: fixed;
border-collapse: collapse;

& td,
& th {
border: 1px solid grey;
}
}

protvista-manager {
&:hover,
&:focus-within {
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
@import 'franklin-sites/src/styles/mixins';
@import 'franklin-sites/src/styles/settings';
@import 'franklin-sites/src/styles/colours';

.conditions-annotations {
margin-bottom: $global-margin;
Expand All @@ -15,7 +14,7 @@
left: 0;
bottom: 0;
width: 1ch;
background-color: $colour-sapphire-blue;
background-color: var(--fr--color-sapphire-blue);
}
}
}
Expand Down Expand Up @@ -56,29 +55,29 @@
margin-bottom: 0;

& > li {
border-color: $colour-sapphire-blue;
border-color: var(--fr--color-sapphire-blue);
border-style: solid;
border-width: 0 0 1em 1ch;

.extra & {
border-color: $colour-sea-blue;
border-color: var(--fr--color-sea-blue);
}
}

& .statement {
background-color: $colour-sapphire-blue;
background-color: var(--fr--color-sapphire-blue);

&::before {
background-color: $colour-sapphire-blue;
background-color: var(--fr--color-sapphire-blue);
}
}

.extra & {
& .statement {
background-color: $colour-sea-blue;
background-color: var(--fr--color-sea-blue);

&::before {
background-color: $colour-sea-blue;
background-color: var(--fr--color-sea-blue);
}
}
}
Expand All @@ -96,10 +95,10 @@
height: 0;
border-left: 0.8em solid transparent;
border-right: 0.8em solid transparent;
border-top: $global-margin solid $colour-sapphire-blue;
border-top: $global-margin solid var(--fr--color-sapphire-blue);

.extra & {
border-top-color: $colour-sea-blue;
border-top-color: var(--fr--color-sea-blue);
}

@include fs-breakpoints('large') {
Expand All @@ -123,15 +122,15 @@
}
}

border-color: $colour-weldon-blue;
border-color: var(--fr--color-weldon-blue);
border-style: solid;
border-width: 0 0 1em 1ch;

& .statement {
background-color: $colour-weldon-blue;
background-color: var(--fr--color-weldon-blue);

&::before {
background-color: $colour-weldon-blue;
background-color: var(--fr--color-weldon-blue);
}
}
}
Expand All @@ -146,7 +145,7 @@
}

.statement {
background-color: $colour-sapphire-blue;
background-color: var(--fr--color-sapphire-blue);
color: white;
padding: 0 1px;
text-transform: uppercase;
Expand Down Expand Up @@ -193,10 +192,10 @@
height: 0;
border-top: 0.8em solid transparent;
border-bottom: 0.8em solid transparent;
border-left: $global-margin solid $colour-sapphire-blue;
border-left: $global-margin solid var(--fr--color-sapphire-blue);

.extra & {
border-left-color: $colour-sea-blue;
border-left-color: var(--fr--color-sea-blue);
}

@include fs-breakpoints('large') {
Expand All @@ -209,7 +208,7 @@
.extra {
position: relative;
margin-top: $global-margin;
border-bottom: $global-margin solid $colour-sea-blue;
border-bottom: $global-margin solid var(--fr--color-sea-blue);

&::before {
content: '';
Expand All @@ -218,11 +217,11 @@
top: 0;
bottom: 0;
width: 1ch;
background-color: $colour-sea-blue;
background-color: var(--fr--color-sea-blue);
}

& > .statement {
background-color: $colour-sea-blue;
background-color: var(--fr--color-sea-blue);
padding-left: 1ch;
margin-bottom: $global-margin;
}
Expand Down
6 changes: 2 additions & 4 deletions src/basket/styles/basket-status.module.scss
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
@import 'franklin-sites/src/styles/colours';

:global(.sliding-panel) {
.basket-status {
display: none;
Expand All @@ -12,7 +10,7 @@
&:hover,
&:focus {
svg {
background-color: $colour-sea-blue;
background-color: var(--fr--color-sea-blue);
}
}

Expand All @@ -23,7 +21,7 @@
margin: 0 0.5ch;
padding: 0.4ch;
border-radius: 50%;
background-color: $colour-sapphire-blue;
background-color: var(--fr--color-sapphire-blue);
color: white;
animation: appear 1 0.5s both ease-out;

Expand Down
2 changes: 1 addition & 1 deletion src/help/components/contextual/ContextualHelp.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ const ContextualHelp = () => {
useEffect(() => {
const eventHandler = (event: MouseEvent) => {
const element = event.target as HTMLElement;
const isInTooltip = Boolean(element.closest('[data-tippy-root'));
const isInTooltip = Boolean(element.closest('[role=tooltip]'));
// If it's a click within a tooltip, stop the propagation of the event
if (isInTooltip) {
event.stopPropagation();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,10 +10,6 @@
cursor: inherit;
opacity: 1;
}

[data-tippy-root] &::after {
display: initial;
}
}

$help-height: 35em;
Expand All @@ -30,11 +26,11 @@ $help-height: 35em;
z-index: 2147483642;

:global(.sliding-panel__header) {
background-color: $colour-help-green;
background-color: var(--fr--color-help-green);
}

:global(.sliding-panel__content) {
@include scroll-shadow($colour-sky-white);
@include scroll-shadow(var(--fr--color-sky-white));
overscroll-behavior: contain;
}

Expand Down
Loading