threema-ch / threema-web

The Threema Web application.
GNU Affero General Public License v3.0
998 stars 107 forks source link

Fix different warnings #850

Closed ovalseven8 closed 5 years ago

ovalseven8 commented 5 years ago

Bug Description

Bildschirmfoto vom 2019-07-10 08-15-13

Warning ``` ARIA: Attribute " aria-label ", required for accessibility, is missing on node: button.md-icon-button.md-button.md-ink-ripple ​ accessKey: "" ​ accessKeyLabel: "" ​ assignedSlot: null ​ attributes: NamedNodeMap(10) [ class="md-icon-button md-button md-ink-ripple", type="button", ng-transclude="", … ] ​ autofocus: false ​ baseURI: "https://web-beta.threema.ch/#!/messenger/conversation/contact/*SUPPORT" ​ childElementCount: 1 ​ childNodes: NodeList(3) [ #text, md-icon.ng-scope.material-icons, #text ] ​ children: HTMLCollection [ md-icon.ng-scope.material-icons ] ​ classList: DOMTokenList(3) [ "md-icon-button", "md-button", "md-ink-ripple" ] ​ className: "md-icon-button md-button md-ink-ripple" ​ clientHeight: 0 ​ clientLeft: 0 ​ clientTop: 0 ​ clientWidth: 0 ​ contentEditable: "inherit" ​ contextMenu: null ​ dataset: DOMStringMap(0) ​ dir: "" ​ disabled: false ​ draggable: false ​ firstChild: #text " " ​ firstElementChild:

Ein iframe, der sowohl "allow-scripts" als auch "allow-same-origin" für sein "sandbox"-Attribut gesetzt hat, kann seine Sandbox entfernen.

onmozfullscreenchange sollte nicht mehr verwendet werden.
onmozfullscreenerror sollte nicht mehr verwendet werden.

Your Environment

dbrgn commented 5 years ago

ARIA: Attribute " aria-label ", required for accessibility, is missing on node:

We do set aria-label, albeit dynamically. This is a false positive.

onmozfullscreenchange sollte nicht mehr verwendet werden. onmozfullscreenerror sollte nicht mehr verwendet werden.

Those don't come from our own source code.

Ein iframe, der sowohl "allow-scripts" als auch "allow-same-origin" für sein "sandbox"-Attribut gesetzt hat, kann seine Sandbox entfernen.

We don't use iframes. Context?