remove support for citecolor from experimental driver

latex3 · Mar 17, 2020 · 7968eb6 · 7968eb6
1 parent 2fa492a
commit 7968eb6
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 33 deletions.
diff --git a/hgeneric-experimental.def b/hgeneric-experimental.def
@@ -436,7 +436,7 @@
   \fi
   \Hy@StartlinkName{}%
   \Hy@testname
-  \expandafter\Hy@colorlink\csname @#1color\endcsname
+  \expandafter\Hy@colorlink\csname @linkcolor\endcsname
 }
 
 \ExplSyntaxOn
@@ -479,22 +479,7 @@
 \cs_new_protected:Npn \hyper@linkstart #1 #2
   {
     \Hy@VerboseLinkStart{#1}{#2}% only for debug
-    % is this still needed or are the hooks handled elsewhere??
-    \tl_if_exist:cTF            % or test for blank??
-      { @#1bordercolor }
-      {
-        \hook_put:nnnn
-          { pdf }
-          { link_begin_link_attr }
-          { C }{ [\tl_use:c { @#1bordercolor }] } %do I need to expand?? seems not
-      }
-      {
-        \hook_remove:nnn
-          { pdf }
-          { link_begin_link_attr }
-          { C }
-      }
-    \find@pdflink{#1}{#2}% #1=color type
+    \find@pdflink{link}{#2}% #1=color type (fix), #2=destination name
   }
 
 % the endlink muss have the same type as the start?
@@ -507,21 +492,7 @@
 \def\hyper@link#1#2#3
  {
   \Hy@VerboseLinkStart{#1}{#2}
-  \tl_if_exist:cTF
-   { @#1bordercolor }
-   {
-    \hook_put:nnnn
-     { pdf }
-     { link_begin_link_attr }
-     { C }{ [\tl_use:c { @#1bordercolor }] } %do I need to expand?? seems not
-   }
-   {
-    \hook_remove:nnn
-     { pdf }
-     { link_begin_link_attr }
-     { C }
-   }
-  \find@pdflink{#1}{#2}#3\Hy@xspace@end
+  \find@pdflink{link}{#2}#3\Hy@xspace@end
   \close@EXPpdflink { link }
 }
 \ExplSyntaxOff

diff --git a/pdfresources.dtx b/pdfresources.dtx
@@ -2653,7 +2653,10 @@
 % all packages know which types they have to handle. It is stored in a constant
 % seq.
 % \begin{NOTE}{UF}
-% Perhaps a |cite| type will be useful at some time.
+% Perhaps a |cite| type will be useful at some time. -- Thinking more about it,
+% a |cite| type is not sensible. hyperref supports it, but it doesn't fit in.
+% Commands like cite, gls, acro, footnote and so one should locally change
+% linkcolor and linkbordercolor. 
 % Probably we will need some commands to add an attribute to all link types
 % at once.
 % hyperref commands for the various type: