feat: instant applying of built-in editor

[jajugoguma]

Please check if the PR fulfills these requirements

• It's submitted to right branch according to our branching model
• It's right issue type on title
• When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
• The commit message follows our guidelines
• Tests for the changes have been added (for bug fixes/features)
• Docs have been added/updated (for bug fixes/features)
• It does not introduce a breaking change or has description for the breaking change

Description

• Added the feature that is instant applying of built-in editor.
  • It works only for built-in select, checkbox(radio), datePicker editor.
  • For use this, pass the instantApply option to true in property of options of editor of column.
  • If use it, apply new value by clicking a new value in the dropdown layer of built-in editor.

Before use it

After use it

---

Thank you for your contribution to TOAST UI product. 🎉 😘 ✨

[adhrinae]

지금은 단발성으로 필요한 부분에만 기능을 추가한 것에 지나지 않아서 전혀 추상화가 되어있지 않네요.

instantApply 속성이 적용될 가능성이 있는 모듈의 경우 그 속성을 가질 수 있도록 공통된 인터페이스를 추출해서 만든뒤 이 옵션을 적용해야 하는 클래스가 implements 키워드를 통해 인터페이스를 쓰도록 강제하는게 좋겠어요.

interface InstantlyAppliable {
  instantApplyCallback: Function | null;
}

// ...
class CheckboxEditor implements CellEditor, InstantlyAppliable {}

왜냐면 나중에 요구사항에 따라 같은 옵션을 적용하는 모듈이 추가될 수도 있고, 해당 기능을 적용해야하는 모듈들이 통일성을 가지기 때문에 코드를 읽고 유지보수하는게 쉬워지니까요.

참고로 Function 의 실제 타입을 보면 놀라우리만치 함수랑 별로 상관이 없다는걸 알 수 있어요. 그래서 가급적이면 함수 타입에 Function 을 쓰지 말고 직접적으로 함수 시그니처를 가진 형태로 직접 타입을 만들어 쓰는게 좋을거에요. 가령 캘린더는 이렇게 하고 있습니다.

type AnyFunc = (...args: any[]) => any;

근데 가급적이면 확실히 void 인 경우는 명시를 해주는게 좋아요. 리턴 타입이 있는거로 잘못 이용할 확률이 있으니까요.

https://www.typescriptlang.org/docs/handbook/declaration-files/do-s-and-don-ts.html#callback-types

[lja1018]

리뷰 완료합니다.

[lja1018]

주석처리 없인 룰 만족이 안되나요?

[jajugoguma]

넵, 기존에 다른 부분에서는 어떻게 사용했나 봤더니 주석처리 해서 사용하고 있었습니다...