Skip to content

Commit 180eb86

Browse files
committed
Use img to load avatars
* Stop sending referrer request data to Gravatar (reduces data sharing with Gravatar, improves user privacy). * Enable lazy loading avatar images (reduces image download request priority, defers downloading off-screen images).
1 parent 3a1e0cf commit 180eb86

3 files changed

Lines changed: 10 additions & 5 deletions

File tree

backend/classes/commentsui.php

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -259,8 +259,11 @@ public function userAvatar ($src = '{src}', $href = '{href}', $text = '{text}')
259259

260260
if ($this->setup->iconMode !== 'count') {
261261
// Create avatar image element
262-
$comments_avatar = new HTMLTag ('div', array (
263-
'style' => 'background-image: url(\'' . $src . '\');'
262+
$comments_avatar = new HTMLTag ('img', array (
263+
'src' => $src,
264+
'referrerpolicy' => 'no-referrer',
265+
'loading' => 'lazy',
266+
'alt' => ''
264267
), false);
265268
} else {
266269
// Avatars set to count

backend/classes/formui.php

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -257,8 +257,10 @@ protected function avatar ($text)
257257
}
258258

259259
// Create avatar image element
260-
$avatar = new HTMLTag ('div', array (
261-
'style' => 'background-image: url(\'' . $avatar_src . '\');'
260+
$avatar = new HTMLTag ('img', array (
261+
'src' => $avatar_src,
262+
'alt' => '',
263+
'referrerpolicy' => 'no-referrer'
262264
), false);
263265
} else {
264266
// Avatars set to count

themes/default/email-notification.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@
8484
<body>
8585
<div id="main">
8686
<div id="header">
87-
<img id="avatar" src="{avatar}" width="128" height="128" alt="avatar">
87+
<img id="avatar" src="{avatar}" width="128" height="128" alt="" referrerpolicy="no-referrer">
8888
<p id="name">{name}</p>
8989
<p>{domain}</p>
9090
</div>

0 commit comments

Comments
 (0)