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

Update manifest to fully support FF 66 keyboard shortcut manager #899

Closed
setyb opened this issue Mar 20, 2019 · 8 comments
Closed

Update manifest to fully support FF 66 keyboard shortcut manager #899

setyb opened this issue Mar 20, 2019 · 8 comments
Labels

Comments

@setyb
Copy link
Contributor

setyb commented Mar 20, 2019

manifest.json currently contains a blank description for the keyboard shortcut to open bitwarden in the sidebar. This causes confusion in the new FF 66 keyboard shortcut manager.

I can submit a pull request to fix this, or you can do it, whichever you prefer.

@kspearrin
Copy link
Member

kspearrin commented Mar 20, 2019

What kind of confusion? I am not sure why it is blank to begin with. Maybe we should remove it like _execute_browser_action? Does the browser provide a description for those special commands in Firefox like on Chrome?

@setyb
Copy link
Contributor Author

setyb commented Jan 5, 2020

What kind of confusion? I am not sure why it is blank to begin with. Maybe we should remove it like _execute_browser_action? Does the browser provide a description for those special commands in Firefox like on Chrome?

Here is how it currently looks in Firefox:
bw_screenshot

I'm working on the fix now, and will have it done soon.

setyb added a commit to setyb/browser that referenced this issue Jan 5, 2020
setyb added a commit to setyb/browser that referenced this issue Jan 5, 2020
@setyb
Copy link
Contributor Author

setyb commented Jan 5, 2020

I fixed the missing string, replaced a default string with a custom string, removed non-standard periods, and improved the ordering.

(And to avoid an inevitable future pending conflict, I incremented the version string.)

I hope this helps! :)

kspearrin pushed a commit that referenced this issue Jan 5, 2020
kspearrin pushed a commit that referenced this issue Jan 5, 2020
@jay-dee7
Copy link

Would we be ever able to add custom shortcuts to specific credentials? as in there's a few websites i have to login daily, it would be a great addition i think. If it's already a thing, please let me know/ I couldn't find anything so far

@setyb
Copy link
Contributor Author

setyb commented Feb 12, 2021

@jay-dee7 That's an interesting idea. To my knowledge, it's not currently possible.

@cscharf
Copy link
Contributor

cscharf commented Feb 12, 2021

@setyb , sorry just saw this from the recent conversation/update. Can we close this particular issue based on your resolution of the manifest earlier?

@setyb
Copy link
Contributor Author

setyb commented Feb 20, 2021

@cscharf Good question. I think 2 things maybe need to occur:

  1. Perhaps move the suggestion by @jay-dee7 to another issue so it doesn't get lost.
  2. See if my code resolving this issue made its way into the release. I did not sign the CLA (see Implement a fair and bilateral Contributor Agreement or remove it completely server#1115).

@bitwarden-bot
Copy link

Hi @setyb,
We're cleaning up our repositories in preparation for a major reorganization. Issues from last year will be marked as stale and closed after two weeks. If you still need help, comment to let us know and we'll look into it.
Thanks!

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

6 participants