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

Removed auto-selection of Row 1 of data table on concepts tab #120

Merged
merged 1 commit into from
Dec 19, 2024

Conversation

DW10
Copy link
Collaborator

@DW10 DW10 commented Dec 19, 2024

Fixes #119

Removed the default selected row 1 in the dateable on the Concepts tab.

Image from branch run_app() showing the top row is no longer blue and selected.

image

@DW10 DW10 requested a review from milanmlft December 19, 2024 14:14
Copy link

codecov bot commented Dec 19, 2024

The author of this PR, DW10, is not an activated member of this organization on Codecov.
Please activate this user on Codecov to display this PR comment.
Coverage data is still being uploaded to Codecov.io for purposes of overall coverage calculations.
Please don't hesitate to email us at [email protected] with any questions.

Copy link
Member

@milanmlft milanmlft left a comment

Choose a reason for hiding this comment

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

Looking good, thanks for fixing this Dylan! 🙏

@milanmlft milanmlft merged commit 24b0f51 into main Dec 19, 2024
8 checks passed
@milanmlft milanmlft deleted the issue-119 branch December 19, 2024 17:07
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.

Don't pre-select the first concept when starting the app
2 participants