Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Sequence Detail: Wrap concordances table in table-responsive div #1356

Merged
merged 1 commit into from
Mar 15, 2024
Merged

Sequence Detail: Wrap concordances table in table-responsive div #1356

merged 1 commit into from
Mar 15, 2024

Conversation

jacobdgm
Copy link
Contributor

Fixes #1355.

@jacobdgm jacobdgm mentioned this pull request Mar 12, 2024
Merged
14 tasks
dchiller
dchiller approved these changes Mar 13, 2024
View reviewed changes
Copy link
Contributor

@dchiller dchiller left a comment

Choose a reason for hiding this comment

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

Would suggest an "after" screenshot, but looks good!

@jacobdgm
Copy link
Contributor Author

Would suggest an "after" screenshot, but looks good!

I know, I know. It was a lower-priority issue that I was doing my best to fix in under 5 minutes. Thanks for the review :)

@jacobdgm jacobdgm merged commit 83299b6 into DDMAL:develop Mar 15, 2024
@jacobdgm jacobdgm deleted the 1355-sequence-concordances-table branch March 15, 2024 19:16
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@dchiller dchiller dchiller approved these changes

@lucasmarchd01 lucasmarchd01 Awaiting requested review from lucasmarchd01

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Sequence Detail Page: Concordances table lacking class="table-responsive" div
2 participants
@jacobdgm @dchiller