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: resize base node terminal on startup #3827

Merged
merged 1 commit into from
Feb 14, 2022

Conversation

zhangcheng
Copy link
Contributor

@zhangcheng zhangcheng commented Feb 12, 2022

feat(cli): resolves (#1728) Command Line Resize.

Description

Use crossterm crate to resize cli properly.

Motivation and Context

My very first good first issue to work on tari codebase. ;-)

How Has This Been Tested?

Tested on macOS Monterey (12.1).
Works for system Terminal app.
Doesn't work for iTerm2.

@delta1
Copy link
Contributor

delta1 commented Feb 12, 2022

Thank you @zhangcheng ! Will test it when I can

@delta1 delta1 changed the title Use crossterm crate to resize cli properly. feat: resize base node terminal on startup Feb 12, 2022
@delta1
Copy link
Contributor

delta1 commented Feb 13, 2022

Tested on macos, can confirm it resizes Terminal.app (width only) but not iterm.

@zhangcheng
Copy link
Contributor Author

Tested on macos, can confirm it resizes Terminal.app (width only) but not iterm.

@delta1 Do we also want to resize to fit the height of the box?

@delta1
Copy link
Contributor

delta1 commented Feb 14, 2022

Tested on macos, can confirm it resizes Terminal.app (width only) but not iterm.

@delta1 Do we also want to resize to fit the height of the box?

I think that would be great @zhangcheng 👍 take a look and if you can include that, if there's issues then this can go in first while you hack on it!

stringhandler
stringhandler previously approved these changes Feb 14, 2022
Copy link
Collaborator

@stringhandler stringhandler left a comment

Choose a reason for hiding this comment

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

Nice. Working on powershell

@stringhandler
Copy link
Collaborator

Just need signed commits and can merge it

Description
---
Use [crossterm](https://github.com/crossterm-rs/crossterm) crate to resize cli properly.

Motivation and Context
---
My very first `good first issue` to work on tari codebase. ;-)

How Has This Been Tested?
---
Tested on macOS Monterey (12.1).
Works for system Terminal app.
Doesn't work for iTerm2.
@zhangcheng
Copy link
Contributor Author

Tested on macos, can confirm it resizes Terminal.app (width only) but not iterm.

@delta1 Do we also want to resize to fit the height of the box?

I think that would be great @zhangcheng 👍 take a look and if you can include that, if there's issues then this can go in first while you hack on it!

I will try to add the height resize tomorrow.

@aviator-app aviator-app bot merged commit 00bc6e2 into tari-project:development Feb 14, 2022
zhangcheng added a commit to zhangcheng/tari that referenced this pull request Feb 15, 2022
Description
---
To resolve tari-project#1728 better

Motivation and Context
---
As a follow-up of tari-project#3827.

How Has This Been Tested?
---
Tested on macOS Monterey (12.1).
Works for system Terminal app.
Doesn't work for iTerm2.
@zhangcheng
Copy link
Contributor Author

A follow-up PR #3838 , please take a look when you have time. @delta1 , @mikethetike

aviator-app bot pushed a commit that referenced this pull request Feb 15, 2022
Description
---
To resolve #1728 better

Motivation and Context
---
As a follow-up of #3827.

How Has This Been Tested?
---
Tested on macOS Monterey (12.1).
Works for system Terminal app.
Doesn't work for iTerm2.
sdbondi added a commit to sdbondi/tari that referenced this pull request Feb 17, 2022
* development: (28 commits)
  fix(wallet): fix aggressive disconnects in wallet connectivity (tari-project#3807)
  chore: honor decimals in ERC20 (tari-project#3809)
  chore: add icons to applications (tari-project#3812)
  fix: daily test (tari-project#3815)
  ci: improvements to macos pkg (tari-project#3824)
  ci: cargo test speedups (tari-project#3843)
  feat: add persistence of transaction cancellation reason to wallet db (tari-project#3842)
  fix: update RFC links and README (tari-project#3675) (tari-project#3839)
  chore: change NodeIdentity debug (tari-project#3817)
  fix(dan): include state_root in node hash (tari-project#3836)
  feat(cli): resize terminal height (tari-project#3838)
  feat(validator-node): initial state sync implementation (partial) (tari-project#3826)
  feat: update console wallet tui (tari-project#3837)
  feat: resize base node terminal on startup (tari-project#3827)
  feat(wallet): add grpc method for setting base node (tari-project#3828)
  chore(deps): bump follow-redirects from 1.14.5 to 1.14.8 in /applications/tari_web_extension_example (tari-project#3832)
  chore(deps): bump follow-redirects from 1.14.7 to 1.14.8 in /applications/tari_collectibles/web-app (tari-project#3833)
  chore(deps): bump follow-redirects from 1.14.5 to 1.14.8 in /applications/tari_web_extension (tari-project#3834)
  chore(deps): bump follow-redirects from 1.14.7 to 1.14.8 in /applications/launchpad/gui-vue (tari-project#3831)
  chore(deps): bump follow-redirects from 1.14.4 to 1.14.8 in /integration_tests (tari-project#3829)
  ...
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.

3 participants