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 \transn Boarder and Bump Version to 2.1.1 #20

Merged

Conversation

shadinaif
Copy link
Contributor

@shadinaif shadinaif commented Apr 9, 2019

The \transn operator is not creating its own reserved space. The effect of that can be seen in operations like:

\sqrt{\transn{1.234}}
Selection_004

\transn{1.234}\\\transn{1.234}
Selection_003

This PR is intended to fix the issue using the CSS file. The result will be:

\sqrt{\transn{1.234}}
Selection_001

\transn{1.234}\\\transn{1.234}
Selection_002

Fixes #19

@shadinaif shadinaif force-pushed the shadinaif-fix-transn-boarder branch from 56f8565 to 05b352f Compare April 9, 2019 11:08
@shadinaif shadinaif changed the title Fix Boarder for transn Operator Fix \transn Boarder and Bump Version to 2.2 Apr 9, 2019
@shadinaif
Copy link
Contributor Author

@OmarIthawi would you please review this 👼

@barillax
Copy link

barillax commented Apr 9, 2019

Nice work, @shadinaif . I have verified locally that these changes fix the issues I was experiencing!

I wonder if we should add these example equations to https://github.com/OmarIthawi/arabic-mathjax/blob/master/testcases/testcases.yml ? I don't know what your policy is on when to add new cases there.

  • \sqrt{\transn{1.234}}
  • \transn{1.234}\\\transn{1.234}
  • \sqrt{{\transt{speed}{السرعة}}

Copy link
Owner

@OmarIthawi OmarIthawi left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks @shadinaif and @barillax!

package.json Outdated Show resolved Hide resolved
@shadinaif shadinaif force-pushed the shadinaif-fix-transn-boarder branch from 05b352f to 9d1c9b5 Compare April 9, 2019 17:05
@shadinaif shadinaif changed the title Fix \transn Boarder and Bump Version to 2.2 Fix \transn Boarder and Bump Version to 2.1.1 Apr 9, 2019
@shadinaif shadinaif force-pushed the shadinaif-fix-transn-boarder branch from 9d1c9b5 to 7dc0159 Compare April 9, 2019 17:08
@shadinaif
Copy link
Contributor Author

@OmarIthawi Thank you. Yes, this can be considered a patch. I have changed it to 2.1.1
Please review again :)

@OmarIthawi OmarIthawi merged commit 0a453e2 into OmarIthawi:master Apr 10, 2019
@barillax
Copy link

@shadinaif @OmarIthawi Thank you! Could you npm publish the new release so we can start using it?

@shadinaif shadinaif deleted the shadinaif-fix-transn-boarder branch April 10, 2019 18:23
@shadinaif
Copy link
Contributor Author

Thank you @OmarIthawi

The npm package has been published @barillax 👍

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.

\transn breaks vertical spacing inside of a square root
3 participants