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

dot-viewer in TUI #7

Merged
merged 273 commits into from
Feb 22, 2023
Merged

dot-viewer in TUI #7

merged 273 commits into from
Feb 22, 2023

Conversation

jaehyun1ee-furiosa
Copy link
Contributor

No description provided.

@jaehyun1ee-furiosa
Copy link
Contributor Author

jaehyun1ee-furiosa/dot-viewer#17

위 PR로 viewer::modes::Mode를 더 간단하게 정리해서 ui 모듈에서 불필요한 function parameter들을 간략화했습니다.

추가로, Keybinding을 기준으로 동작하던 것을 Command를 기준으로 동작하게 수정했습니다. Custom Command를 정의하고 파싱하는 것은 clapsubcommand를 활용했습니다.

시행착오를 겪으면서 구현해서 커밋 수가 많지만, 이전 코드와 비교하여 로직 자체는 크게 변하지 않았습니다.

Copy link

@maxstate maxstate left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

약간 수정해야 할 것을 제외하고는 문제가 없는 것 같습니다.

첨언하자면, 지금 상황에서는 구조가 많이 바뀌어서 dot-viewer.svg는 유효하지 않을 것 같아서 제외해도 좋을 것 같습니다.

src/ui/utils/htmlparser.rs Outdated Show resolved Hide resolved
src/ui/input.rs Outdated Show resolved Hide resolved
src/ui/view.rs Outdated Show resolved Hide resolved
src/viewer/utils/trie.rs Show resolved Hide resolved
src/viewer/command.rs Outdated Show resolved Hide resolved
src/viewer/utils/table.rs Outdated Show resolved Hide resolved
src/viewer/utils/table.rs Outdated Show resolved Hide resolved
src/viewer/utils/tree.rs Show resolved Hide resolved
Cargo.toml Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@maxstate maxstate merged commit 8de9671 into furiosa-ai:main Feb 22, 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.

2 participants