Skip to content

Commit

Permalink
Fixes background color
Browse files Browse the repository at this point in the history 
Follow proposed resolution of background color bug tabu-issues-for-future-maintainer#18 (comment)
  • Loading branch information
@meneldal
meneldal committed May 13, 2019
1 parent b45244d commit 2318a9d
Showing 1 changed file with 6 additions and 5 deletions.
11 changes: 6 additions & 5 deletions tabu.sty
View file
Expand Up @@ -858,11 +858,12 @@
%%TABU-FIXED define classz to use vcenter, reverting a change in array package
\let\tabu@classzORI\@classz
\def\tabu@classz{%
\ifx\tabu@prepnext@tok\prepnext@tok% in tabu
\expandafter\tabu@classz@oldarray
\else
\tabu@classzORI
\fi}%
% \ifx\tabu@prepnext@tok\prepnext@tok% in tabu
% \expandafter\tabu@classz@oldarray
% \else
\tabu@classzORI
% \fi
}%
\def\tabu@classz@oldarray{%
\@classx
\@tempcnta \count@
Expand Down

2 comments on commit 2318a9d

@muzimuzhi
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code lines, which I posted in tabu-issues-for-future-maintainer#18 (comment), are far away from "correct fix", but only an "interesting and unintentional discovery". I hardly known anything about why those changes fix the problem, nor do the changes cause new problems.

So, in general, it would be very dangerous to use those changes in any important docs. I strongly recommend not to use it.

@meneldal
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your comment, I am aware it is the wrong fix, it did end up breaking my file.

I only applied the broken patch that so I could have a file to pull for my docker image.

Please sign in to comment.