Your message dated Sun, 27 Apr 2025 12:42:01 +0200
with message-id <10e96b3a-43e9-41d2-85e1-669a8c787...@sury.org>
and subject line Closing old tidy-html5 bugs
has caused the Debian Bug report #831458,
regarding tidy: adds too much vertical space
to be marked as done.
This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.
(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact ow...@bugs.debian.org
immediately.)
--
831458: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=831458
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Package: tidy
Version: 1:5.2.0-1.1
Severity: normal
Hello again,
besides bug #830066, another thing that I noticed after the upgrade
[UPGRADE] tidy:amd64 20091223cvs-1.5 -> 1:5.2.0-1.1
is that tidy started to add too much vertical space.
Please let me explain the steps to reproduce the issue.
Let's start with an HTML document, such as, for instance:
$ wget http://www.inventati.org/frx/progs/scripts/refresh-pubring.html
Please note that this HTML file is already formatted by an older
version of Tidy (as shown by the <meta name="generator"> item).
Hence, I expect that that the re-formatting performed by the current
version of Tidy should not change too much in the file.
Let's set the HTML_TIDY environment variable, in order to work around
bug #830066:
$ export HTML_TIDY=~/.tidyrc
The ~/.tidyrc file is attached and includes:
$ grep vertical-space ~/.tidyrc
vertical-space: yes
Let's start by reformatting the file:
$ tidy refresh-pubring.html > refresh-pubring_T1.html
Tidy adds two lines of vertical space around <h1> and <h2> headers,
between <li> items, around <div> items, around <p> items.
I really think that two lines are too much vertical space: I am
convinced that the behavior of older versions of Tidy was saner
(only one line of vertical space was added).
Please restore the previous behavior.
Moreover each </li> end tag is moved to the next line for no
good reason. Again, I think the behavior of older versions of
Tidy was saner: please fix this regression, too.
On the other hand, I appreciate that the <!-- start tag for
comments is moved to the beginning of a line: I think this is
an improvement with respect to the behavior of older versions.
Please note that disabling the vertical-space option produces an
output HTML file with no vertical space at all (as expected), which
is not what I want:
$ tidy --vertical-space no refresh-pubring.html > refresh-pubring_T2.html
Setting the option to auto produces an almost unreadable HTML code
(as I expected, since the documentation says that this eliminates
nearly all newline characters):
$ tidy --vertical-space auto refresh-pubring.html > refresh-pubring_T3.html
Again, this is not what I want.
In conclusion, the best choice is setting vertical-space to "yes", which
unfortunately adds _too much_ vertical space.
I think this is a regression with respect to the previous versions
of Tidy.
Please fix this regression and/or forward my bug report upstream.
Thanks for your time!
Bye.
-- System Information:
Debian Release: stretch/sid
APT prefers testing
APT policy: (800, 'testing'), (500, 'unstable')
Architecture: amd64 (x86_64)
Kernel: Linux 4.6.0-1-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)
Versions of packages tidy depends on:
ii libc6 2.23-1
ii libtidy5 1:5.2.0-1.1
tidy recommends no packages.
tidy suggests no packages.
-- no debconf information
decorate-inferred-ul: no
doctype: auto
drop-empty-paras: yes
drop-proprietary-attributes: yes
enclose-text: yes
hide-comments: no
logical-emphasis: yes
lower-literals: yes
merge-divs: no
merge-spans: no
numeric-entities: no
output-xhtml: yes
add-xml-decl: yes
quote-ampersand: yes
quote-marks: yes
quote-nbsp: yes
show-body-only: auto
accessibility-check: 0
indent: auto
indent-attributes: no
indent-spaces: 2
markup: yes
wrap: 72
wrap-attributes: no
vertical-space: yes
input-encoding: utf8
output-encoding: utf8
quiet: no
write-back: no
--- End Message ---
--- Begin Message ---
Version: 2:5.8.0-1
I am closing all the bugs that had been filled before 5.8.0 upstream version
that are **upstream** issues.
If the bug that you've reported can still be found in 5.8.0, please retest this,
make sure that the upstream bugs are actually filled upstream and reopen
the bug in Debian BTS and correctly set the "forwarded" attribute on the
issue.
Ondrej
--
Ondřej Surý (He/Him)
ond...@sury.org
--- End Message ---
