![[tokkee]](http://tokkee.org/images/avatar.png){kind=avatar}
diff --git a/src/sp-font.cpp b/src/sp-font.cpp
index 9ba4730be10ebbf3d88f6e8238734e6defab7090..b2003cf08cd84eebfd25fd4756ad85b8df21349c 100644 (file)
--- a/src/sp-font.cpp
+++ b/src/sp-font.cpp
#endif
#ifdef ENABLE_SVG_FONTS
-#define __SP_FONT_C__
/*
* SVG <font> element implementation
*
* Author:
- * Felipe C. da S. Sanches <felipe.sanches@gmail.com>
+ * Felipe C. da S. Sanches <juca@members.fsf.org>
*
* Copyright (C) 2008, Felipe C. da S. Sanches
*
#include "sp-glyph.h"
#include "sp-missing-glyph.h"
#include "document.h"
-#include "helper-fns.h"
#include "display/nr-svgfonts.h"
if (!type) {
GTypeInfo info = {
sizeof(SPFontClass),
- NULL, /* base_init */
- NULL, /* base_finalize */
+ NULL, /* base_init */
+ NULL, /* base_finalize */
(GClassInitFunc) sp_font_class_init,
- NULL, /* class_finalize */
- NULL, /* class_data */
+ NULL, /* class_finalize */
+ NULL, /* class_data */
sizeof(SPFont),
- 16, /* n_preallocs */
+ 16, /* n_preallocs */
(GInstanceInitFunc) sp_font_init,
- NULL, /* value_table */
+ NULL, /* value_table */
};
type = g_type_register_static(SP_TYPE_OBJECT, "SPFont", &info, (GTypeFlags) 0);
}
sp_object_class->update = sp_font_update;
}
+//I think we should have extra stuff here and in the set method in order to set default value as specified at http://www.w3.org/TR/SVG/fonts.html
+
+// TODO determine better values and/or make these dynamic:
+double FNT_DEFAULT_ADV = 90; // TODO determine proper default
+double FNT_DEFAULT_ASCENT = 90; // TODO determine proper default
+double FNT_UNITS_PER_EM = 90; // TODO determine proper default
+
static void sp_font_init(SPFont *font)
{
font->horiz_origin_x = 0;
font->horiz_origin_y = 0;
- font->horiz_adv_x = 0;
-//I think we should have extra stuff here and in the set method in order to set default value as specified at http://www.w3.org/TR/SVG/fonts.html
- font->vert_origin_x = 0;
- font->vert_origin_y = 0;
- font->vert_adv_y = 0;
+ font->horiz_adv_x = FNT_DEFAULT_ADV;
+ font->vert_origin_x = FNT_DEFAULT_ADV / 2.0;
+ font->vert_origin_y = FNT_DEFAULT_ASCENT;
+ font->vert_adv_y = FNT_UNITS_PER_EM;
}
static void sp_font_build(SPObject *object, SPDocument *document, Inkscape::XML::Node *repr)
@@ -104,24 +108,12 @@ static void sp_font_build(SPObject *object, SPDocument *document, Inkscape::XML:
sp_object_read_attr(object, "vert-origin-y");
sp_object_read_attr(object, "vert-adv-y");
- SvgFont* svgfont = new SvgFont(SP_FONT(object));
+ sp_document_add_resource(document, "font", object);
}
-static void sp_font_children_modified(SPFont *sp_font)
+static void sp_font_children_modified(SPFont */*sp_font*/)
{
- SPObject* node = sp_font->children;
- for(;node;node=node->next){
- if (SP_IS_GLYPH(node)){
- g_warning("We have a <glyph> childnode:\n\td=%s\n\tvert-origin-x=%f\n\tvert-origin-y=%f\n\tvert-adv-y=%f", ((SPGlyph*)node)->d, ((SPGlyph*)node)->vert_origin_x, ((SPGlyph*)node)->vert_origin_y, ((SPGlyph*)node)->vert_adv_y );
-
- }
- if (SP_IS_MISSING_GLYPH(node)){
-g_warning("We have a <missing-glyph> childnode:\n\td=%s\n\thoriz-origin-x=%f\n\thoriz-origin-y=%f\n\thoriz-adv-x=%f", ((SPMissingGlyph*)node)->d, ((SPMissingGlyph*)node)->vert_origin_x, ((SPMissingGlyph*)node)->vert_origin_y, ((SPMissingGlyph*)node)->vert_adv_y );
- }
-// if (SP_IS_FONT_FACE_SRC(node)){
-// }
- }
}
/**
static void sp_font_release(SPObject *object)
{
//SPFont *font = SP_FONT(object);
+ sp_document_remove_resource(SP_OBJECT_DOCUMENT(object), "font", object);
if (((SPObjectClass *) parent_class)->release) {
((SPObjectClass *) parent_class)->release(object);
static void sp_font_set(SPObject *object, unsigned int key, const gchar *value)
{
SPFont *font = SP_FONT(object);
- double number;
+ // TODO these are floating point, so some epsilon comparison would be good
switch (key) {
- case SP_ATTR_HORIZ_ORIGIN_X:
- number = helperfns_read_number(value);
+ case SP_ATTR_HORIZ_ORIGIN_X:
+ {
+ double number = value ? g_ascii_strtod(value, 0) : 0;
if (number != font->horiz_origin_x){
font->horiz_origin_x = number;
-g_warning("<font>: SP_ATTR_HORIZ_ORIGIN_X: %f", number);
object->requestModified(SP_OBJECT_MODIFIED_FLAG);
}
break;
- case SP_ATTR_HORIZ_ORIGIN_Y:
- number = helperfns_read_number(value);
+ }
+ case SP_ATTR_HORIZ_ORIGIN_Y:
+ {
+ double number = value ? g_ascii_strtod(value, 0) : 0;
if (number != font->horiz_origin_y){
font->horiz_origin_y = number;
-g_warning("<font>: SP_ATTR_HORIZ_ORIGIN_Y: %f", number);
object->requestModified(SP_OBJECT_MODIFIED_FLAG);
}
break;
- case SP_ATTR_HORIZ_ADV_X:
- number = helperfns_read_number(value);
+ }
+ case SP_ATTR_HORIZ_ADV_X:
+ {
+ double number = value ? g_ascii_strtod(value, 0) : FNT_DEFAULT_ADV;
if (number != font->horiz_adv_x){
font->horiz_adv_x = number;
-g_warning("<font>: SP_ATTR_HORIZ_ADV_X: %f", number);
object->requestModified(SP_OBJECT_MODIFIED_FLAG);
}
break;
- case SP_ATTR_VERT_ORIGIN_X:
- number = helperfns_read_number(value);
+ }
+ case SP_ATTR_VERT_ORIGIN_X:
+ {
+ double number = value ? g_ascii_strtod(value, 0) : FNT_DEFAULT_ADV / 2.0;
if (number != font->vert_origin_x){
font->vert_origin_x = number;
-g_warning("<font>: SP_ATTR_VERT_ORIGIN_X: %f", number);
object->requestModified(SP_OBJECT_MODIFIED_FLAG);
}
break;
- case SP_ATTR_VERT_ORIGIN_Y:
- number = helperfns_read_number(value);
+ }
+ case SP_ATTR_VERT_ORIGIN_Y:
+ {
+ double number = value ? g_ascii_strtod(value, 0) : FNT_DEFAULT_ASCENT;
if (number != font->vert_origin_y){
font->vert_origin_y = number;
-g_warning("<font>: SP_ATTR_VERT_ORIGIN_Y: %f", number);
object->requestModified(SP_OBJECT_MODIFIED_FLAG);
}
break;
- case SP_ATTR_VERT_ADV_Y:
- number = helperfns_read_number(value);
+ }
+ case SP_ATTR_VERT_ADV_Y:
+ {
+ double number = value ? g_ascii_strtod(value, 0) : FNT_UNITS_PER_EM;
if (number != font->vert_adv_y){
font->vert_adv_y = number;
-g_warning("<font>: SP_ATTR_VERT_ADV_Y: %f", number);
object->requestModified(SP_OBJECT_MODIFIED_FLAG);
}
break;
- default:
+ }
+ default:
if (((SPObjectClass *) (parent_class))->set) {
((SPObjectClass *) (parent_class))->set(object, key, value);
}