knqyf263 / pet

Simple command-line snippet manager
MIT License
4.44k stars 223 forks source link

feat: support multiple default values #203

Closed hitzhangjie closed 6 months ago

hitzhangjie commented 2 years ago

This CL supports multiple default values like , this parameter will rendered like this in generateView:

--proxy=cfg1|cfg2|cfg3-----------|
|cfg1
|cfg2
|cfg3
|---------------------------------|

then we can use up/down arrow to select the wanted item, after press enter evaluateResult will be executed.

This CL also let the parameter view's title use larger width.

close #192

hitzhangjie commented 2 years ago

@knqyf263 Please review this PR. Thanks.

hitzhangjie commented 2 years ago

@knqyf263 Hi, could you take a little time to review this PR? I think this PR will solve the issue that some people encountered and suggested.

RamiAwar commented 7 months ago

@hitzhangjie Thanks for the PR! Will check this out! Really like the idea

RamiAwar commented 7 months ago

@hitzhangjie Interested in amending your PR to support the variable character proposal here https://github.com/knqyf263/pet/discussions/247#discussioncomment-8349705? 😁