Skip to content

Feat add calendar#33

Open
likecodeboys wants to merge 14 commits intomainfrom
feat-add-calendar
Open

Feat add calendar#33
likecodeboys wants to merge 14 commits intomainfrom
feat-add-calendar

Conversation

@likecodeboys
Copy link
Collaborator

增加日历组件

Copy link
Collaborator

Choose a reason for hiding this comment

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

console.warn 就好

Copy link
Collaborator

Choose a reason for hiding this comment

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

抛一个事件出去,开发者可以自定义处理逻辑,组件内 toast 相关可以移出了

.cube-calendar
display: flex
flex-direction: column
height: $var(calendar-height)
Copy link
Collaborator

Choose a reason for hiding this comment

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

高度需要调整,现在正好一个月的高度,看不出来有多个月,试了下310px还行?

Copy link
Collaborator

Choose a reason for hiding this comment

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

组件没有slot,可删

Copy link
Collaborator

Choose a reason for hiding this comment

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

默认为当前年第一天,可参考datePicker min、max

Copy link
Collaborator

Choose a reason for hiding this comment

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

默认为明年最后一天

Copy link
Collaborator

Choose a reason for hiding this comment

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

删除无用变量。可以检查下,比如errTipTxt、angleOffset

Copy link
Collaborator

Choose a reason for hiding this comment

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

monthList ref 未使用

<script src="./calendar-modal.ts" lang="ts"></script>

<style lang="stylus" >
@require "@mpxjs/mpx-cube-ui/src/common/stylus/helper.styl"
Copy link
Collaborator

Choose a reason for hiding this comment

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

相对路径

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.

2 participants

Comments