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

关于isLive的参数配置 #14

Closed
skyline0705 opened this issue Nov 8, 2017 · 5 comments
Closed

关于isLive的参数配置 #14

skyline0705 opened this issue Nov 8, 2017 · 5 comments
Assignees
Labels

Comments

@skyline0705
Copy link
Contributor

目前isLive的的选项除了在controller-plugin中看到使用外,地方并无看到使用,而且此处又与plugin中细粒度控制图标展示与否冲突,有一定的歧义性,希望对于此参数做进一步考虑

@skyline0705
Copy link
Contributor Author

抱歉刚看到在自动播放上还有使用,综合考虑,isLive的歧义性较多,还是希望对此参数能有一个相对明确的定义,配置一般是作为组件层级的使用,但是目前isLive看起来更接近于业务层级的配置,而不同的业务场景下对isLive的定义其实或多或少都有自己的定义

@toxic-johann
Copy link
Member

isLive 主要用于标记是否是直播,在 kernel 层处理会有所区别。

另外其他表层插件会根据是否是直播做不同区别的展示。

@skyline0705
Copy link
Contributor Author

skyline0705 commented Nov 9, 2017

我觉得还是需要明确一下isLive的定义,目前isLive直接跟autoplay设置冲突,设置isLive之后autoplay参数直接默认为true,这块很容易造成使用歧义性

抱歉这块是我设置失误…
不过我还是倾向于isLive更倾向于是对kernel层的设置,而非对表现层插件,表现层的逻辑还是尽量交由开发者来决定比较合适。

@toxic-johann
Copy link
Member

关于表现层的行为这是因为当时设置皮肤综合考虑了点播和直播的情况。

这一块 @yandeqiang 关注下。

@yandeqiang
Copy link
Contributor

目前表现层,如控制条, 拿 isLive 这个参数, 是为了给一个默认图标组合而已,如果,你自己有对图标的组合有做具体的设置,则采用用户的设置。可以看下。https://github.com/Chimeejs/chimee-plugin-controlbar/blob/master/README.md#组件相关问题

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

No branches or pull requests

3 participants