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

Closes #974 DNA misalignment #975

Merged

Conversation

rayzhuca
Copy link
Member

@rayzhuca rayzhuca commented Jun 7, 2024

Fixes letter spacing for all font types by switching back to textLength as opposed to letterSpacing.

Description

We can see that the screenshot in #974, the user is using "Lucida Console", which explains the misalignment because the letterSpacing value is hardcoded for "Consolas" font. The fix the bug, we just switched back to using textLength.

Screenshots:

2_staple_2_helix_origami_deletions_insertions_mods_main (1)

@rayzhuca rayzhuca requested a review from dave-doty as a code owner June 7, 2024 06:56
@dave-doty
Copy link
Member

Did you base this branch from dev or main? I see all these strange commits which makes me think it was based from main:

image

Copy link
Member

@dave-doty dave-doty left a comment

Choose a reason for hiding this comment

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

Hopefully it's good but please check the comment I have about commits.

@dave-doty dave-doty changed the title Closes #974 DNA misalignemtn Closes #974 DNA misalignment Jun 7, 2024
Copy link
Member

@dave-doty dave-doty left a comment

Choose a reason for hiding this comment

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

See above comment.

@rayzhuca
Copy link
Member Author

rayzhuca commented Jun 7, 2024

I'm not really sure why these commits showed up. I remember branching from dev and the most recent commit, I rebased to dev.

@rayzhuca rayzhuca changed the base branch from dev to main June 7, 2024 19:36
@rayzhuca rayzhuca changed the base branch from main to dev June 7, 2024 19:36
@rayzhuca rayzhuca changed the base branch from dev to main June 7, 2024 19:37
@rayzhuca rayzhuca changed the base branch from main to dev June 7, 2024 19:37
@rayzhuca rayzhuca force-pushed the 974-dna-sequence-rendering-larger-font-size-than-grid-boxes branch from 8fa4396 to 9334aba Compare June 7, 2024 19:50
@rayzhuca
Copy link
Member Author

rayzhuca commented Jun 7, 2024

I had to git push --force after rebase for some reason but it works now.

@dave-doty dave-doty merged commit 04203b4 into dev Jun 7, 2024
4 checks passed
@dave-doty dave-doty deleted the 974-dna-sequence-rendering-larger-font-size-than-grid-boxes branch June 7, 2024 21:48
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.

2 participants