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

feat(Calendar): new component #188

Closed
wants to merge 12 commits into from
Closed

feat(Calendar): new component #188

wants to merge 12 commits into from

Conversation

Sxs7513
Copy link
Collaborator

@Sxs7513 Sxs7513 commented Jul 15, 2022

No description provided.

src/Calendar/props.js Show resolved Hide resolved
demo/pages/Calendar/index.js Outdated Show resolved Hide resolved
<view class="amd-calendar-default-title">
<text>日期选择</text>
<image onTap="onClose" src="https://gw.alipayobjects.com/mdn/rms_6d74f8/afts/img/A*8wn2TKSKSSUAAAAAAAAAAAAAARQnAQ" class="amd-calendar-default-title-close" />
</view>
Copy link
Collaborator

Choose a reason for hiding this comment

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

只能是弹出式?怎么满足类似12306的场景(平铺式)需求?

src/Calendar/index.ts Show resolved Hide resolved
src/Calendar/index.ts Outdated Show resolved Hide resolved
src/Calendar/manager.ts Outdated Show resolved Hide resolved
src/Calendar/manager.ts Outdated Show resolved Hide resolved
src/Calendar/manager.ts Show resolved Hide resolved
demo/pages/Calendar/index.axml Show resolved Hide resolved
@Sxs7513 Sxs7513 force-pushed the feat/Calendar branch 4 times, most recently from 082bcdb to bbdfeb3 Compare July 29, 2022 09:17
@Sxs7513 Sxs7513 force-pushed the feat/Calendar branch 2 times, most recently from defdbee to 56c531b Compare August 2, 2022 08:22

| 属性 | 类型 | 必填 | 默认值 | 说明 |
| -----|-----|-----|-----|----- |
| startDate | string | 否 | - | 日历开始时间 |
Copy link
Collaborator

Choose a reason for hiding this comment

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

startDate,endDate, selectDate等感觉是Date类型更好一点

| holidayList | {name: string; range: [string, string]; type: 'holiday' | 'work'; day: string}[] | 否 | - | 节假日信息 |
| showConfirmButton | boolean | 否 | true | 是否展示确认按钮 |
| showResetButton | boolean | 否 | false | 是否展示重置按钮 |
| showlunar | boolean | 否 | false | 是否展示农历 |
Copy link
Collaborator

Choose a reason for hiding this comment

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

showlunar 建议改成showLunar

| startDate | string | 否 | - | 日历开始时间 |
| endDate | string | 否 | - | 日历结束时间 |
| selectDate | string | 否 | - | 默认选择日期 |
| selectRange | [string, string] | 否 | - | 默认选择范围 |
Copy link
Collaborator

Choose a reason for hiding this comment

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

image
选择相邻的两个,样式有点奇怪


| 事件名 | 说明 | 类型 |
| -----|-----|-----|
| onClose | 点击关闭按钮回调 | (void) => void |
Copy link
Collaborator

Choose a reason for hiding this comment

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

onChange缺失

| showConfirmButton | boolean | 否 | true | 是否展示确认按钮 |
| showResetButton | boolean | 否 | false | 是否展示重置按钮 |
| showlunar | boolean | 否 | false | 是否展示农历 |
| titleShow | boolean | 否 | - | 是否展示标题 |
Copy link
Collaborator

Choose a reason for hiding this comment

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

建议改成showTitle

</view>
<view class="amd-calendar-button-wrapper" a:if="{{showConfirmButton}}">
<view class="amd-calendar-botton-inner-cancel" a:if="{{showCancelButton}}" onTap="onCancel">重 置</view>
<view class="amd-calendar-botton-inner-check" onTap="onCheck">确 认</view>
Copy link
Collaborator

Choose a reason for hiding this comment

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

未选择状态确定按钮给个状态?现在点不动也没有反馈

@jc9702507 jc9702507 closed this Nov 30, 2022
@DiamondYuan DiamondYuan deleted the feat/Calendar branch October 27, 2023 06:37
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.

None yet

4 participants