[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [lmi] wxHTML table header alignment
From: |
Vaclav Slavik |
Subject: |
Re: [lmi] wxHTML table header alignment |
Date: |
Wed, 11 Mar 2009 14:28:01 +0100 |
On Wed, 2009-03-11 at 08:48 +0000, Greg Chicares wrote:
> The more I consider this patch, the more I like it. Yet it'd be
> good to hear your thoughts before I commit it, because I might be
> missing some fragility.
It's a hack that takes advantage of wxHTML's implementation details, but
I think it's OK as long as you're willing to take the (low) risk that
the hack could stop working in the future in case of some wxHTML parser
improvements. But any change that would make the parser ignore this
trailing space is likely to ignore such whitespace on all lines (and so
fix this bug too). In other words, I think it's safe enough.
Vaclav
- Re[2]: [lmi] wxHTML table header alignment, Vadim Zeitlin, 2009/03/09
- Re: [lmi] wxHTML table header alignment, Greg Chicares, 2009/03/11
- Re[2]: [lmi] wxHTML table header alignment, Vadim Zeitlin, 2009/03/11
- Re: [lmi] wxHTML table header alignment, Greg Chicares, 2009/03/11
- Re[2]: [lmi] wxHTML table header alignment, Vadim Zeitlin, 2009/03/11
- [lmi] Complete replacement of help engine [Re: wxHTML table header alignment], Greg Chicares, 2009/03/12
- Re: [lmi] Complete replacement of help engine [Re: wxHTML table header alignment], Wendy Boutin, 2009/03/12
- Re: [lmi] Complete replacement of help engine [Re: wxHTML table header alignment], Greg Chicares, 2009/03/12
- Re: [lmi] Complete replacement of help engine [Re: wxHTML table header alignment], Wendy Boutin, 2009/03/18
- Re: [lmi] Complete replacement of help engine [Re: wxHTML table header alignment], Greg Chicares, 2009/03/18