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

Fix the props and func for marker gene heatmap tab #351

Merged

Conversation

lingyun1010
Copy link
Contributor

Marker gene heatmap page has a bug on the clusters dropdown menu, the selected value is not showing properly like in the screenshot below. I fixed the bug by testing the in the JS bundle with webpack server, i.e. without in the scxa server. The reviewer can also review it in the same way.

@ke4 ke4 added the bug Something isn't working label Jul 20, 2023
@ke4 ke4 self-requested a review July 25, 2023 10:12
Copy link
Member

@alfonsomunozpomer alfonsomunozpomer left a comment

Choose a reason for hiding this comment

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

One minor change and good to go.

selectedClusterId={this.state.selectedClusterId}
onChangeClusterId={(colourByCategory, colourByValue) => {
selectedK={this.state.selectedClusterId}
onSelectK={(colourByValue,colourByCategory) => {
Copy link
Member

Choose a reason for hiding this comment

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

The parameter colourByCategory isn’t passed and it’s also not in the function signature. Let’s remove it. I guess it’s a leftover from the previous onChangeClusterId.

Copy link
Member

@alfonsomunozpomer alfonsomunozpomer left a comment

Choose a reason for hiding this comment

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

LGTM.

@lingyun1010 lingyun1010 merged commit 34cf9b3 into develop Aug 10, 2023
2 checks passed
@lingyun1010 lingyun1010 deleted the bugfix/marker-gene-heatmap-ks-dropdown-not-loading branch August 10, 2023 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants