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
Editor: Updated close icon and fixed input text overlap #4167
Conversation
9c56b31
to
0eff70a
Compare
Size Change: -49 B (0%) Total Size: 1.16 MB ℹ️ View Unchanged
|
@@ -19,6 +19,7 @@ module.exports = { | |||
'./stories/**/*.js', | |||
'../assets/src/dashboard/**/stories/*.@(js|mdx)', | |||
'../assets/src/edit-story/**/stories/*.@(js|mdx)', | |||
'../assets/src/animation/**/stories/*.@(js|mdx)', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added our effect/part animations back to storybook.
@@ -100,7 +100,7 @@ export function AnimationPart(type, args) { | |||
[ANIMATION_EFFECTS.ZOOM.value]: EffectZoom, | |||
[ANIMATION_EFFECTS.DROP.value]: EffectDrop, | |||
[ANIMATION_EFFECTS.ROTATE_IN.value]: EffectRotateIn, | |||
}[type] || throughput; | |||
}[type?.value || type] || throughput; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small fix so storybook effect animations get back to working.
Codecov Report
@@ Coverage Diff @@
## main #4167 +/- ##
==========================================
+ Coverage 80.09% 82.46% +2.37%
==========================================
Files 809 812 +3
Lines 14120 14147 +27
==========================================
+ Hits 11310 11667 +357
+ Misses 2810 2480 -330
Flags with carried forward coverage won't be shown. Click here to find out more.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Text is now clipped on link and assistive inputs. Tested chrome, firefox, safari.
Also verified that the animations are working again in storybook.
Summary
This PR updates the text input used in the Editor. It updates the clear text icon, and prevents the text from being clipped by the icon.
User-facing changes
The text input component used in the "Link" panel and the "Accessibility" panel has been updated.
Testing Instructions
1.) Open a story in the Editor.
2.) Add an image to the page.
3.) Click on the image and type in really long text into the "Link" and/or "Accessibility" panel and notice that the "clear text" icon is a nice big X, and notice that the input text does not get overlapped by the icon (it gets clipped).
Fixes #1874
Screenshot