Skip to content

executeing pass show before editpassword dialog shows up#254

Merged
annejan merged 3 commits intoIJHack:masterfrom
YoshiMan:feature/#243
Dec 11, 2016
Merged

executeing pass show before editpassword dialog shows up#254
annejan merged 3 commits intoIJHack:masterfrom
YoshiMan:feature/#243

Conversation

@YoshiMan
Copy link

Double-clicking might open previous entry instead of one double-clicked on #243

Janosch Knack added 2 commits December 11, 2016 11:38
Double-clicking might open previous entry instead of one double-clicked on IJHack#243
Double-clicking might open previous entry instead of one double-clicked on IJHack#243
@YoshiMan
Copy link
Author

While working on this ticket, this lastDecrypt property was very confusing for me. Do we really need it?
Should it be removed? I think we could handle this with the signal/slot concept?

changed override to Q_DECL_OVERRIDE
changed nullptr to Q_NULLPTR
@annejan
Copy link
Member

annejan commented Dec 11, 2016

I think lastDecrypt is/was only needed to check weather or not the clipboard should be cleared . .
But that might be able to be done within the signal/slot concept too by giving the clear clipboard scheduled call the data to check/clear . .

We don't want to clear the clipboard if it doesn't contain the password (or something else from our app) but the user has copied something else . .

@annejan annejan merged commit c88b9f2 into IJHack:master Dec 11, 2016
@YoshiMan
Copy link
Author

hm. i think its the MainWindow::clippedText, you are talking about. This one handles the clipboard related stuff.

@annejan
Copy link
Member

annejan commented Dec 11, 2016

Yep .. so we could/should probably completely refactor out the lastDecrypt stuff . .

That's still a remnant from when QtPass was a simple read-only pass viewer for management ;)

@YoshiMan YoshiMan deleted the feature/#243 branch December 11, 2016 12:08
@YoshiMan
Copy link
Author

Alright. I'll open an Issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants