Skip to content

LASFD: Navigate to prev / next exon#699

Merged
garrettjstevens merged 3 commits intomainfrom
692-lasfd-navigate-to-prevnext-exon
Oct 3, 2025
Merged

LASFD: Navigate to prev / next exon#699
garrettjstevens merged 3 commits intomainfrom
692-lasfd-navigate-to-prevnext-exon

Conversation

@haessar
Copy link
Copy Markdown
Contributor

@haessar haessar commented Oct 1, 2025

Moved common functions from LinearApolloDisplay GeneGlyph to glyphUtils.
Replicated context menu functionality in LASFD

@haessar haessar linked an issue Oct 1, 2025 that may be closed by this pull request
@haessar haessar self-assigned this Oct 1, 2025
@haessar
Copy link
Copy Markdown
Contributor Author

haessar commented Oct 1, 2025

For some reason, CI is failing due to a lint error 'ApolloSessionModel' is defined but never used. However, it is indeed used at line 135 LinearApolloDisplay/glyphs/GeneGlyph.ts

@garrettjstevens
Copy link
Copy Markdown
Contributor

For some reason, CI is failing due to a lint error 'ApolloSessionModel' is defined but never used. However, it is indeed used at line 135 LinearApolloDisplay/glyphs/GeneGlyph.ts

That happens because in the GitHub actions it merges the main branch into the PR branch before running the steps. I merged main manually and removed that line to fix it.

@garrettjstevens garrettjstevens merged commit ec9fb8e into main Oct 3, 2025
7 checks passed
cmdcolin pushed a commit to cmdcolin/Apollo3 that referenced this pull request Mar 16, 2026
* move common functionality to utils

* replicate prev/next exon functionality in LASFD using common utils

---------

Co-authored-by: Garrett Stevens <stevens.garrett.j@gmail.com>
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.

LASFD: Navigate to prev/next exon

2 participants