DO NOT REPLY [Bug 47347] New: [PATCH] auto table layout - yet another patch

View: New views
4 Messages — Rating Filter:   Alert me  

DO NOT REPLY [Bug 47347] New: [PATCH] auto table layout - yet another patch

by Bugzilla from bugzilla@apache.org :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

https://issues.apache.org/bugzilla/show_bug.cgi?id=47347

           Summary: [PATCH] auto table layout - yet another patch
           Product: Fop
           Version: 1.0dev
          Platform: PC
        OS/Version: Windows XP
            Status: NEW
          Severity: normal
          Priority: P2
         Component: page-master/layout
        AssignedTo: fop-dev@...
        ReportedBy: achim.derigs@...
        Depends on: 40271


Hi,

this is yet another patch to enable auto table layout. I've done some work for
one of our customers who needs auto table layout. This patch is doing the job.

I've tried several ways to minimize side effects (and implementation efforts)
as much as possible. Now it hardly affects the existing process.

May be there are cases which are not considered currently. It's certainly an
improvable approach.

Best Regards
Achim

--
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 47347] [PATCH] auto table layout - yet another patch

by Bugzilla from bugzilla@apache.org :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

https://issues.apache.org/bugzilla/show_bug.cgi?id=47347





--- Comment #1 from Achim Derigs <achim.derigs@...>  2009-06-10 07:21:24 PST ---
Created an attachment (id=23787)
 --> (https://issues.apache.org/bugzilla/attachment.cgi?id=23787)
auto-table-layout.diff

--
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 47347] [PATCH] auto table layout - yet another patch

by Bugzilla from bugzilla@apache.org :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

https://issues.apache.org/bugzilla/show_bug.cgi?id=47347


Jens Bannmann <jens.b@...> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jens.b@...




--
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 47347] [PATCH] auto table layout - yet another patch

by Bugzilla from bugzilla@apache.org :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

https://issues.apache.org/bugzilla/show_bug.cgi?id=47347





--- Comment #2 from Vincent Hennebert <vhennebert@...>  2009-06-11 03:54:03 PST ---
Hi Achim,

(In reply to comment #0)

> Hi,
>
> this is yet another patch to enable auto table layout. I've done some work for
> one of our customers who needs auto table layout. This patch is doing the job.
>
> I've tried several ways to minimize side effects (and implementation efforts)
> as much as possible. Now it hardly affects the existing process.
>
> May be there are cases which are not considered currently. It's certainly an
> improvable approach.
>
> Best Regards
> Achim

Thank you very much for your patch. It may prove useful to other users who need
this feature to work on simple tables.

We are not going to apply it to the Trunk, though, as it only cover very simple
cases. For example, it doesn't handle the case of a cell consisting of a long
paragraph that must be broken into lines. Also it doesn't seem to do well with
nested tables.

Still, like I said above some users with only simple cases may be interested in
applying this patch to their local copy.

I hope you understand.
Thanks,
Vincent

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