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: add presentation mode #72

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Conversation

seo-rii
Copy link
Contributor

@seo-rii seo-rii commented Sep 5, 2021

#69 를 패키지 형식에 맞게 다시 작성했습니다.

@seo-rii
Copy link
Contributor Author

seo-rii commented Apr 21, 2022

@hahnlee 혹시 PR 내용 한번만 확인해주실수 있나요?

@hahnlee
Copy link
Owner

hahnlee commented Aug 20, 2022

먼저 구현 감사드립니다. 그동안 패키지를 관리할 시간이 거의 없어서 신경을 많이 못쓰고 있었네요.
우선 저는 viewer 패키지의 역할을 많이 축소하고 싶습니다. 가능한 최소한의 기능을 만들고 싶고, 대신 확장성을 늘리고 싶습니다.
왜냐하면 기능은 결국 비용이 되어서 돌아가고, 한 패키지의 역할이 너무 커지기를 바라지 않습니다.

이러한 이유로 현 뷰어의 기능중에도 일부 기능은 제거할 생각도 가지고 있습니다.
아직 아이디어가 생각나지 않으나 플러그인을 구현하여 기능을 구현하도록 하는 등의 방법이 있을것 같습니다.
그래서 이 기능을 지금 포함 시키고 유지할 수 있는가에 대해서 고민이 됩니다. (추가로 이런 기능은 이슈로 만들어 먼저 논의하였다면 더 좋았을것 같습니다)

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