Command Line Resize

[StriderDM]

The Tari Base Node, does not automatically resize the terminal window on startup, leading to text being wrapped onto subsequent lines and breaking up the intended UI

To Reproduce
In a new un-resized terminal window, run cargo run --bin tari_base_node

Expected behavior
The terminal should resize to fit the longest string on startup when displaying the UI

Screenshots
Current:

Expected

[dunxen]

I could take a crack at this. 🙂

[StriderDM]

@duncandean contributions are welcome :)

[flipchan]

redo this function then

tari/applications/tari_base_node/src/cli.rs

Line 106 in 4039de1

pub fn print_banner(commands: Vec<String>, chunk_size: i32) {

[flipchan]

you want to fingerprint stdout then , which is different libc functions on different sys

https://github.com/eminence/terminal-size/blob/master/src/unix.rs here is one with unsafe rust

[flipchan]

cat src/main.rs

root@burk0:/tmp/asd# cat src/main.rs 
use libc::ioctl;
use libc::isatty;
use libc::{winsize as WinSize, TIOCGWINSZ};



fn main() {
        let fd = libc::STDOUT_FILENO;
        let is_tty: bool = unsafe { isatty(fd) == 1 };
    if !is_tty {
        panic!("no tty");
    }
    let mut winsize = WinSize {
        ws_row: 0,
        ws_col: 0,
        ws_xpixel: 0,
        ws_ypixel: 0,
    };
    if unsafe { ioctl(fd, TIOCGWINSZ.into(), &mut winsize) } == -1 {
        panic!("nope");
    }
    let rows = winsize.ws_row as usize;
    let cols = winsize.ws_col as usize;
    if rows > 0 && cols > 0 {
        println!("fluff: {} {}",cols, rows) 
        }
        let fluff = "h".repeat(cols);
    println!("{}", fluff);

}

[flipchan]

Working:

[flipchan]

working code:

use libc::ioctl;
use libc::isatty;
use libc::{winsize as WinSize, TIOCGWINSZ};

fn box_line(length: usize, is_top: bool) -> String {
    if length < 2 {
        return format!("");
    }
    if is_top {
        format!("{}{}{}", "┌", "─".repeat(length - 2), "┐")
    } else {
        format!("{}{}{}", "└", "─".repeat(length - 2), "┘")
    }
}

fn box_data(data: String, target_length: usize) -> String {
    let padding = if ((target_length - 2) / 2) > (data.chars().count() / 2) {
        ((target_length - 2) / 2) - (data.chars().count() / 2)
    } else {
        0
    };
    let mut s = format!("{}{}{}{}", " ".repeat(padding), data, " ".repeat(padding), "│");
    // for integer rounding error, usually only 1 char, to ensure border lines up
    while s.chars().count() < target_length - 1 {
        s = format!("{}{}", " ", s);
    }
    format!("{}{}", "│", s)
}


fn main() {
	let fd = libc::STDOUT_FILENO;
	let is_tty: bool = unsafe { isatty(fd) == 1 };
    if !is_tty {
        panic!("no tty");
    }
    let mut winsize = WinSize {
        ws_row: 0,
        ws_col: 0,
        ws_xpixel: 0,
        ws_ypixel: 0,
    };
    if unsafe { ioctl(fd, TIOCGWINSZ.into(), &mut winsize) } == -1 {
        panic!("nope");
    }
    let rows = winsize.ws_row as usize;
    let cols = winsize.ws_col as usize;
    if rows > 0 && cols > 0 {
        println!("fluff: {} {}",cols, rows) 
	}
	let fluff = "h".repeat(cols);
	println!("{}",box_line(cols, true));
	let rows = rows-3;
	for tari in 0..rows{
		println!("{}", box_data("test".to_string(),cols))
		}
	println!("{}",box_line(cols, false));
//    println!("{}", fluff);

}

[flipchan]

Working when resizing terminals in tmux
first smaller one:

then make the window bigger:

and re run the script it finds the new screen width

[flipchan]

lets give find terminal size in to a function:

//give me the width and height of your terminal size
fn gimmesize() -> (usize, usize) {
        let fd = libc::STDOUT_FILENO;
        let is_tty: bool = unsafe { isatty(fd) == 1 };
    if !is_tty {
        panic!("no tty");
    }
    let mut winsize = WinSize {
        ws_row: 0,
        ws_col: 0,
        ws_xpixel: 0,
        ws_ypixel: 0,
    };
    if unsafe { ioctl(fd, TIOCGWINSZ.into(), &mut winsize) } == -1 {
        panic!("nope");
    }
    let height = winsize.ws_row as usize;
    let width = winsize.ws_col as usize;
        return (width, height);
        }

[flipchan]

tried with gimmesize function and works will push to dev branch

upper one is with the update and lower one is regular

[StriderDM]

@flipchan Looks good, mind submitting a PR for this and then we can review it?

[zhangcheng]

The above code snippets from @flipchan can only determine the size of the current terminal, but lack of a way to resize it.
Nonetheless, the knowledge in it leads me to discover the crossterm crate, which solves the problem nicely. Hence PR #3827.