Minor accessibility fix with image alt text, replace img with dvg ver…

…sion as preferred by ORCID, and update version number
eprints · Apr 11, 2022 · b349687 · b349687
1 parent 1eebadc
commit b349687
Show file tree

Hide file tree

Showing 6 changed files with 668 additions and 460 deletions.
diff --git a/cfg/cfg.d/z_orcid_support.pl b/cfg/cfg.d/z_orcid_support.pl
@@ -220,7 +220,7 @@ sub run_people_with_orcids
                 "href" => "https://orcid.org/$1",
                 "target" => "_blank",
             );
-            $orcid_link->appendChild( $session->make_element( "img", "src" => "/images/orcid_16x16.png", "alt" => "ORCID logo" ) );
+            $orcid_link->appendChild( $session->make_element( "img", "src" => "/images/orcid_id.svg", "class" => "orcid-icon", "alt" => "ORCID logo" ) );
 
             my $orcid_span = $session->make_element( "span", "class" => "orcid-tooltip" );
 

diff --git a/lib/plugins/EPrints/MetaField/Orcid.pm b/lib/plugins/EPrints/MetaField/Orcid.pm
@@ -77,7 +77,7 @@ sub render_single_value
        	my $url = "https://orcid.org/$value";
 
         my $link = $session->render_link( $url, "_blank" );
-	$link->appendChild( $session->make_element( "img", src => "/images/orcid_16x16.png", class => "orcid-icon" ) );
+	$link->appendChild( $session->make_element( "img", src => "/images/orcid_id.svg", class => "orcid-icon", alt => "ORCID logo" ) );
         $link->appendChild( $session->make_text( "https://orcid.org/$value" ) );
 
         return $link;

diff --git a/lib/static/images/orcid_id.svg b/lib/static/images/orcid_id.svg
diff --git a/lib/static/style/auto/orcid_support.css b/lib/static/style/auto/orcid_support.css
@@ -8,7 +8,9 @@
 
 .orcid-icon{
 	vertical-align: bottom;
-	padding-right: 5px;
+    padding-right: 0px;
+	margin-right: 5px;
+    width: 16px;
 }
 
 .orcid-icon img{