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

FE-6026 text editor validations #6274

Merged
merged 2 commits into from
Oct 17, 2023
Merged

Conversation

edleeks87
Copy link
Contributor

@edleeks87 edleeks87 commented Aug 25, 2023

Proposed behaviour

Updates CharacterCount styles with tokens
Adds support for new validation and character count layout to TextEditor

image

image

Current behaviour

Character styles using pixel values instead of tokens
TextEditor does not support new validation or character count layout

Checklist

  • Commits follow our style guide
  • Screenshots are included in the PR if useful
  • Unit tests added or updated if required
  • Cypress automation tests added or updated if required
  • Storybook added or updated if required
  • Typescript d.ts file added or updated if required

QA

  • Tested in CodeSandbox/storybook
  • Add new Cypress test coverage if required
  • Carbon implementation matches Design System/designs
  • UI Tests GitHub check reviewed if required

Additional context

Testing instructions

The following CodeSandbox is an example of the broken behaviour.
You can see the new behaviour by looking at the version in the comment by codesandbox[bot].

@codesandbox-ci
Copy link

codesandbox-ci bot commented Aug 25, 2023

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 91bb986:

Sandbox Source
carbon-quickstart Configuration
carbon-quickstart-typescript Configuration
carbon-quickstart PR

@edleeks87 edleeks87 force-pushed the FE-6026-text-editor-validations branch from bdb880b to 31f353a Compare August 29, 2023 08:18
@edleeks87 edleeks87 force-pushed the FE-6026-text-editor-validations branch 2 times, most recently from 38de305 to 7ac7888 Compare October 10, 2023 09:02
@DipperTheDan DipperTheDan self-requested a review October 10, 2023 13:15
@robinzigmond robinzigmond self-requested a review October 10, 2023 13:51
@edleeks87 edleeks87 force-pushed the FE-6026-text-editor-validations branch from 7ac7888 to be55bd6 Compare October 10, 2023 15:48
@DipperTheDan DipperTheDan marked this pull request as ready for review October 10, 2023 16:06
@DipperTheDan DipperTheDan requested review from a team as code owners October 10, 2023 16:06
@edleeks87 edleeks87 force-pushed the FE-6026-text-editor-validations branch from be55bd6 to 91bb986 Compare October 17, 2023 09:04
@edleeks87 edleeks87 merged commit 87d8728 into master Oct 17, 2023
@edleeks87 edleeks87 deleted the FE-6026-text-editor-validations branch October 17, 2023 09:30
@carbonci
Copy link
Collaborator

🎉 This PR is included in version 121.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

6 participants