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

inv-106 블럭 > 보조 레이어 > 동작 연동 #42

Merged
merged 6 commits into from
Aug 16, 2024
Merged

Conversation

bepyan
Copy link
Collaborator

@bepyan bepyan commented Aug 15, 2024

1. 기존 element-wrapper를 리팩토링

  • element 별로 있는 보조 레이어 → 하나의 전역 보조 레이어로 추출
  • 기존 element 적용되는 스타일이 보조 레이어에 영향을 주지 않도록

2. element 위아래 이동 시키는 기능

  • 추후 확장성을 위해 특정 index로 이동시키는 MOVE_ELEMENT 액션 바탕으로 MOVE_ELEMENT_UP MOVE_ELEMENT_DOWN를 만들었어요.
  • 클로드 데리고 일단 기능 구현했는데 코드가 드럽습니다..; 추후 만약 element 드래그 이동 기능 개발할 때 같이 만져볼게요

추후 개선해야 할 점

보조 레이어가 띄워져 있을 때 하위 element를 클릭할 수 없는 현상
→ box로 border치는 것이 아닌 테두리 선을 일일이 그리는 접근으로 해야할지도..?

복제 기능

드래그 이동 기능

@bepyan bepyan self-assigned this Aug 15, 2024
Copy link

linear bot commented Aug 15, 2024

@github-actions github-actions bot added the feat label Aug 15, 2024
Copy link

The latest updates on your projects.

Name Status Preview Updated (UTC)
invi ✅ Ready (Inspect) Visit Preview 2024-08-15 23:58:00

@bepyan bepyan merged commit b3b8bb4 into main Aug 16, 2024
3 checks passed
@bepyan bepyan deleted the feature/inv-106 branch August 16, 2024 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant