* 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.
* Make <content> element explicitly non-focusable
Some browsers (Firefox, upcoming versions of Chrome https://bugs.chromium.org/p/chromium/issues/detail?id=585413) make containers with `overflow: auto` / `overflow: scroll` (and the `-x` / `-y` variants) keyboard-focusable. This leads to a very awkward experience for assistive technology users navigating through the extension window, as for every view, the entire `<content>` container receives focus (and screen readers try to announce all its content in one go) before you reach the actual first control. To counteract this behaviour, this adds an explicit `tabindex="-1"` to these elements.
* Change `<content>` to `<main>`
More semantically accurate. See https://github.com/bitwarden/browser/pull/2245#issuecomment-1104553312
* Update scss to target `main` instead of `content`
* Change the previously existing `<main>` to generic `<div>`, tweak styles accordingly
Can't have two separate `<main>` elements, especially not nested ones. The original `<main>` was really just a handy wrapper for the component, but semantically should be neutral
Add "Vault Health Reports can be used to evaluate the security of your Bitwarden Personal or Organization Vault" description to the source string, "Identify and close security gaps in your online accounts by clicking the reports below."
Making corrections to the mobile contributions doc:
Update Crowdin contact from Kyle to dwbit.
Update 'User-to-User Support' forum category to 'Ask the Bitwarden Community'
Oscar Hinton
Vince Grassia
dwbit
Patrick H. Lauke
Oscar Hinton
Kyle Spearrin
Thomas Rittson
Federico Maccaroni
github-actions[bot]
Matt Gibson
Nils Fahldieck
Joseph Flinn
Daniel James Smith
Dave Nicolson