copying password adds two spaces

Issue #41 resolved
Fred Snyder created an issue

I found the following issue: copying preselected password adds a space both at the start and at the end of the copied text.

So for example: lets say your password is 12345. Click "Show". The text is selected. If you copy the password and then paste it, it becomes 12345 (where the _ character is a space).

This also happens with the title, username or URL. I tested a little bit and found out that it depends on how you select the text. If you double-click, two spaces are added. If you click-drag and you're not precise, two spaces are added. If you click-drag and start a little right from the text (in the white) and end at precisely the first character (so that the mouse is still a text-select cursor) it does NOT add two spaces.

This doesn't happen with the texts in the first column (Field, Title, UserName, Password, URL or Notes).

Tested this with Firefox 38.01 on Windows 8.1. I though this may be just a Firefox related issue so I also tried this in Chrome, but Chrome doesn't work at all. See screenshot.

Comments (3)

  1. Nam Nguyen repo owner

    @frederiksnijder This is not under the app control. Or perhaps the app can do something about it, but I am not sure. If you open a random page (such as this page that you are viewing), and double click on any word, you'll find that the text is copied with space(s).

    By the way, you DO NOT need to double click. When you click on "Show", the text is properly selected, you just need to hit Ctrl-C.

    Hope that helps.

  2. Nam Nguyen repo owner

    @frederiksnijder The spaces issue should be fixed in the latest version. I was wrong. The text was not properly selected.

    By the way, due to tight security, Chrome does not allow content on your local drive to access another content on local drive. That's why you see such error in Chrome.

  3. Nam Nguyen repo owner

    Fix #41: copying password adds two spaces

    A little clean up to make the selection more accurate. Added a dummy "select-this" class so that JS can pin point what to select.

    → <<cset 4f46db75cd59>>

  4. Log in to comment