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(input-number): add keyboard props #2592

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

zengjingke
Copy link

@zengjingke zengjingke commented Jul 27, 2023

Types of changes

  • New feature
  • Bug fix
  • Enhancement
  • Component style change
  • Typescript definition change
  • Documentation change
  • Coding style change
  • Refactoring
  • Test cases
  • Continuous integration
  • Breaking change
  • Others

Background and context

举个例子,当我在商品表格编辑中,我需要输入数量,这时用到input-number组件,我想通过上下按键快速切换到上一行或者下一行的input-number用于快速输入完 整列的数量,但是目前的input-number存在默认上下键盘加减值的事件,导致了我无法实现自行上下按键事件

Solution

增加keyboard属性,当我keyboard设置为false时,上下按键的事件自行定义,不进行上下按键加减逻辑
当我keyboard设置为ture时,依旧保持上下加减逻辑

How is the change tested?

Changelog

Component Changelog(CN) Changelog(EN) Related issues
input-number 增加keyboad属性 add keyboard props

Checklist:

  • Test suite passes (npm run test)
  • Provide changelog for relevant changes (e.g. bug fixes and new features) if applicable.
  • Changes are submitted to the appropriate branch (e.g. features should be submitted to feature branch and others
    should be submitted to main branch)

Other information

@zengjingke zengjingke changed the title feat: add keyboard props feat: (input-number)add keyboard props Jul 27, 2023
@zengjingke zengjingke changed the title feat: (input-number)add keyboard props feat(input-number) add keyboard props Jul 27, 2023
@zengjingke zengjingke changed the title feat(input-number) add keyboard props feat(input-number): add keyboard props Jul 27, 2023
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

2 participants