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

Adds support for size prop in Editor and EditorContent components. #1250

Merged

Conversation

deepakjosp
Copy link
Contributor

@deepakjosp deepakjosp commented Oct 22, 2024

Fixes #1247

Description

  • Sets class names based on the size prop on Editor and EditorContent component for font scaling.

Checklist

  • I have made corresponding changes to the documentation.
  • I have updated the types definition of modified exports.
  • I have verified the functionality in some of the neeto web-apps.
  • I have added the necessary label (patch/minor/major - If package publish
    is required).

Reviewers

@neetodeploy neetodeploy bot temporarily deployed to neeto-editor-ht2z-pr-1250 October 22, 2024 12:15 Inactive
@deepakjosp deepakjosp added the patch Releases small requests or bug fixes. label Oct 22, 2024
@deepakjosp
Copy link
Contributor Author

@AbhayVAshokan _a please review.

cc: @praveen-murali-ind

Copy link
Member

@AbhayVAshokan AbhayVAshokan left a comment

Choose a reason for hiding this comment

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

Looks good. @praveen-murali-ind _a I am assigning this to you so that you can add necessary CSS overrides when size is large.

@praveen-murali-ind
Copy link
Contributor

@AbhayVAshokan The sizing styles can be addressed as a separate issue.

@praveen-murali-ind praveen-murali-ind merged commit 47927f4 into main Oct 22, 2024
@praveen-murali-ind praveen-murali-ind deleted the 1247-add-support-for-sizing-in-editor-and-editoroutput branch October 22, 2024 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch Releases small requests or bug fixes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for sizing in Editor and EditorOutput
3 participants