-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
渲染时,文本前后空格的问题 #2624
Comments
也可以提个pr,给string setter增加一个配置项来控制是否trim |
Hello @beautiful-boyyy. We totally like your proposal/feedback, PR wanted。 你好 @beautiful-boyyy,我们完全同意你的提议/反馈,欢迎 PR。 |
Ok,我搞搞 |
@1ncounter 看一下渲染模块。这里我还发现两个其他的问题。 问题 1:packages/renderer-core/src/utils/common.ts 这里的 checkPropTypes 函数好像有问题。 比如下面的单测没有办法通过
问题 2:packages/renderer-core/src/renderer/base.tsx 这里的 propType 一直是 undefined。 比如综合场景下的 Message 组件有定义 propType。 看看是代码问题还是使用问题。 |
|
@beautiful-boyyy 你的问题应该不存在。 对于 props 和 children 都是正常处理的,你截图的地方只是做了 check,没有直接使用该值。 |
问题一已解决,并补充了相关的单测:#2646 |
基础fusion组件demo,拖个button就不行,貌似放在区块才没trim |
建议先自查一下, |
schema是正常的,element有问题 |
那这个就是 React 渲染相关的问题了,可以去了解一下。 |
你意思是不是引擎的问题??? |
可以写一个 React 组件,前后带上空格,看一下。 |
那这里说的 element 的问题,截图详细说一下。或者提供可以直接导入复现的 schema。 |
@1ncounter 看了一下,的确有这个问题。可以排查一下是组件问题还是渲染器的问题。 |
@liujuping 看了一下这个用例的文本显示问题原因有两个:一方面在__getComponentProps方法中调用 __parseProps 方法解析 props 时的将字符串 trim 掉了,另外将第一点的trim取消之后,button也进行了字符串的trim。 |
可以先解决第一个问题,组件的问题可以让用户/物料侧解决。 |
我来提个PR吧,不过我比较不解的是,为什么demo-general中放在自由布局里的button没问题... |
@liujuping 奇怪的是我并没有复现这个例子。始终都是被 trim 过的 |
@beautiful-boyyy 因为需要 render 的 children 是先经过处理的,props中的 children 会被这个 children 覆盖。 |
fixed with pr2654 |
引擎会对设置器传过来的string类型的变量进行trim操作,个人认为没必要。string主要来源于string setter(input组件) 或者是 textArea setter(textArea组件),如果trim的话,将会使 textArea不能首行缩进,而且对于input来说,前后存在空格并不是什么大问题。
The text was updated successfully, but these errors were encountered: