Skip to content

Conversation

@linxz
Copy link
Contributor

@linxz linxz commented Sep 6, 2019

0.4.32 中对部分组件做了功能优化以及 bug 修复

组件功能

  • Notice 组件:通告栏notice show属性默认为不显示,根据点击判断后显示正常,但是不会执行滚动文字效果(bug)
  • Grid 宫格组件中 3 列宫格时,修改无边框需要通过 CSS 样式来辅助(bug)
  • SwipeAction 可滑动单元格组件可修改滑动后出现的选项元素背景颜色(feat)
  • Calendar 日历组件新增 (feat)
    • 年份回调
    • 禁用日期中去掉 disable 字样
  • Tabs 选项卡组件中 tab-content 高度自适应(Tabs 组件切换bug #71)(feat)
  • List 列表组件无限加载 list-item 优化([Feature] List 组件增加下拉 loading 功能 #73)(feat)

文档内容

  • InputItem 组件新增 type 属性值的介绍
  • modal 组件属性对于 disableScroll 描述说明优化
  • notice 组件新增一条 demo 示例代码

@linxz linxz requested a review from oimz1989 September 6, 2019 08:43
onPrevYearTap() {
const { selectedMonth, selectedYear } = this.data;
let year = selectedYear;
const month = selectedMonth;
Copy link
Contributor

@oimz1989 oimz1989 Sep 6, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

month看起来没什么用,refreshdates中可以直接用 selectdMonth?

<slot />
</view>
<view class="am-list-load-more" a:if="{{loadMore === 'load'}}"><icon type="loading" size="16"/><text class="am-list-load-more-txt">{{loadContent[0]?loadContent[0]:''}}</text></view>
<view class="am-list-load-over" a:if="{{loadMore === 'over'}}">{{loadContent[1]?loadContent[1]:''}}</view>
Copy link
Contributor

@oimz1989 oimz1989 Sep 6, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里只能定义文字形式的加载更多,如果支持slot会更灵活点。antd貌似可以引入组件的。

const overTxt = this.props.loadContent[1] ? this.props.loadContent[1] : this.data.loadContent[1];
this.setData({
loadContent: [loadTxt, overTxt],
});
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

如果loadMore默认是false的话,大部分情况下,这里会出现 setData to unmount component

@oimz1989 oimz1989 merged commit f351aff into master Sep 9, 2019
@linxz linxz deleted the verion/0.4.32 branch September 9, 2019 02:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants