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

Some minor improvements on the templating part #93

Closed
wgroenewold opened this issue Sep 9, 2015 · 3 comments
Closed

Some minor improvements on the templating part #93

wgroenewold opened this issue Sep 9, 2015 · 3 comments

Comments

@wgroenewold
Copy link

@wgroenewold wgroenewold commented Sep 9, 2015

Hi,

@dennisdegreef pointed me to this magnificent tool, much appreciation for your work! I currently work on OSX (10.9) and have a small issue with multiline and the templating. I defined some fields in QTPass and this works great with the CLI version of pass. However at the end of the password I get an extra % in a different color.

screen shot 2015-09-09 at 13 59 52

Further I would love to have a copy-field to clipboard for the template fields. These could be invoked by clicking on the field itself or its label to copy them.

Last thing is that the template fields in the add/edit form don't have a tabindex, so filling them is overly complicated. Would love to see some improvement here as well.

Keep up the good work!

@dennisdegreef
Copy link
Contributor

@dennisdegreef dennisdegreef commented Sep 9, 2015

Hi @wgroenewold I think the % sign is part of how zsh works, with the contents of your password entry missing a final newline \n-character.

@annejan
Copy link
Member

@annejan annejan commented Sep 9, 2015

That I can fix :)
Will also be nasty in bash and other shells.

@annejan annejan closed this in b0ade68 Sep 24, 2015
@annejan
Copy link
Member

@annejan annejan commented Sep 24, 2015

Will be in release 1.0.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants