@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.) hq.recaptime.dev/wiki/Phorge
phorge phabricator
1
fork

Configure Feed

Select the types of activity you want to include in your feed.

a11y: Add alt text to main picture on profile page

Summary:
Images should have alternate text per https://dequeuniversity.com/rules/axe/4.10/image-alt
Ideally we'd set `alt=""` but in that case Phorge just drops the empty `alt` parameter completely.

Ref T16072

Test Plan:
Go to http://phorge.localhost/p/user/ or http://phorge.localhost/project/profile/1/ and inspect the `<img>` element inside the `people-menu-image-container` class. See the `alt` parameter.
Optionally, run an Accessibility check in Chromium Lighthouse.

Reviewers: O1 Blessed Committers, chris

Reviewed By: O1 Blessed Committers, chris

Subscribers: tobiaswiese, valerio.bozzolan, Matthew, Cigaryno

Maniphest Tasks: T16072

Differential Revision: https://we.phorge.it/D26049

+1
+1
src/applications/search/engine/PhabricatorProfileMenuItemView.php
··· 195 195 array( 196 196 'src' => $image_src, 197 197 'class' => implode(' ', $classes), 198 + 'alt' => pht('Profile picture'), 198 199 )); 199 200 200 201 $image = phutil_tag(