-
-
Notifications
You must be signed in to change notification settings - Fork 217
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
add lessText and moreText props for renaming the button text #249
base: master
Are you sure you want to change the base?
Conversation
This PR currently has a merge conflict. Please resolve this and then re-add the |
Is it going to be merged? I'd like to have this feature |
This PR currently has a merge conflict. Please resolve this and then re-add the |
Up, is it possible to find a solution or an alternative? Can we opt for another PR with the new version? I mean, this is an important feature, and it seems nobody cares, so if I can figure out how to do it, let me know. @cbuchwald @prabhuignoto |
I have an alternative proposal of a PR: #504 |
Description
I added new props
lessText
andmoreText
to overwrite the default button textsread more
andread less
Additional context
What is the purpose of this pull request?
Before submitting the PR, please make sure you do the following
fixes #123
).