-
Notifications
You must be signed in to change notification settings - Fork 0
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
Client work for AI Expression Summary on gene page #1330
Open
bobular
wants to merge
33
commits into
main
Choose a base branch
from
ai-expression
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.
Open
Changes from all commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
945caa8
WIP wired up to reporter service call
bobular d5506fa
Merge remote-tracking branch 'origin/main' into ai-expression
bobular 8979816
WIP
bobular 01a4ec9
bare bones UI is working, yay
bobular 3e57501
unrelated import path fix in @veupathdb/components
bobular a40901b
basic table done - needs expandable child rows
bobular 99bcada
loading flow improvements, number of experiments column
bobular 4dfb84f
div height consistency when clicking button
bobular 98c1e1e
expandable rows and some tweaks
bobular 26e3cd8
SCSS styling the structured summary
bobular 64438fa
equal margins in expandable rows
bobular 3ac1f94
Merge remote-tracking branch 'origin/main' into ai-expression
bobular 48b1016
remove failed debugging
bobular c316147
refactor expression graph opener into util function
bobular 6e9d617
wired in scroll-to and redux
bobular 7899216
use numeric rowIds
bobular eed0427
add clickable rows to toggle child row expansion
bobular 59ed257
US spelling
bobular 5165e0d
rename sections to topics
bobular 27c9013
can take up to three minutes
bobular 8a8065d
fix mouse pointer for experiment links
bobular f8c10f2
add gate for less than 5 datasets
bobular 0713886
improve scroll-to behaviour
bobular d6fae35
Merge remote-tracking branch 'origin/main' into ai-expression
bobular 9ae9932
Merge remote-tracking branch 'origin/ai-expression' into ai-expressio…
bobular 8a9cf8a
pre-open transcript expression table when ai results render
bobular eb7adf1
Merge pull request #1338 from VEuPathDB/ai-expression-topics
bobular fc37dbb
Merge remote-tracking branch 'origin/main' into ai-expression
bobular 032f4f1
first stab at this
bobular 8bf5bb1
handle initial loading properly
bobular ad4c07e
fixed up Loading spinner behaviour
bobular ccfb767
Merge pull request #1340 from VEuPathDB/progress-bar
bobular b3a5764
final progress styling tweaks
bobular File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
This file contains 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
20 changes: 20 additions & 0 deletions
20
...nomics-site/webapp/wdkCustomization/js/client/components/records/AiExpressionSummary.scss
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,20 @@ | ||
div.ai-summary { | ||
p { | ||
ul { | ||
margin-top: 0.5em; | ||
margin-bottom: 1em; | ||
} | ||
} | ||
} | ||
|
||
a.javascript-link { | ||
cursor: grab; | ||
} | ||
|
||
.AiExpressionResult-Loading { | ||
text-align: center; | ||
padding: 0; | ||
margin-top: 3em; | ||
height: unset; | ||
max-width: 75px; | ||
} |
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was just a tiny fix while in the neighbourhood. Has no impact on the AI stuff!