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

용량이 작은 라이브러리로 교체하기 #330

Open
1 of 2 tasks
disjukr opened this issue May 13, 2019 · 4 comments
Open
1 of 2 tasks

용량이 작은 라이브러리로 교체하기 #330

disjukr opened this issue May 13, 2019 · 4 comments

Comments

@disjukr
Copy link
Owner

disjukr commented May 13, 2019

  • jquery => cash
  • moment => dayjs 그냥 모멘트 의존성을 제거함
@disjukr disjukr changed the title jquery => cash 용량이 작은 라이브러리로 교체하기 Jul 7, 2019
@cometkim
Copy link
Contributor

https://github.com/argyleink/blingblingjs

jQuery alt 로 요건 어때요? 초단순 DOM API 래퍼라 600 바이트 밖에 안하던데 VizBug 쪽 코드에서 쓰는거 보고 발견했어요.

@disjukr
Copy link
Owner Author

disjukr commented Oct 27, 2019

@cometkim

https://github.com/argyleink/blingblingjs/blob/master/src/index.js
여기 들어있는게 on, off, attr가 전부인데, on, off는 전혀 안 쓰이고 있어서 필요가 없고, 뉴스사이트 구현에서 attr말고도 다양한 jquery 메서드들을 쓰고 있어서 어려울 것 같아요.

일단 대충 뉴스사이트 구현체들 둘러봤을 때, eq, text, html, remove, parent, ... 등등을 이미 쓰고 있어요.

@cometkim
Copy link
Contributor

(말씀하신) 쓰이는 것들 둘러보니 굳이 jQuery 까지 필요 없겠고 적당한 슈가만 있으면 되지 싶은데 cash 도 좀 커보였어요. 실제로 지금도 jQuery 안쓰고 쌩 DOM API로만 구현한것들도 꽤 되던데요

@disjukr
Copy link
Owner Author

disjukr commented Oct 27, 2019

아주 예전 커밋을 보시면 jquery 서브셋을 직접 만들어서 쓰던 시절이 있어요.

당시에 어떤 메서드들을 구현했는지를 보면 아직까지 어떤 녀석들이 쓰이고 있을지 추측해볼 수 있을것 같아요.


중간에 jquery로 옮겼던게 제 기억에는 그 당시 들어있던 자체구현이 진짜 jquery가 아니다보니 동작이 다르거나 부족한 메서드들이 있어서 기여자들이 당혹스러워하는 부분이 있었는데,

그 부분에 대한 문서화를 제대로 하느니 (그 당시) 웹팩으로 옮기는 김에 jquery를 붙이는게 쉽겠다고 판단해서 그랬어요


cash를 쓰고싶은건 drop-in replacement를 하고싶어서 그런데,
cash보다 작은 것들을 찾으려면 지금처럼 너무 작거나 아니면 인터페이스가 달라서 구현체 코드를 전체적으로 고쳐야 될 것 같아서 일단은 피하고 싶어요.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants