-
Notifications
You must be signed in to change notification settings - Fork 5
WIP: 🧪 Iobio Viewer #239
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
Draft
demariadaniel
wants to merge
11
commits into
main
Choose a base branch
from
iobio
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
WIP: 🧪 Iobio Viewer #239
+9,600
−908
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Add package * Dependencies, types, & more * Update Components * Remove transpile, update import * Remove changes, bug investigation * Remove egoJwt change * Basic Table Template * First Draft BamTable * SSR Shims for Web Components * Correct package-lock * Update import for latest package version * Stable testing stage * Elements All Rendering * Fix Labels * Dynamics Constants * Remove Window dependency * All Charts + Histograms * Move Toggle Button * Clean Up async const import * Async util function * Organize Variables * Histogram Ignore Outliers * Outliers v2 * Use published version * Massive cleanup using function default export * Improved Import * Import Components, no init required * Latest Package w/ updated imports * Fix Uneven Chart CSS * Cleanup Import and Empty Tag * Remove 2nd empty tag
* Buttons + first Layout * Tweak Chart size * Adjust Container CSS * Update File Name * Toggle Button CSS * Toggle state * Table Markup / CSS * Updated Button colors w/ Icon placeholder * Style var cleanup * Display: flex / use flex-start * Missing read coverage toggle * Import context from Iobio Library * Add Icons * Reusable Functions + Variables * JSX rule names * Copyright corrections * Latest iobio package * isFileTableActive var name * Remove Excess Fragment * Remove empty line * Remove extra fragment (bam table) * Reorg imports * Updated Var names * Add emoty line
* Buttons + first Layout * Tweak Chart size * Adjust Container CSS * Update File Name * Toggle Button CSS * Toggle state * Table Markup / CSS * Updated Button colors w/ Icon placeholder * Style var cleanup * Display: flex / use flex-start * Missing read coverage toggle * Import context from Iobio Library * Add Icons * Reusable Functions + Variables * JSX rule names * Copyright corrections * Latest iobio package * isFileTableActive var name * Remove Excess Fragment * Remove empty line * Remove extra fragment (bam table) * Reorg imports * Context Consumer / SelectedRows * Get Selected Bam File * Download URL Function + Config * Extend error message * Draft Score Download * Testing State * Add Metadata Type * Fix useMemo bug * Handle URL dependency * Working Demo Data Button * Major Reorg * Falsy Metadata keys * Table error handling * Updated Type Name * Improve Error Message * Brackets * Use ts * Separate Files * fileUrl Dependency * Improve getButtonStyles * Combine filter calls * Use .find & undefined * useContext * File Imports + Type Check * Remove 'return false' * Straighten out promise/await mix * Remove unused import * Updated Imports * Separate Demo Logic * Push to Error Page * Update conditional, router and variable * Move fileMetadata definition * Use Latest Package Version * Condense Error condition * Use positive conditional * Add nullish coalescing * Refactor selected Bam / disabled button * Updated Toggle Navigation Logic * Add Error Todo * Refactor Asyncs * Updated Async / Import * Null handling + remove extra 'if' * Score Url env.schema * Move Demo Data to its own file * Add missing await * Remove useMemo * Rename setElementState * Add env schema comment * Remove Unused useRouter * Change undefined syntax * Change Axios Promise handling * Return axios promise * Remove unused await * Prefer async for consistency * Fix catch syntax * Remove error testing * Type Narrowing * Move Demo File function * Update Loading Hook
* Stub Stats Table, separate files * Very Basic Table * Updated file table properties * Improved StatsTable CSS * Reorg components, use theme colors, add selected count * Show / Hide Button * Use span -/+ icon * Unused DemoStats * 1px adjustment, Remove nested span CSS * Header padding * File Size Label, get data function * Rename function * PR feedback, tableUtil file
* Move DemoData button out of header * Add + Organize Buttons * FullScreen Button Added * Updated Header Button Styles * Fix Icon alignments * Fixed Full Screen Icon * Updated Table Button Colors * Fixed table button disabled states * Position CountDisplay & page elements, Adjust button proportions * Remove Todo comment * Renamed BarGraph icon, 2025 Copyright * Move TableHeader to its own file, tweak button borders * Separate header buttons / toggle button definitions * Working FullScreen button, page reorg & icon color fixes * Use explicit return; additional centering * Updated toggle panel prop naming * Refactor buttons & theme * BarGraph default export fixed * Improved button sizing to fit table + naming * 1px adjustment Count Display tweak * Last detail, corrected button padding top/bottom * Merge TableHeader & Table Buttons * Use individual Footer components
* Install react-modal * Most Basic Modal Setup * First Pass CSS & Navigation; icons alphabetized * Modal File & images * Improved modal/button/image styles * File Format badges & visualizer descriptions * Finalized container proportions, cursor, semantics & reorg * Disabled states * Handle tables enabled/disabled w/ env * Fix package-lock * PR improvements * Default Iobio disabled * First batch code review updates * className use strings * Use derived state for current files * Prefer css-in-js * Break Modal into components * Handle key warning * Standardize use of Visualizer, Colocate Styles * Remove duplicate Modal.setAppElement * Anchor files/badges to bottom * Remove unused import
* Move DemoData button out of header * Add + Organize Buttons * FullScreen Button Added * Updated Header Button Styles * Fix Icon alignments * Fixed Full Screen Icon * Updated Table Button Colors * Fixed table button disabled states * Position CountDisplay & page elements, Adjust button proportions * Remove Todo comment * Renamed BarGraph icon, 2025 Copyright * Move TableHeader to its own file, tweak button borders * Separate header buttons / toggle button definitions * Working FullScreen button, page reorg & icon color fixes * Use explicit return; additional centering * Updated toggle panel prop naming * Refactor buttons & theme * BarGraph default export fixed * Improved button sizing to fit table + naming * 1px adjustment Count Display tweak * Last detail, corrected button padding top/bottom * Get index file data * Remove Demo Data logic * Prettier / cleanup changes * Fix table semantics error * Use previous Score function name * iobio version bump w/ some type updates * Add server URL * Fix package-lock * Update Index File definitions * Updated function name * Remove optional chaining & unused import * Move loadAndSetFile definition * Remove lib type import * Reorganize getFileMetaData * Separate Utility files * Remove vertical-align middle * Remove letter-spacing * Type & Var updates * Add Type Comment * Updated Type guards * Move functions out of component files
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Issues
Description of Changes
Readiness Checklist
.env.schema
file and documented in the README