From: Matthew Fernandez <matthew.fernandez@gmail.com>
Date: Thu, 17 Nov 2022 15:56:29 +0000 (-0800)
Subject: core plugin: stop emitting an empty 'alt' tag in cmap output
X-Git-Tag: 7.0.2~1^2
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=a214b9dc2be6b4a9d27196e2c43f7bf2b0ff0002;p=graphviz

core plugin: stop emitting an empty 'alt' tag in cmap output

This was working around a behavior on an old version of Microsoft Internet
Explorer that is no longer supported. Tooltips should show consistently on
modern browsers and the `alt` tag is once again available to be set dynamically
by Javascript or inherited.

Gitlab: closes #265
Reporte-by: ib
---

diff --git a/CHANGELOG.md b/CHANGELOG.md
index b808b1712..ecc479dd8 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -17,6 +17,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
   attributes file, `../share/graphviz/gvedit/attrs.txt`. This should make it
   more relocatable and make its behavior more consistent across operating
   systems.
+- `alt` tags are no longer set in the cmap output. #265
 
 ### Fixed
 
diff --git a/plugin/core/gvrender_core_map.c b/plugin/core/gvrender_core_map.c
index 3f62d4a37..a5a304e3d 100644
--- a/plugin/core/gvrender_core_map.c
+++ b/plugin/core/gvrender_core_map.c
@@ -114,17 +114,6 @@ static void map_output_shape (GVJ_t *job, map_shape_t map_shape, pointf * AF, in
 	    gvputs_xml(job, tooltip);
 	    gvputs(job, "\"");
 	}
-        /*
-	 * alt text is intended for the visually impaired, but such
-	 * folk are not likely to be clicking around on a graph anyway.
-	 * IE on the PC platform (but not on Macs) incorrectly
-	 * uses (non-empty) alt strings instead of title strings for tooltips.
-	 * To make tooltips work and avoid this IE issue,
-	 * while still satisfying usability guidelines
-	 * that require that there is always an alt string,
-	 * we generate just an empty alt string.
-	 */
-        gvputs(job, " alt=\"\"");
 
         gvputs(job, " coords=\"");
         switch (map_shape) {