[FFmpeg-devel] [PATCH v2] doc/t2h: Support texinfo 7.0

Stefano Sabatini stefasab at gmail.com
Wed Nov 8 02:15:17 EET 2023


On date Tuesday 2023-11-07 23:32:23 +0100, epirat07 at gmail.com wrote:
> On 7 Nov 2023, at 22:44, Frank Plowman wrote:
> 
> > On 07/11/2023 20:38, epirat07 at gmail.com wrote:
[...]
> >> Just had a quick look and was wondering why your patch touches the
> >> bootstrap.min.css?
> >

> > Texinfo 7.0 produces quite different HTML to Texinfo 6.8. Without
> > that change to the CSS, enumerated option flags (i.e. Possible
> > values of x are...) render as white text on a white background
> > with Texinfo 7.0 and are unreadable. The change removes a style
> > for the selector `.table .table` which causes the background to
> > turn white. As far as I can tell, it is not actually used anywhere
> > in files generated by Texinfo 6.8.
> >
> 

> Thanks for the clarification, I think it might be a good idea to split this change
> into a second commit with the explanation you gave in the commit message. That way
> if it ever causes an issue, it is easier to figure out, as this minified css is
> nearly unreviewable…

+1

That said, I tested the latest patch and seems to work fine with
texinfo 6.8.


More information about the ffmpeg-devel mailing list