r19173 - gnucash/trunk/src/report - Merge CSS and plain stylesheets, leaving the plain and remove the CSS stylesheet. Plain now has font specifications.

[J. Alex Aycinena]

Phil,

I traced the changes to the Tax report through the various revisions
and determined that it was r19083 where the behaviour changed. This is
when the default was changed from 'Default' to 'Default-CSS'. The
report looked OK when I switched back to 'Default' in the GUI after
creating the report under both webkit and gtkhtml. Now that the two
have been merged, the situation is this:

Under gtkhtml, I had to make a minor change to taxtxf.scm to get it to
display properly. It seems that the "number-cell" style included in
stylesheet-plain is not right-justified. I think it should be. I was
able to define an override style in taxtxf.scm that was used when the
report is generated with gtkhtml to get it to display correctly. But
if the "number-cell" style included in stylesheet-plain were changed
to be right-justified, no change would be need to taxtxf.scm.

Under webkit, however, the report doesn't display correctly because
webkit doesn't seem to honor any styles defined in the report. If I
reference a style included in stylesheet-plain, that works, but if a
needed style is not in stylesheet-plain, the report can't be displayed
correctly.

Shouldn't webkit and gtkhtml work similarly in this respect?

Alex

> From: "J. Alex Aycinena" <alex.aycinena at gmail.com>
> To: gnucash-devel at gnucash.org, plongstaff at code.gnucash.org
> Date: Thu, 20 May 2010 17:11:03 -0700
> Subject: Re: r19173 - gnucash/trunk/src/report - Merge CSS and plain stylesheets, leaving the plain and remove the CSS stylesheet. Plain now has font specifications.
> Phil,
>
> I don't know if it's this change, or one of the other recent changes
> related to reporting, but the display of the new Tax Schedule Report
> has recently changed and the justification of amounts and descriptions
> isn't right. It looks better when gnucash is built with gtkhtml than
> webkit, but both are not right. Can you give me some clues as to how I
> can go about adjusting the report to get it right given the changes
> you made?
>
> Thanks,
>
> Alex
>
> On Sat, May 15, 2010 at 8:04 AM, Phil Longstaff
> <plongstaff at code.gnucash.org> wrote:
>> Author: plongstaff
>> Date: 2010-05-15 11:04:54 -0400 (Sat, 15 May 2010)
>> New Revision: 19173
>> Trac: http://svn.gnucash.org/trac/changeset/19173
>>
>> Removed:
>>   gnucash/trunk/src/report/stylesheets/stylesheet-css.scm
>> Modified:
>>   gnucash/trunk/src/report/report-system/report.scm
>>   gnucash/trunk/src/report/stylesheets/Makefile.am
>>   gnucash/trunk/src/report/stylesheets/stylesheet-plain.scm
>>   gnucash/trunk/src/report/stylesheets/stylesheets.scm
>> Log:
>> Merge CSS and plain stylesheets, leaving the plain and remove the CSS stylesheet.  Plain now has font specifications.
>>
>>