-
Notifications
You must be signed in to change notification settings - Fork 194
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
增加对custom tag非data-ui-前缀的attribtue解析支持 #488
Comments
这样的话我们得有一个名单来过滤一些元素通用的属性,比如 另外可能还有 |
Global
ARIA
Event handlers
事件相关属性或许也可以简单当成 |
看上去除了事件以外, @Exodia 对于这些属性我们怎么搞,忽略之并且允许 |
polymer 是在定义控件类的时候,有个白名单属性去控制要解析标签上的哪些属性名,但我们的控件体系一开始不是这样设计的,用白名单的方式修改范围和影响面太大; 我觉得可以在Control 上增加一个静态属性或者方法(excludeParseAtrributes),返回要排除的解析属性,其余控件可以根据自己的需求覆盖。 解析流程:
|
我理解不是静态的,放在原型上覆盖方便多了 然后还要考虑给扩展用的特别剔除,这个依旧是以 |
一开始我也这么想,但是我们是先解析 html attribute,然后调用控件构造函数,传入解析后的属性,作为原型方法在未实例化控件前,也无法自动完成向上寻找,所以还不如作为静态方法,从语义上来说,也是控件类层次的东西,而不是实例上的? |
或者你的意思是,在 initOptions 前增加一个 parseOption? |
好像如你所说放在原型上确实有些问题,不过不在原型上就没办法继承,只能选择控件自己的和一个默认的实现 如果这样,我建议默认实现放在 另外一个办法,就是改控件构造函数,传一个 |
用 custom tag 意味着这个标签有区别于标准 html 标签的含义,自然也不需要 data-ui 前缀来和 html 标准属性区分了,同时也能够让整个标签的书写更加简化和利于阅读。
为了向后兼容,可以保留对 custom tag,data-ui-*的支持,同时增加非 data-ui 前缀的解析。
两者并存时,个人觉得以非 data-ui前缀的属性为准,主要是觉得 data-有自定义数据的语义存在。
The text was updated successfully, but these errors were encountered: