* Change links to buttons, expose `aria-pressed` for toggles
- also make existing `<a routerLink...>` type controls keyboard focusable with the addition of `tabindex="0"`
* Correctly set aria-pressed
now that I have a working build environment, could verify correct way to set this with my limited Angular knowledge
* Change more links to buttons, initial style changes
* Fix layout of <button> elements with .box-content-row
* Update jslib submodule
* Add `aria-expanded` to the send view's "Show options" expand/collapse control
* Fix position of "Edit" pencil when hovering over folders
* Update jslib
* Change sends list links to buttons
* Add `aria-pressed` to vault and send list buttons
Programmatically denote which of the buttons is currently active/shown in the right-most panel
* Fix incorrect "Options" expand/collapse button in add-edit view
Currently, that buttons lacks an accName because the "Options" text is outside of it.
* Add `aria-pressed` to the send left-hand column filters
* Simplify base, list, and vault styles
Since links are now buttons, no need to double up selectors for both types of elements. No need to double-up theming in base, as this also causes incorrect "x" in toasts.
* Remove unnecessary `position:relative`
Fixes issue with cut-off focus outlines, has no other adverse effect
* Fix styling for last child of action buttons
Old approach of making right padding smaller results in unsightly, off-center icon (noticeable when focus outline is visible). This visually remains the same, but reduces right-hand margin instead.
* username generator implemented
* disable type when coming from add/edit
* restyle buttons to new icon-btn
* update generated-wrapper styles
* only show policy messages for passwords
* make generated-wrapper a standalone style
* Update src/app/vault/password-generator.component.html
Co-authored-by: Thomas Rittson <31796059+eliykat@users.noreply.github.com>
* aria-expanded on show options
Co-authored-by: Thomas Rittson <31796059+eliykat@users.noreply.github.com>
* [bug] Ensure accounts logging out in the background doesn't impact active account ui
The main issue here: inactive accounts with a logout timeout actually log out the active account" is fixed by pulling in jslib.
These changes are for some asthetic issues I noticed, where inactive accounts logging out still fires a switchAccount event, which causes a loading spinner to appear and a sync that redraws the vault.
* Only load if the account being logged out is the active account:
* Replaced any calls to `stateService.activeAccount.getValue` with references to `this.activeUserId`, since we subscribe to that in the component now.
* Only send a "switchAccount" method if the active user before a clean and after a clean don't match
* [bug] Ensure default vault timeout is set to On Restart
We dont override the StateMigrationService instance that is injected in desktop, so it is not aware of desktop defaults.
This results in fresh accounts having a "Never" timeout action insteads of "On Restart"
* Use the correct StateMigrationService instance
* update jslib
* [bug] Move bulk of system lock checks into app.component
* [review] Extract shared system timeout logic
* [review] Correct an improper number
* [review] Opt for a more locally scoped system timeout helper than a dedicated enum
* [Icons] Update Font Sheet
* Added import statement for styles
* updated to clone icon
* Changed save to save changes icon
* Revert to using base bwi class
* Updated import order for bwi icon styles
* Converted new account switcher ui updates
* Bump jslib
* Fix occurances where bwi-eye-slash was used instead of btw-eye-slash-2
* Move settings cog to the left side
* Updated eye/eye-slash icon references
* Update jslib
* Update jslib
* Update fallback image for cipher icon
* Update jslib
Co-authored-by: Hinton <oscar@oscarhinton.com>
The DI refactor created a bad initlizer for SystemService that left out the reload callback.
This callback is null in prod, so I just set up a factory initlizer that used null for the callback value.
This fixes a bug causing clipboard clearing to not function, as platformUtilsService was not correctly injected.