Closed ThibaultJanBeyer closed 5 years ago
Note: after merging this PR we will have to adapt the stylesheet, since #1 uses background: url('/icon.svg'); on the list title which should be background: url('/icons/key.svg'); after this PR
Why don't you just add this to this PR?
Why don't you just add this to this PR?
1 is merged now :)
Ah perfect 👌
Done, can be merged now :)
As a user I want to be able to differentiate the extension easily from other extensions.
Currently, the extensions icon is just the default material design key It would make sense to have an icon that somehow represents Vault as well since this is a Vault companion app
Before:
App icon:
Extension icon inline:
After:
App icon:
Extension icon inline:
Note: after merging this PR we will have to adapt the stylesheet, since https://github.com/mulbc/vaultPass/pull/1 uses
background: url('/icon.svg');
on the list title which should bebackground: url('/icons/key.svg');
after this PR