Skip to content

transparency on start#472

Merged
yazgoo merged 1 commit intoskim-rs:masterfrom
yazgoo:tranparency-on-start
Oct 28, 2022
Merged

transparency on start#472
yazgoo merged 1 commit intoskim-rs:masterfrom
yazgoo:tranparency-on-start

Conversation

@yazgoo
Copy link
Copy Markdown
Collaborator

@yazgoo yazgoo commented Aug 30, 2022

WIP (points to a branch of tuikit): depends on skim-rs/tuikit#44

Adds a flag so that skim does not clear the whole screen before displaying

example

Running in this branch:

clear
blockish /home/yazgoo/Pictures/vimpapers/73lfih7qkrr61.png
echo "\e[0;0f"
./target/debug/sk --margin 20,20,20,20  --no-clear-start

Results in:

image

new flag

--no-clear-start will disable clearing the screen and enter alternate screen, (see skim-rs/tuikit#44)

@yazgoo yazgoo force-pushed the tranparency-on-start branch from 8c64d8f to 84edd23 Compare August 30, 2022 21:32
@yazgoo yazgoo force-pushed the tranparency-on-start branch from 84edd23 to 4273161 Compare October 28, 2022 19:44
@yazgoo yazgoo changed the title WIP: transparency on start transparency on start Oct 28, 2022
@yazgoo yazgoo force-pushed the tranparency-on-start branch from 4273161 to 04d4544 Compare October 28, 2022 19:45
@yazgoo yazgoo merged commit 446fb19 into skim-rs:master Oct 28, 2022
@yazgoo yazgoo deleted the tranparency-on-start branch October 28, 2022 19:56
@lilydjwg
Copy link
Copy Markdown
Contributor

lilydjwg commented Dec 3, 2022

Hi, this breaks my old usage. E.g. run sk --height 20 < some_list when there are more than 20 lines below, before this pr it returns to the line where it starts and prints the result, but after this pr, it seems to jump to the middle of my terminal screen. (--no-clear-start works fine however.)

@yazgoo
Copy link
Copy Markdown
Collaborator Author

yazgoo commented Dec 6, 2022

Hi, sorry about this, will look at it

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