Skip to content

Improve column hover-over text#826

Merged
jamie-m-a merged 3 commits intoEnsembl:mainfrom
ainefairbrother:modify-hover-text
Feb 26, 2026
Merged

Improve column hover-over text#826
jamie-m-a merged 3 commits intoEnsembl:mainfrom
ainefairbrother:modify-hover-text

Conversation

@ainefairbrother
Copy link
Copy Markdown
Contributor

@ainefairbrother ainefairbrother marked this pull request as draft February 13, 2026 09:46
@ainefairbrother ainefairbrother marked this pull request as ready for review February 13, 2026 10:31
@ainefairbrother ainefairbrother removed the request for review from dglemos February 25, 2026 11:41
@jamie-m-a jamie-m-a self-requested a review February 25, 2026 11:41
@jamie-m-a jamie-m-a self-assigned this Feb 25, 2026
Copy link
Copy Markdown
Contributor

@jamie-m-a jamie-m-a left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Has this been tested in sandbox to confirm hover text working as expected?

Comment thread SpliceAI.pm Outdated
$header{"SpliceAI_pred_DP_AG"} = "SpliceAI ($spliceai_version) delta position (bp) for acceptor gain: position (relative to variant) of max AG change (+ downstream, − upstream).";
$header{"SpliceAI_pred_DP_AL"} = "SpliceAI ($spliceai_version) delta position (bp) for acceptor loss: position (relative to variant) of max AL change (+ downstream, − upstream).";
$header{"SpliceAI_pred_DP_DG"} = "SpliceAI ($spliceai_version) delta position (bp) for donor gain: position (relative to variant) of max DG change (+ downstream, − upstream).";
$header{"SpliceAI_pred_DP_DL"} = "SpliceAI ($spliceai_version) delta position (bp) for donor loss: position (relative to variant) of max DL change (+ downstream, − upstream)."; else {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like there's a bracket missing before the else?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah yes, good spot. Fixed, then ran:

perl -c SpliceAI.pm 
SpliceAI.pm syntax OK

Copy link
Copy Markdown
Contributor

@jamie-m-a jamie-m-a left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jamie-m-a jamie-m-a merged commit 81626ec into Ensembl:main Feb 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants