DO NOT REPLY [Bug 49801] Region-Body Column balancing incorrect if content is table with header

2012-04-23 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49801

Alex Giotis alex.gio...@gmail.com changed:

   What|Removed |Added

 CC||alex.gio...@gmail.com

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.


DO NOT REPLY [Bug 49801] Region-Body Column balancing incorrect if content is table with header

2012-04-23 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49801

--- Comment #11 from Alex Giotis alex.gio...@gmail.com 2012-04-23 22:49:00 
UTC ---
Even though this is not explicitly specified in XSL-FO 1.X, columns are
expected to be balanced. For our documents, unbalanced columns, are not an
acceptable output. Unfortunately, we were not able to find a solution. I can
only attach a small patch of a dirty hack that gives the desired result for the
initially attached test file, has been working well for us for the last 2 years
and (of course) passes all the unit tests.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.


DO NOT REPLY [Bug 49801] Region-Body Column balancing incorrect if content is table with header

2012-04-23 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49801

--- Comment #13 from Alex Giotis alex.gio...@gmail.com 2012-04-23 22:53:48 
UTC ---
Created attachment 28663
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=28663
Current output PDF

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.


DO NOT REPLY [Bug 49801] Region-Body Column balancing incorrect if content is table with header

2012-04-23 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49801

--- Comment #14 from Alex Giotis alex.gio...@gmail.com 2012-04-23 22:54:46 
UTC ---
Created attachment 28664
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=28664
Expected output PDF (after patch)

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.


DO NOT REPLY [Bug 49801] Region-Body Column balancing incorrect if content is table with header

2012-04-23 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49801

--- Comment #15 from Alex Giotis alex.gio...@gmail.com 2012-04-23 23:00:30 
UTC ---
The patch is against revision 1329104 of trunk.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.


DO NOT REPLY [Bug 49801] Region-Body Column balancing incorrect if content is table with header

2012-04-20 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49801

Glenn Adams gad...@apache.org changed:

   What|Removed |Added

   Priority|P3  |P5
   Severity|normal  |enhancement

--- Comment #10 from Glenn Adams gad...@apache.org 2012-04-21 05:33:07 UTC ---
lowering priority and changing to enhancement since (1) a fix is likely to
require a fairly significant design change and (2) column balancing is a
(potential) XSL-FO 2.0 feature [1], not an XSL-FO 1.X feature;

[1] http://www.w3.org/TR/xslfo20-req/#N66487

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.


DO NOT REPLY [Bug 49801] Region-Body Column balancing incorrect if content is table with header

2012-04-06 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49801

Glenn Adams gl...@skynav.com changed:

   What|Removed |Added

   Severity|major   |normal

--- Comment #8 from Glenn Adams gl...@skynav.com 2012-04-07 01:37:32 UTC ---
resetting severity from major to normal pending further review

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.


DO NOT REPLY [Bug 49801] Region-Body Column balancing incorrect if content is table with header

2012-04-06 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49801

--- Comment #9 from Glenn Adams gl...@skynav.com 2012-04-07 01:42:15 UTC ---
resetting P2 open bugs to P3 pending further review

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.


DO NOT REPLY [Bug 49801] Region-Body Column balancing incorrect if content is table with header

2011-03-21 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49801

--- Comment #7 from Andreas L. Delmelle adelme...@apache.org 2011-03-21 
15:27:29 EDT ---

Thinking more about a possible fix, I am suddenly asking myself why we would
even fiddle with the demerits at all to balance columns. Playing with the
demerits alone is not enough to make a break more/less favorable. Ultimately,
the adjustment ratio is a deciding factor too, and that is based on the
difference between available width and the width of the line.

Since the algorithm works with the full bpd of the page as line-width, this is
bound to cause the algorithm to 'ignore' the better demerits at some points.

In a way, that feels like we could handle column-balancing better by (also)
forcing the available line-width down to the minimal width needed by the
content (i.e. taking into account pieces that must be kept together within the
same column).

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.


DO NOT REPLY [Bug 49801] Region-Body Column balancing incorrect if content is table with header

2011-02-22 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49801

alpa...@gmail.com changed:

   What|Removed |Added

 CC||alpa...@gmail.com

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.


DO NOT REPLY [Bug 49801] Region-Body Column balancing incorrect if content is table with header

2011-02-08 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49801

--- Comment #6 from Andreas L. Delmelle adelme...@apache.org 2011-02-08 
15:19:45 EST ---

Just thought of similar potential issues, due to the limitations of
ElementListUtils.calcContentLength(), but unrelated to column-balancing per se,
that are to be expected in list-items and table-cells.
More precisely those that have a table with header and/or footer as a
descendant, and are divided over more than one column/page.
For lists, problems may arise in
ListItemLayoutManager.getCombinedKnuthElementsForListItem(), I think, where the
calculated lengths could prove to be wrong (due to the content eventually being
broken at least once).
For table-cells, similarly, the totalLength in the ActiveCell constructor might
be too low if the cell contains a long nested table that causes multiple
breaks...

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.


DO NOT REPLY [Bug 49801] Region-Body Column balancing incorrect if content is table with header

2011-02-04 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49801

--- Comment #4 from Andreas L. Delmelle adelme...@apache.org 2011-02-04 
13:18:28 EST ---
Created an attachment (id=26606)
 -- (https://issues.apache.org/bugzilla/attachment.cgi?id=26606)
altered test case

The fullLen in computeDemerits() is indeed off, because
ElementListUtils.calcContentLength() naively sums box and glue lengths. It just
counts the header box once, and does not contain any logic to take into account
possible special behavior of the penalties. Adding the header box multiple
times to the list will not immediately solve the issue.
Question that needs to be answered in order to do that: how many times would it
need to be duplicated? Answer: the number of times the table is broken plus 1.
But we don't know that number, at that point... That is precisely the reason
the table-layout code only adds the box once.

See also the attached altered test case for a slightly more complicated
example. The first and last table's headers only need to be counted once, while
the header and footer of the second table need to be counted --how many times?
The algorithm currently only breaks once.

Seems like the chicken and the egg... :-/

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.


DO NOT REPLY [Bug 49801] Region-Body Column balancing incorrect if content is table with header

2011-02-04 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49801

--- Comment #5 from Andreas L. Delmelle adelme...@apache.org 2011-02-04 
13:27:07 EST ---

Additionally, it's not only fullLen which is off, but obviously also partLen
and restLen, for the same reason (calcContentLength()). There will be cases
where the header will have to be taken into account for partLen, but not for
restLen or vice versa, and cases where it will need to be counted for both.

Currently, the algorithm operates as if the header and footer only count for
the part where the table happens to end.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.


DO NOT REPLY [Bug 49801] Region-Body Column balancing incorrect if content is table with header

2010-09-06 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49801

--- Comment #3 from Jeremias Maerki jerem...@apache.org 2010-09-06 09:46:13 
EDT ---
I've been asked to look closer into this, and after about 10 hours spent
brooding over it and consulting the Digital Typography book I'm no closer to
solving it. I've been able to calculate the demerits which I expected to result
in the right break decision (for one case), but the algorithm still chose
another breaking point for reasons I just don't understand. The
BalancingColumnBreakingAlgorithm class was some kind of hack to begin with
which I knew had some short-comings. That is now pretty obvious, and I don't
know how to proceed. I have to give up for now since this is simply a bit too
high for me. I'll copy my local changes away so if anyone wants to give it a
shot, I can provide them. But I'm not sure if they would really help.

If anyone can help Adam, please get in touch with him.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.


DO NOT REPLY [Bug 49801] Region-Body Column balancing incorrect if content is table with header

2010-08-23 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49801

--- Comment #1 from a.kov...@i-docs.com 2010-08-23 06:05:19 EDT ---
Created an attachment (id=25928)
 -- (https://issues.apache.org/bugzilla/attachment.cgi?id=25928)
test file

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.