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

Use context menu #43

Open
bitboxer opened this issue Nov 28, 2011 · 3 comments
Open

Use context menu #43

bitboxer opened this issue Nov 28, 2011 · 3 comments
Labels

Comments

@bitboxer
Copy link
Contributor

Use context menu to fill form

https://developer.chrome.com/extensions/contextMenus

@lazynooblet
Copy link
Contributor

I'm thinking of taking a stab at this over the next few weeks/months. To start with I'd move all the temp variables and logic to the background page so it can be shared, then use message passing to perform the queries. Is that a logical way of thinking? I haven't learnt Chrome specific ways of doing things yet.

Once the code is split away from the popup it opens up other possibilities like a coolkey, or a little visible button in the password fields.

@bitboxer
Copy link
Contributor Author

It depends. If it is just logic, you coud make it a separate class and use it everywhere you need it. That would remove the message overhead for this.

@lazynooblet
Copy link
Contributor

Yes, okay, if that's possible thats the way to do it. I think I need to make a test chrome extension to get around how the API works and what the limitations are so I can make the changes here more efficiently.

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

No branches or pull requests

2 participants