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

feat(axis): Intent to ship axis.evalTextSize #3890

Merged
merged 1 commit into from
Oct 7, 2024
Merged

Conversation

netil
Copy link
Member

@netil netil commented Oct 7, 2024

Issue

#3889

Details

Implement axis text's size evaluation option.

New axis.evalTextSize option has three different usage options.

axis: {
    // 1: default, which will memoize evaluated axis text size
    evalTextSize: true,

    // 2: will evaluate the dimension of axis text size every time.
    evalTextSize: false,

    // 3: customize dimension evaluator
    evalTextSize: function(text) {
        ...
        return {width, height):
   }
}

Implement axis text's size evaluation option

Ref naver#3889
@netil netil added the feature label Oct 7, 2024
@netil netil self-assigned this Oct 7, 2024
@netil netil merged commit 87048e9 into naver:master Oct 7, 2024
9 checks passed
@coveralls
Copy link

Coverage Status

coverage: 94.215% (+0.002%) from 94.213%
when pulling 28ae3e8 on netil:axis#3889
into 9c668e6 on naver:master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants