Age | Commit message (Collapse) | Author |
|
Attached is another patch I made for the richtext edit. Now the
red/green/blue/black tags can/must be closed by a correct
red/green/blue/black closing tag.
That means that except for the margin tags, all tags can/must now
be closed with a matching tag.
|
|
1) h1, h2 h3 are now ended by /h1 /h2 /h3
2) <wrap yes> and <wrap no> are changed to <nowrap> </nowrap>
3) The size part of the font tag is now optional.
I suspect a small change is still needed in order to apply
the font size if one is specified anyway. I put a comment
in the text for that.
4) I adapted the sample01.txt so it reflects the 1. and 2. changes.
|
|
Very weird, there was even a code comment saying that it was wrong!! The constructor of
TRichTextSettings already sets the "normal font". So by ignoring the ASettings parameter,
the end-user couldn't assign a new normal font values.
Anyway, this is now fixed.
|
|
We checked for the same fontdesc, but never for the same instance. So the NewFont
instance might not have been freed, causing the memory leak.
|
|
This gives an overall improved margin size. In future we should make this
user selectable in Docview's Preferences dialog.
|
|
- Rename Settings parameter to ASettings. This is to prevent possible
confusion with the Settings global variable.
- ApplyStyleTag had many issues, applying the style changes to the
wrong properties. Manyly, styling was applied to the ASettings parameter
when they should have been applied to the Style parameter.
- Code formatting improvements.
|
|
Settings is a global variable for RichView component. I renamed the
parameter Settings to ASettings so as to prevent any possible confusion
between the two.
|
|
|
|
The global variable DefaultTopicFont already contained the font size, so
we didn't need to append it.
|
|
The CanvasFontManager unit has totally been reworked to use the native
TfpgFont class instead of TLogicalFont, TFontFace etc... The code is
a lot more simplified now and DocView can actually display content
but there are some width wrapping issues and AV's on changing topics.
This will be fixed next.
|
|
A reference to a default font or fixed font are now the same
throughout docview. If you change one, you change them all.
|
|
|
|
Not sure when it became the case but it makes sense that var parameters
must use the exact type and not a descendant class. This patch puts that
into effect.
|
|
|
|
|