[jira] [Commented] (FOP-2293) Whitespace management extension

["Seifeddine Dridi (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/FOP-2293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13959864#comment-13959864 ] 

Seifeddine Dridi commented on FOP-2293:
---------------------------------------

I was doubtful that I would be facing this issue but I didn't think it can actually happen. Thanks for pointing it out.

The problem is that the DC is inserted into different pages (page 1 et page 2) which is correct behavior, but later the breaking algorithm chose the first active node for page 1 which breaks before the BFP (node position < BFP position) and assigned it the second variant of the DC.
I think I have to filter out variants when updateData2() is called in such a way that a penalty can only be activated if its position is less than the current node position.

Regarding the name of the extension, I agree with you that the name "best-fit" is a bit misleading and does not accurately describe what the extension is doing. I suggest "first-fit" or "first-fit-within-active-page".

About the name of the penalty, what about MultiSwitchPenalty? The BFP is supposed to be only used by MSLM anyway...

{quote}
Likewise, the value "best-fit" for the auto-toggle option could have a better name. Although it seems that this option is not looked up at all in the code? Does MultiSwitchLM implement it by default?
{quote}

Yes. If auto-toggle was not set, all FO multi-case will be rendered. In such a case, maybe it would be better to fire an exception and tell the user that FO multi-switch is not yet supported?


> Whitespace management extension
> -------------------------------
>
>                 Key: FOP-2293
>                 URL: https://issues.apache.org/jira/browse/FOP-2293
>             Project: Fop
>          Issue Type: New Feature
>          Components: general
>    Affects Versions: trunk
>            Reporter: Seifeddine Dridi
>            Priority: Minor
>              Labels: XSL-FO
>             Fix For: trunk
>
>         Attachments: FO_multi-switch_best-fit_ext_rev10.patch, FO_multi-switch_best-fit_ext_rev11.patch, FO_multi-switch_best-fit_ext_rev2.patch, FO_multi-switch_best-fit_ext_rev3.patch, FO_multi-switch_best-fit_ext_rev4.patch, FO_multi-switch_best-fit_ext_rev5.patch, FO_multi-switch_best-fit_ext_rev6.patch, FO_multi-switch_best-fit_ext_rev7.patch, FO_multi-switch_best-fit_ext_rev8.patch, FO_multi-switch_best-fit_ext_rev9.patch, FO_multi-switch_test.fo, FO_multi-switch_with_best-fit_extension.patch, bestfit.fo, doc.pdf, empty-last-page.fo, multi-switch-testcases.zip, multi-switch_bestfit.fo, multiple-feasible-nodes.fo, patch-rev1.1.patch, patch-rev1.patch, patch-rev2.patch, patch.patch, rev11_bug.fo, two-valid-variants.fo
>
>
> I have been working on an extension for whitespace management, similar to what's described here: http://wiki.apache.org/xmlgraphics-fop/WhitespaceManagement
> The logic of the extension is very simple: the user defines a set of alternatives that he wishes to insert at the end of a page, then if there is enough space left, FOP will pick the alternative that best matches the user's selection criteria (first fit, smallest fit, biggest fit).
> This is my first work on FOP and it took me almost 2 months to reach this stage in development. But it's not the end of course, so I'm relying on your feedback to improve it.
> Thank you



--
This message was sent by Atlassian JIRA
(v6.2#6252)