X-Git-Url: https://git.tokkee.org/?a=blobdiff_plain;f=src%2Ftext-editing.cpp;h=5b9db13d4532a29c2926344b9f5e0322133afa45;hb=530f7fc7cfa4351ec33163abd795460bbc351609;hp=f0cb4bd9da12967f29a32a30863485bc1154a367;hpb=9b6c50478ce8c03e775b9e77204377d4a8cc2b41;p=inkscape.git diff --git a/src/text-editing.cpp b/src/text-editing.cpp index f0cb4bd9d..5b9db13d4 100644 --- a/src/text-editing.cpp +++ b/src/text-editing.cpp @@ -14,10 +14,17 @@ # include "config.h" #endif +#include +#include +#include + #include "desktop.h" +#include "inkscape.h" +#include "message-stack.h" #include "style.h" #include "unit-constants.h" +#include "document.h" #include "xml/repr.h" #include "xml/attribute-record.h" @@ -25,10 +32,13 @@ #include "sp-flowtext.h" #include "sp-flowdiv.h" #include "sp-flowregion.h" +#include "sp-tref.h" #include "sp-tspan.h" #include "text-editing.h" +static const gchar *tref_edit_message = _("You cannot edit cloned character data."); + static bool tidy_xml_tree_recursively(SPObject *root); Inkscape::Text::Layout const * te_get_layout (SPItem const *item) @@ -68,47 +78,57 @@ sp_te_input_is_empty (SPObject const *item) } Inkscape::Text::Layout::iterator -sp_te_get_position_by_coords (SPItem const *item, NR::Point &i_p) +sp_te_get_position_by_coords (SPItem const *item, Geom::Point const &i_p) { - NR::Matrix im=sp_item_i2d_affine (item); + Geom::Matrix im (sp_item_i2d_affine (item)); im = im.inverse(); - NR::Point p = i_p * im; + Geom::Point p = i_p * im; Inkscape::Text::Layout const *layout = te_get_layout(item); return layout->getNearestCursorPositionTo(p); } -std::vector sp_te_create_selection_quads(SPItem const *item, Inkscape::Text::Layout::iterator const &start, Inkscape::Text::Layout::iterator const &end, NR::Matrix const &transform) +std::vector sp_te_create_selection_quads(SPItem const *item, Inkscape::Text::Layout::iterator const &start, Inkscape::Text::Layout::iterator const &end, Geom::Matrix const &transform) { if (start == end) - return std::vector(); + return std::vector(); Inkscape::Text::Layout const *layout = te_get_layout(item); if (layout == NULL) - return std::vector(); + return std::vector(); return layout->createSelectionShape(start, end, transform); } void -sp_te_get_cursor_coords (SPItem const *item, Inkscape::Text::Layout::iterator const &position, NR::Point &p0, NR::Point &p1) +sp_te_get_cursor_coords (SPItem const *item, Inkscape::Text::Layout::iterator const &position, Geom::Point &p0, Geom::Point &p1) { Inkscape::Text::Layout const *layout = te_get_layout(item); double height, rotation; - layout->queryCursorShape(position, &p0, &height, &rotation); - p1 = NR::Point(p0[NR::X] + height * sin(rotation), p0[NR::Y] - height * cos(rotation)); + layout->queryCursorShape(position, p0, height, rotation); + p1 = Geom::Point(p0[Geom::X] + height * sin(rotation), p0[Geom::Y] - height * cos(rotation)); } SPStyle const * sp_te_style_at_position(SPItem const *text, Inkscape::Text::Layout::iterator const &position) +{ + SPObject const *pos_obj = sp_te_object_at_position(text, position); + if (pos_obj) + return SP_OBJECT_STYLE(pos_obj); + return NULL; +} + +SPObject const * sp_te_object_at_position(SPItem const *text, Inkscape::Text::Layout::iterator const &position) { Inkscape::Text::Layout const *layout = te_get_layout(text); if (layout == NULL) return NULL; - SPObject const *pos_obj = NULL; - layout->getSourceOfCharacter(position, (void**)&pos_obj); - if (pos_obj == NULL) pos_obj = text; + SPObject const *pos_obj = 0; + void *rawptr = 0; + layout->getSourceOfCharacter(position, &rawptr); + pos_obj = SP_OBJECT(rawptr); + if (pos_obj == 0) pos_obj = text; while (SP_OBJECT_STYLE(pos_obj) == NULL) - pos_obj = SP_OBJECT_PARENT(pos_obj); // SPStrings don't have style - return SP_OBJECT_STYLE(pos_obj); + pos_obj = SP_OBJECT_PARENT(pos_obj); // not interested in SPStrings + return pos_obj; } /* @@ -128,8 +148,9 @@ char * dump_hexy(const gchar * utf8) Inkscape::Text::Layout::iterator sp_te_replace(SPItem *item, Inkscape::Text::Layout::iterator const &start, Inkscape::Text::Layout::iterator const &end, gchar const *utf8) { - Inkscape::Text::Layout::iterator new_start = sp_te_delete(item, start, end); - return sp_te_insert(item, new_start, utf8); + iterator_pair pair; + sp_te_delete(item, start, end, pair); + return sp_te_insert(item, pair.first, utf8); } @@ -138,23 +159,34 @@ Inkscape::Text::Layout::iterator sp_te_replace(SPItem *item, Inkscape::Text::Lay static bool is_line_break_object(SPObject const *object) { - return SP_IS_TEXT(object) - || (SP_IS_TSPAN(object) && SP_TSPAN(object)->role != SP_TSPAN_ROLE_UNSPECIFIED) - || SP_IS_TEXTPATH(object) - || SP_IS_FLOWDIV(object) - || SP_IS_FLOWPARA(object) - || SP_IS_FLOWLINE(object) - || SP_IS_FLOWREGIONBREAK(object); + bool is_line_break = false; + + if (object) { + if (SP_IS_TEXT(object) + || (SP_IS_TSPAN(object) && SP_TSPAN(object)->role != SP_TSPAN_ROLE_UNSPECIFIED) + || SP_IS_TEXTPATH(object) + || SP_IS_FLOWDIV(object) + || SP_IS_FLOWPARA(object) + || SP_IS_FLOWLINE(object) + || SP_IS_FLOWREGIONBREAK(object)) { + + is_line_break = true; + } + } + + return is_line_break; } /** returns the attributes for an object, or NULL if it isn't a text, -tspan or textpath. */ +tspan, tref, or textpath. */ static TextTagAttributes* attributes_for_object(SPObject *object) { if (SP_IS_TSPAN(object)) return &SP_TSPAN(object)->attributes; if (SP_IS_TEXT(object)) return &SP_TEXT(object)->attributes; + if (SP_IS_TREF(object)) + return &SP_TREF(object)->attributes; if (SP_IS_TEXTPATH(object)) return &SP_TEXTPATH(object)->attributes; return NULL; @@ -174,7 +206,9 @@ unsigned sp_text_get_length(SPObject const *item) unsigned length = 0; if (SP_IS_STRING(item)) return SP_STRING(item)->string.length(); + if (is_line_break_object(item)) length++; + for (SPObject const *child = item->firstChild() ; child ; child = SP_OBJECT_NEXT(child)) { if (SP_IS_STRING(child)) length += SP_STRING(child)->string.length(); else length += sp_text_get_length(child); @@ -188,15 +222,20 @@ unsigned sp_text_get_length_upto(SPObject const *item, SPObject const *upto) { unsigned length = 0; + // The string is the lowest level and the length can be counted directly. if (SP_IS_STRING(item)) { return SP_STRING(item)->string.length(); } + + // Take care of new lines... if (is_line_break_object(item) && !SP_IS_TEXT(item)) { if (item != SP_OBJECT_PARENT(item)->firstChild()) { // add 1 for each newline length++; } } + + // Count the length of the children for (SPObject const *child = item->firstChild() ; child ; child = SP_OBJECT_NEXT(child)) { if (upto && child == upto) { // hit upto, return immediately @@ -219,11 +258,11 @@ unsigned sp_text_get_length_upto(SPObject const *item, SPObject const *upto) return length; } -static Inkscape::XML::Node* duplicate_node_without_children(Inkscape::XML::Node const *old_node) +static Inkscape::XML::Node* duplicate_node_without_children(Inkscape::XML::Document *xml_doc, Inkscape::XML::Node const *old_node) { switch (old_node->type()) { case Inkscape::XML::ELEMENT_NODE: { - Inkscape::XML::Node *new_node = sp_repr_new(old_node->name()); + Inkscape::XML::Node *new_node = xml_doc->createElement(old_node->name()); Inkscape::Util::List attributes = old_node->attributeList(); GQuark const id_key = g_quark_from_string("id"); for ( ; attributes ; attributes++) { @@ -234,10 +273,13 @@ static Inkscape::XML::Node* duplicate_node_without_children(Inkscape::XML::Node } case Inkscape::XML::TEXT_NODE: - return sp_repr_new_text(old_node->content()); + return xml_doc->createTextNode(old_node->content()); case Inkscape::XML::COMMENT_NODE: - return sp_repr_new_comment(old_node->content()); + return xml_doc->createComment(old_node->content()); + + case Inkscape::XML::PI_NODE: + return xml_doc->createPI(old_node->name(), old_node->content()); case Inkscape::XML::DOCUMENT_NODE: return NULL; // this had better never happen @@ -273,8 +315,9 @@ parent of the first line break node encountered. */ static SPObject* split_text_object_tree_at(SPObject *split_obj, unsigned char_index) { + Inkscape::XML::Document *xml_doc = sp_document_repr_doc(SP_OBJECT_DOCUMENT(split_obj)); if (is_line_break_object(split_obj)) { - Inkscape::XML::Node *new_node = duplicate_node_without_children(SP_OBJECT_REPR(split_obj)); + Inkscape::XML::Node *new_node = duplicate_node_without_children(xml_doc, SP_OBJECT_REPR(split_obj)); SP_OBJECT_REPR(SP_OBJECT_PARENT(split_obj))->addChild(new_node, SP_OBJECT_REPR(split_obj)); Inkscape::GC::release(new_node); split_attributes(split_obj, SP_OBJECT_NEXT(split_obj), char_index); @@ -284,7 +327,7 @@ static SPObject* split_text_object_tree_at(SPObject *split_obj, unsigned char_in unsigned char_count_before = sum_sibling_text_lengths_before(split_obj); SPObject *duplicate_obj = split_text_object_tree_at(SP_OBJECT_PARENT(split_obj), char_index + char_count_before); // copy the split node - Inkscape::XML::Node *new_node = duplicate_node_without_children(SP_OBJECT_REPR(split_obj)); + Inkscape::XML::Node *new_node = duplicate_node_without_children(xml_doc, SP_OBJECT_REPR(split_obj)); SP_OBJECT_REPR(duplicate_obj)->appendChild(new_node); Inkscape::GC::release(new_node); @@ -314,25 +357,41 @@ Inkscape::Text::Layout::iterator sp_te_insert_line (SPItem *item, Inkscape::Text { // Disable newlines in a textpath; TODO: maybe on Enter in a textpath, separate it into two // texpaths attached to the same path, with a vertical shift - if (SP_IS_TEXT_TEXTPATH (item)) + if (SP_IS_TEXT_TEXTPATH (item) || SP_IS_TREF(item)) return position; + + SPDesktop *desktop = SP_ACTIVE_DESKTOP; Inkscape::Text::Layout const *layout = te_get_layout(item); - SPObject *split_obj; + SPObject *split_obj = 0; Glib::ustring::iterator split_text_iter; - if (position == layout->end()) - split_obj = NULL; - else - layout->getSourceOfCharacter(position, (void**)&split_obj, &split_text_iter); + if (position != layout->end()) { + void *rawptr = 0; + layout->getSourceOfCharacter(position, &rawptr, &split_text_iter); + split_obj = SP_OBJECT(rawptr); + } - if (split_obj == NULL || is_line_break_object(split_obj)) { - if (split_obj == NULL) split_obj = item->lastChild(); + if (split_obj == 0 || is_line_break_object(split_obj)) { + if (split_obj == 0) split_obj = item->lastChild(); + + if (SP_IS_TREF(split_obj)) { + desktop->messageStack()->flash(Inkscape::ERROR_MESSAGE, tref_edit_message); + return position; + } + if (split_obj) { - Inkscape::XML::Node *new_node = duplicate_node_without_children(SP_OBJECT_REPR(split_obj)); + Inkscape::XML::Document *xml_doc = sp_document_repr_doc(SP_OBJECT_DOCUMENT(split_obj)); + Inkscape::XML::Node *new_node = duplicate_node_without_children(xml_doc, SP_OBJECT_REPR(split_obj)); SP_OBJECT_REPR(SP_OBJECT_PARENT(split_obj))->addChild(new_node, SP_OBJECT_REPR(split_obj)); Inkscape::GC::release(new_node); } } else if (SP_IS_STRING(split_obj)) { + // If the parent is a tref, editing on this particular string is disallowed. + if (SP_IS_TREF(SP_OBJECT_PARENT(split_obj))) { + desktop->messageStack()->flash(Inkscape::ERROR_MESSAGE, tref_edit_message); + return position; + } + Glib::ustring *string = &SP_STRING(split_obj)->string; unsigned char_index = 0; for (Glib::ustring::iterator it = string->begin() ; it != split_text_iter ; it++) @@ -347,7 +406,7 @@ Inkscape::Text::Layout::iterator sp_te_insert_line (SPItem *item, Inkscape::Text // TODO // I think the only case to put here is arbitrary gaps, which nobody uses yet } - item->updateRepr(SP_OBJECT_REPR(item),SP_OBJECT_WRITE_EXT); + item->updateRepr(); unsigned char_index = layout->iteratorToCharIndex(position); te_update_layout_now(item); item->requestDisplayUpdate(SP_OBJECT_MODIFIED_FLAG); @@ -402,23 +461,34 @@ sp_te_insert(SPItem *item, Inkscape::Text::Layout::iterator const &position, gch g_warning("Trying to insert invalid utf8"); return position; } + + SPDesktop *desktop = SP_ACTIVE_DESKTOP; Inkscape::Text::Layout const *layout = te_get_layout(item); - SPObject *source_obj; + SPObject *source_obj = 0; + void *rawptr = 0; Glib::ustring::iterator iter_text; // we want to insert after the previous char, not before the current char. // it makes a difference at span boundaries Inkscape::Text::Layout::iterator it_prev_char = position; bool cursor_at_start = !it_prev_char.prevCharacter(); bool cursor_at_end = position == layout->end(); - layout->getSourceOfCharacter(it_prev_char, (void**)&source_obj, &iter_text); + layout->getSourceOfCharacter(it_prev_char, &rawptr, &iter_text); + source_obj = SP_OBJECT(rawptr); if (SP_IS_STRING(source_obj)) { - // the simple case + // If the parent is a tref, editing on this particular string is disallowed. + if (SP_IS_TREF(SP_OBJECT_PARENT(source_obj))) { + desktop->messageStack()->flash(Inkscape::ERROR_MESSAGE, tref_edit_message); + return position; + } + + // Now the simple case can begin... if (!cursor_at_start) iter_text++; SPString *string_item = SP_STRING(source_obj); insert_into_spstring(string_item, cursor_at_end ? string_item->string.end() : iter_text, utf8); } else { // the not-so-simple case where we're at a line break or other control char; add to the next child/sibling SPString + Inkscape::XML::Document *xml_doc = SP_OBJECT_REPR(item)->document(); if (cursor_at_start) { source_obj = item; if (source_obj->hasChildren()) { @@ -431,7 +501,7 @@ sp_te_insert(SPItem *item, Inkscape::Text::Layout::iterator const &position, gch } } if (source_obj == item && SP_IS_FLOWTEXT(item)) { - Inkscape::XML::Node *para = sp_repr_new("svg:flowPara"); + Inkscape::XML::Node *para = xml_doc->createElement("svg:flowPara"); SP_OBJECT_REPR(item)->appendChild(para); source_obj = item->lastChild(); } @@ -442,17 +512,23 @@ sp_te_insert(SPItem *item, Inkscape::Text::Layout::iterator const &position, gch SPString *string_item = sp_te_seek_next_string_recursive(source_obj); if (string_item == NULL) { // need to add an SPString in this (pathological) case - Inkscape::XML::Node *rstring = sp_repr_new_text(""); + Inkscape::XML::Node *rstring = xml_doc->createTextNode(""); SP_OBJECT_REPR(source_obj)->addChild(rstring, NULL); Inkscape::GC::release(rstring); g_assert(SP_IS_STRING(source_obj->firstChild())); string_item = SP_STRING(source_obj->firstChild()); } + // If the parent is a tref, editing on this particular string is disallowed. + if (SP_IS_TREF(SP_OBJECT_PARENT(string_item))) { + desktop->messageStack()->flash(Inkscape::ERROR_MESSAGE, tref_edit_message); + return position; + } + insert_into_spstring(string_item, cursor_at_end ? string_item->string.end() : string_item->string.begin(), utf8); } } - item->updateRepr(SP_OBJECT_REPR(item),SP_OBJECT_WRITE_EXT); + item->updateRepr(); unsigned char_index = layout->iteratorToCharIndex(position); te_update_layout_now(item); item->requestDisplayUpdate(SP_OBJECT_MODIFIED_FLAG); @@ -519,7 +595,8 @@ static SPObject* delete_line_break(SPObject *root, SPObject *item, bool *next_is

*text

*text

*/ - Inkscape::XML::Node *new_span_repr = sp_repr_new(span_name_for_text_object(root)); + Inkscape::XML::Document *xml_doc = SP_OBJECT_REPR(item)->document(); + Inkscape::XML::Node *new_span_repr = xml_doc->createElement(span_name_for_text_object(root)); if (gchar const *a = this_repr->attribute("dx")) new_span_repr->setAttribute("dx", a); @@ -609,30 +686,38 @@ static void erase_from_spstring(SPString *string_item, Glib::ustring::iterator i quite a complicated operation, partly due to the cleanup that is done if all the text in a subobject has been deleted, and partly due to the difficulty of figuring out what is a line break and how to delete one. Returns the -lesser of \a start and \a end, because that is where the cursor should be -put after the deletion is done. */ -Inkscape::Text::Layout::iterator -sp_te_delete (SPItem *item, Inkscape::Text::Layout::iterator const &start, Inkscape::Text::Layout::iterator const &end) +real start and ending iterators based on the situation. */ +bool +sp_te_delete (SPItem *item, Inkscape::Text::Layout::iterator const &start, + Inkscape::Text::Layout::iterator const &end, iterator_pair &iter_pair) { - if (start == end) return start; - Inkscape::Text::Layout::iterator first, last; - if (start < end) { - first = start; - last = end; - } else { - first = end; - last = start; + bool success = false; + + iter_pair.first = start; + iter_pair.second = end; + + if (start == end) return success; + + if (start > end) { + iter_pair.first = end; + iter_pair.second = start; } + + SPDesktop *desktop = SP_ACTIVE_DESKTOP; + Inkscape::Text::Layout const *layout = te_get_layout(item); - SPObject *start_item, *end_item; + SPObject *start_item = 0, *end_item = 0; + void *rawptr = 0; Glib::ustring::iterator start_text_iter, end_text_iter; - layout->getSourceOfCharacter(first, (void**)&start_item, &start_text_iter); - layout->getSourceOfCharacter(last, (void**)&end_item, &end_text_iter); - if (start_item == NULL) - return first; // start is at end of text + layout->getSourceOfCharacter(iter_pair.first, &rawptr, &start_text_iter); + start_item = SP_OBJECT(rawptr); + layout->getSourceOfCharacter(iter_pair.second, &rawptr, &end_text_iter); + end_item = SP_OBJECT(rawptr); + if (start_item == 0) + return success; // start is at end of text if (is_line_break_object(start_item)) move_to_end_of_paragraph(&start_item, &start_text_iter); - if (end_item == NULL) { + if (end_item == 0) { end_item = item->lastChild(); move_to_end_of_paragraph(&end_item, &end_text_iter); } @@ -644,7 +729,13 @@ sp_te_delete (SPItem *item, Inkscape::Text::Layout::iterator const &start, Inksc if (start_item == end_item) { // the quick case where we're deleting stuff all from the same string if (SP_IS_STRING(start_item)) { // always true (if it_start != it_end anyway) - erase_from_spstring(SP_STRING(start_item), start_text_iter, end_text_iter); + // If the parent is a tref, editing on this particular string is disallowed. + if (SP_IS_TREF(SP_OBJECT_PARENT(start_item))) { + desktop->messageStack()->flash(Inkscape::ERROR_MESSAGE, tref_edit_message); + } else { + erase_from_spstring(SP_STRING(start_item), start_text_iter, end_text_iter); + success = true; + } } } else { SPObject *sub_item = start_item; @@ -652,8 +743,15 @@ sp_te_delete (SPItem *item, Inkscape::Text::Layout::iterator const &start, Inksc while (sub_item != item) { if (sub_item == end_item) { if (SP_IS_STRING(sub_item)) { + // If the parent is a tref, editing on this particular string is disallowed. + if (SP_IS_TREF(SP_OBJECT_PARENT(sub_item))) { + desktop->messageStack()->flash(Inkscape::ERROR_MESSAGE, tref_edit_message); + break; + } + Glib::ustring *string = &SP_STRING(sub_item)->string; erase_from_spstring(SP_STRING(sub_item), string->begin(), end_text_iter); + success = true; } break; } @@ -663,6 +761,7 @@ sp_te_delete (SPItem *item, Inkscape::Text::Layout::iterator const &start, Inksc erase_from_spstring(string, start_text_iter, string->string.end()); else erase_from_spstring(string, string->string.begin(), string->string.end()); + success = true; } // walk to the next item in the tree if (sub_item->hasChildren()) @@ -688,11 +787,12 @@ sp_te_delete (SPItem *item, Inkscape::Text::Layout::iterator const &start, Inksc } } - while (tidy_xml_tree_recursively(common_ancestor)); + while (tidy_xml_tree_recursively(common_ancestor)){}; te_update_layout_now(item); item->requestDisplayUpdate(SP_OBJECT_MODIFIED_FLAG); - layout->validateIterator(&first); - return first; + layout->validateIterator(&iter_pair.first); + layout->validateIterator(&iter_pair.second); + return success; } @@ -748,9 +848,11 @@ sp_te_get_string_multiline (SPItem const *text, Inkscape::Text::Layout::iterator Glib::ustring result; // not a particularly fast piece of code. I'll optimise it if people start to notice. for ( ; first < last ; first.nextCharacter()) { - SPObject *char_item; + SPObject *char_item = 0; + void *rawptr = 0; Glib::ustring::iterator text_iter; - layout->getSourceOfCharacter(first, (void**)&char_item, &text_iter); + layout->getSourceOfCharacter(first, &rawptr, &text_iter); + char_item = SP_OBJECT(rawptr); if (SP_IS_STRING(char_item)) result += *text_iter; else @@ -765,6 +867,7 @@ sp_te_set_repr_text_multiline(SPItem *text, gchar const *str) g_return_if_fail (text != NULL); g_return_if_fail (SP_IS_TEXT(text) || SP_IS_FLOWTEXT(text)); + Inkscape::XML::Document *xml_doc = SP_OBJECT_REPR(text)->document(); Inkscape::XML::Node *repr; SPObject *object; bool is_textpath = false; @@ -798,12 +901,12 @@ sp_te_set_repr_text_multiline(SPItem *text, gchar const *str) if (e) *e = '\0'; Inkscape::XML::Node *rtspan; if (SP_IS_TEXT(text)) { // create a tspan for each line - rtspan = sp_repr_new ("svg:tspan"); + rtspan = xml_doc->createElement("svg:tspan"); rtspan->setAttribute("sodipodi:role", "line"); } else { // create a flowPara for each line - rtspan = sp_repr_new ("svg:flowPara"); + rtspan = xml_doc->createElement("svg:flowPara"); } - Inkscape::XML::Node *rstr = sp_repr_new_text(p); + Inkscape::XML::Node *rstr = xml_doc->createTextNode(p); rtspan->addChild(rstr, NULL); Inkscape::GC::release(rstr); repr->appendChild(rtspan); @@ -812,7 +915,7 @@ sp_te_set_repr_text_multiline(SPItem *text, gchar const *str) p = (e) ? e + 1 : NULL; } if (is_textpath) { - Inkscape::XML::Node *rstr = sp_repr_new_text(content); + Inkscape::XML::Node *rstr = xml_doc->createTextNode(content); repr->addChild(rstr, NULL); Inkscape::GC::release(rstr); } @@ -832,9 +935,11 @@ text_tag_attributes_at_position(SPItem *item, Inkscape::Text::Layout::iterator c return NULL; // flowtext doesn't support kerning yet SPText *text = SP_TEXT(item); - SPObject *source_item; + SPObject *source_item = 0; + void *rawptr = 0; Glib::ustring::iterator source_text_iter; - text->layout.getSourceOfCharacter(position, (void**)&source_item, &source_text_iter); + text->layout.getSourceOfCharacter(position, &rawptr, &source_text_iter); + source_item = SP_OBJECT(rawptr); if (!SP_IS_STRING(source_item)) return NULL; Glib::ustring *string = &SP_STRING(source_item)->string; @@ -846,13 +951,13 @@ text_tag_attributes_at_position(SPItem *item, Inkscape::Text::Layout::iterator c } void -sp_te_adjust_kerning_screen (SPItem *item, Inkscape::Text::Layout::iterator const &start, Inkscape::Text::Layout::iterator const &end, SPDesktop *desktop, NR::Point by) +sp_te_adjust_kerning_screen (SPItem *item, Inkscape::Text::Layout::iterator const &start, Inkscape::Text::Layout::iterator const &end, SPDesktop *desktop, Geom::Point by) { // divide increment by zoom // divide increment by matrix expansion gdouble factor = 1 / desktop->current_zoom(); - NR::Matrix t = sp_item_i2doc_affine(item); - factor = factor / NR::expansion(t); + Geom::Matrix t (sp_item_i2doc_affine(item)); + factor = factor / t.descrim(); by = factor * by; unsigned char_index; @@ -873,20 +978,22 @@ sp_te_adjust_rotation_screen(SPItem *text, Inkscape::Text::Layout::iterator cons // divide increment by zoom // divide increment by matrix expansion gdouble factor = 1 / desktop->current_zoom(); - NR::Matrix t = sp_item_i2doc_affine(text); - factor = factor / NR::expansion(t); - SPObject *source_item; + Geom::Matrix t (sp_item_i2doc_affine(text)); + factor = factor / t.descrim(); Inkscape::Text::Layout const *layout = te_get_layout(text); if (layout == NULL) return; - layout->getSourceOfCharacter(std::min(start, end), (void**)&source_item); - if (source_item == NULL) return; + SPObject *source_item = 0; + void *rawptr = 0; + layout->getSourceOfCharacter(std::min(start, end), &rawptr); + source_item = SP_OBJECT(rawptr); + if (source_item == 0) return; gdouble degrees = (180/M_PI) * atan2(pixels, SP_OBJECT_PARENT(source_item)->style->font_size.computed / factor); sp_te_adjust_rotation(text, start, end, desktop, degrees); } void -sp_te_adjust_rotation(SPItem *text, Inkscape::Text::Layout::iterator const &start, Inkscape::Text::Layout::iterator const &end, SPDesktop *desktop, gdouble degrees) +sp_te_adjust_rotation(SPItem *text, Inkscape::Text::Layout::iterator const &start, Inkscape::Text::Layout::iterator const &end, SPDesktop */*desktop*/, gdouble degrees) { unsigned char_index; TextTagAttributes *attributes = text_tag_attributes_at_position(text, std::min(start, end), &char_index); @@ -913,11 +1020,13 @@ sp_te_adjust_tspan_letterspacing_screen(SPItem *text, Inkscape::Text::Layout::it Inkscape::Text::Layout const *layout = te_get_layout(text); gdouble val; - SPObject *source_obj; + SPObject *source_obj = 0; + void *rawptr = 0; unsigned nb_let; - layout->getSourceOfCharacter(std::min(start, end), (void**)&source_obj); + layout->getSourceOfCharacter(std::min(start, end), &rawptr); + source_obj = SP_OBJECT(rawptr); - if (source_obj == NULL) { // end of text + if (source_obj == 0) { // end of text source_obj = text->lastChild(); } if (SP_IS_STRING(source_obj)) { @@ -954,7 +1063,7 @@ sp_te_adjust_tspan_letterspacing_screen(SPItem *text, Inkscape::Text::Layout::it gdouble const zoom = desktop->current_zoom(); gdouble const zby = (by / (zoom * (nb_let > 1 ? nb_let - 1 : 1)) - / NR::expansion(sp_item_i2doc_affine(SP_ITEM(source_obj)))); + / to_2geom(sp_item_i2doc_affine(SP_ITEM(source_obj))).descrim()); val += zby; if (start == end) { @@ -985,8 +1094,21 @@ sp_te_adjust_tspan_letterspacing_screen(SPItem *text, Inkscape::Text::Layout::it text->requestDisplayUpdate(SP_OBJECT_MODIFIED_FLAG | SP_TEXT_LAYOUT_MODIFIED_FLAG); } +double +sp_te_get_average_linespacing (SPItem *text) +{ + Inkscape::Text::Layout const *layout = te_get_layout(text); + if (!layout) + return 0; + + unsigned line_count = layout->lineIndex(layout->end()); + double all_lines_height = layout->characterAnchorPoint(layout->end())[Geom::Y] - layout->characterAnchorPoint(layout->begin())[Geom::Y]; + double average_line_height = all_lines_height / (line_count == 0 ? 1 : line_count); + return average_line_height; +} + void -sp_te_adjust_linespacing_screen (SPItem *text, Inkscape::Text::Layout::iterator const &start, Inkscape::Text::Layout::iterator const &end, SPDesktop *desktop, gdouble by) +sp_te_adjust_linespacing_screen (SPItem *text, Inkscape::Text::Layout::iterator const &/*start*/, Inkscape::Text::Layout::iterator const &/*end*/, SPDesktop *desktop, gdouble by) { // TODO: use start and end iterators to delineate the area to be affected g_return_if_fail (text != NULL); @@ -1004,7 +1126,7 @@ sp_te_adjust_linespacing_screen (SPItem *text, Inkscape::Text::Layout::iterator } unsigned line_count = layout->lineIndex(layout->end()); - double all_lines_height = layout->characterAnchorPoint(layout->end())[NR::Y] - layout->characterAnchorPoint(layout->begin())[NR::Y]; + double all_lines_height = layout->characterAnchorPoint(layout->end())[Geom::Y] - layout->characterAnchorPoint(layout->begin())[Geom::Y]; double average_line_height = all_lines_height / (line_count == 0 ? 1 : line_count); if (fabs(average_line_height) < 0.001) average_line_height = 0.001; @@ -1013,8 +1135,8 @@ sp_te_adjust_linespacing_screen (SPItem *text, Inkscape::Text::Layout::iterator gdouble zby = by / (desktop->current_zoom() * (line_count == 0 ? 1 : line_count)); // divide increment by matrix expansion - NR::Matrix t = sp_item_i2doc_affine (SP_ITEM(text)); - zby = zby / NR::expansion(t); + Geom::Matrix t (sp_item_i2doc_affine (SP_ITEM(text))); + zby = zby / t.descrim(); switch (style->line_height.unit) { case SP_CSS_UNIT_NONE: @@ -1032,27 +1154,27 @@ sp_te_adjust_linespacing_screen (SPItem *text, Inkscape::Text::Layout::iterator else style->line_height.value *= (average_line_height + zby) / average_line_height; break; // absolute-type units - case SP_CSS_UNIT_PX: + case SP_CSS_UNIT_PX: style->line_height.computed += zby; style->line_height.value = style->line_height.computed; break; - case SP_CSS_UNIT_PT: + case SP_CSS_UNIT_PT: style->line_height.computed += zby * PT_PER_PX; style->line_height.value = style->line_height.computed; break; - case SP_CSS_UNIT_PC: + case SP_CSS_UNIT_PC: style->line_height.computed += zby * (PT_PER_PX / 12); style->line_height.value = style->line_height.computed; break; - case SP_CSS_UNIT_MM: + case SP_CSS_UNIT_MM: style->line_height.computed += zby * MM_PER_PX; style->line_height.value = style->line_height.computed; break; - case SP_CSS_UNIT_CM: + case SP_CSS_UNIT_CM: style->line_height.computed += zby * CM_PER_PX; style->line_height.value = style->line_height.computed; break; - case SP_CSS_UNIT_IN: + case SP_CSS_UNIT_IN: style->line_height.computed += zby * IN_PER_PX; style->line_height.value = style->line_height.computed; break; @@ -1081,7 +1203,7 @@ as opposed to sp_style_merge_from_style_string which merges its parameter underneath the existing styles (ie ignoring already set properties). */ static void overwrite_style_with_string(SPObject *item, gchar const *style_string) { - SPStyle *new_style = sp_style_new(); + SPStyle *new_style = sp_style_new(SP_OBJECT_DOCUMENT(item)); sp_style_merge_from_style_string(new_style, style_string); gchar const *item_style_string = SP_OBJECT_REPR(item)->attribute("style"); if (item_style_string && *item_style_string) @@ -1106,7 +1228,7 @@ static bool objects_have_equal_style(SPObject const *parent, SPObject const *chi gchar *parent_style = sp_style_write_string(parent->style, SP_STYLE_FLAG_ALWAYS); // we have to write parent_style then read it again, because some properties format their values // differently depending on whether they're set or not (*cough*dash-offset*cough*) - SPStyle *parent_spstyle = sp_style_new(); + SPStyle *parent_spstyle = sp_style_new(SP_OBJECT_DOCUMENT(parent)); sp_style_merge_from_style_string(parent_spstyle, parent_style); g_free(parent_style); parent_style = sp_style_write_string(parent_spstyle, SP_STYLE_FLAG_ALWAYS); @@ -1122,7 +1244,7 @@ static bool objects_have_equal_style(SPObject const *parent, SPObject const *chi } child = SP_OBJECT_PARENT(child); } - SPStyle *child_spstyle = sp_style_new(); + SPStyle *child_spstyle = sp_style_new(SP_OBJECT_DOCUMENT(parent)); sp_style_merge_from_style_string(child_spstyle, child_style_construction.c_str()); gchar *child_style = sp_style_write_string(child_spstyle, SP_STYLE_FLAG_ALWAYS); sp_style_unref(child_spstyle); @@ -1183,7 +1305,8 @@ name of the xml for a text span (ie tspan or flowspan). */ static void recursively_apply_style(SPObject *common_ancestor, SPCSSAttr const *css, SPObject *start_item, Glib::ustring::iterator start_text_iter, SPObject *end_item, Glib::ustring::iterator end_text_iter, char const *span_object_name) { bool passed_start = start_item == NULL ? true : false; - + Inkscape::XML::Document *xml_doc = sp_document_repr_doc(SP_OBJECT_DOCUMENT(common_ancestor)); + for (SPObject *child = common_ancestor->firstChild() ; child != NULL ; child = SP_OBJECT_NEXT(child)) { if (start_item == child) passed_start = true; @@ -1201,7 +1324,7 @@ static void recursively_apply_style(SPObject *common_ancestor, SPCSSAttr const * SPString *string_item = SP_STRING(child); bool surround_entire_string = true; - Inkscape::XML::Node *child_span = sp_repr_new(span_object_name); + Inkscape::XML::Node *child_span = xml_doc->createElement(span_object_name); sp_repr_css_set(child_span, const_cast(css), "style"); // better hope that prototype wasn't nonconst for a good reason SPObject *prev_item = SP_OBJECT_PREV(child); Inkscape::XML::Node *prev_repr = prev_item ? SP_OBJECT_REPR(prev_item) : NULL; @@ -1213,11 +1336,11 @@ static void recursively_apply_style(SPObject *common_ancestor, SPCSSAttr const * unsigned start_char_index = char_index_of_iterator(string_item->string, start_text_iter); unsigned end_char_index = char_index_of_iterator(string_item->string, end_text_iter); - Inkscape::XML::Node *text_before = sp_repr_new_text(string_item->string.substr(0, start_char_index).c_str()); + Inkscape::XML::Node *text_before = xml_doc->createTextNode(string_item->string.substr(0, start_char_index).c_str()); SP_OBJECT_REPR(common_ancestor)->addChild(text_before, prev_repr); SP_OBJECT_REPR(common_ancestor)->addChild(child_span, text_before); Inkscape::GC::release(text_before); - Inkscape::XML::Node *text_in_span = sp_repr_new_text(string_item->string.substr(start_char_index, end_char_index - start_char_index).c_str()); + Inkscape::XML::Node *text_in_span = xml_doc->createTextNode(string_item->string.substr(start_char_index, end_char_index - start_char_index).c_str()); child_span->appendChild(text_in_span); Inkscape::GC::release(text_in_span); SP_OBJECT_REPR(child)->setContent(string_item->string.substr(end_char_index).c_str()); @@ -1229,7 +1352,7 @@ static void recursively_apply_style(SPObject *common_ancestor, SPCSSAttr const * unsigned end_char_index = char_index_of_iterator(string_item->string, end_text_iter); SP_OBJECT_REPR(common_ancestor)->addChild(child_span, prev_repr); - Inkscape::XML::Node *text_in_span = sp_repr_new_text(string_item->string.substr(0, end_char_index).c_str()); + Inkscape::XML::Node *text_in_span = xml_doc->createTextNode(string_item->string.substr(0, end_char_index).c_str()); child_span->appendChild(text_in_span); Inkscape::GC::release(text_in_span); SP_OBJECT_REPR(child)->setContent(string_item->string.substr(end_char_index).c_str()); @@ -1238,11 +1361,11 @@ static void recursively_apply_style(SPObject *common_ancestor, SPCSSAttr const * // eg "abcDEF" -> "abc""DEF" unsigned start_char_index = char_index_of_iterator(string_item->string, start_text_iter); - Inkscape::XML::Node *text_before = sp_repr_new_text(string_item->string.substr(0, start_char_index).c_str()); + Inkscape::XML::Node *text_before = xml_doc->createTextNode(string_item->string.substr(0, start_char_index).c_str()); SP_OBJECT_REPR(common_ancestor)->addChild(text_before, prev_repr); SP_OBJECT_REPR(common_ancestor)->addChild(child_span, text_before); Inkscape::GC::release(text_before); - Inkscape::XML::Node *text_in_span = sp_repr_new_text(string_item->string.substr(start_char_index).c_str()); + Inkscape::XML::Node *text_in_span = xml_doc->createTextNode(string_item->string.substr(start_char_index).c_str()); child_span->appendChild(text_in_span); Inkscape::GC::release(text_in_span); child->deleteObject(); @@ -1325,6 +1448,7 @@ static bool tidy_operator_empty_spans(SPObject **item) the repeated strings will be merged by another operator. */ static bool tidy_operator_inexplicable_spans(SPObject **item) { + if (*item && sp_repr_is_meta_element((*item)->repr)) return false; if (SP_IS_STRING(*item)) return false; if (is_line_break_object(*item)) return false; TextTagAttributes *attrs = attributes_for_object(*item); @@ -1486,7 +1610,8 @@ static bool redundant_semi_nesting_processor(SPObject **item, SPObject *child, b sp_repr_css_attr_unref(css_child_only); if (!equal) return false; - Inkscape::XML::Node *new_span = sp_repr_new(SP_OBJECT_REPR(*item)->name()); + Inkscape::XML::Document *xml_doc = SP_OBJECT_REPR(*item)->document(); + Inkscape::XML::Node *new_span = xml_doc->createElement(SP_OBJECT_REPR(*item)->name()); if (prepend) { SPObject *prev = SP_OBJECT_PREV(*item); SP_OBJECT_REPR(SP_OBJECT_PARENT(*item))->addChild(new_span, prev ? SP_OBJECT_REPR(prev) : NULL); @@ -1552,6 +1677,7 @@ static bool tidy_operator_styled_whitespace(SPObject **item) for ( ; ; ) { // go up one item in the xml test_item = SP_OBJECT_PARENT(test_item); if (is_line_break_object(test_item)) break; + if (SP_IS_FLOWTEXT(test_item)) return false; SPObject *next = SP_OBJECT_NEXT(test_item); if (next) { test_item = next; @@ -1609,7 +1735,7 @@ static bool tidy_xml_tree_recursively(SPObject *root) bool changes = false; for (SPObject *child = root->firstChild() ; child != NULL ; ) { - if (SP_IS_FLOWREGION(child) || SP_IS_FLOWREGIONEXCLUDE(child)) { + if (SP_IS_FLOWREGION(child) || SP_IS_FLOWREGIONEXCLUDE(child) || SP_IS_TREF(child)) { child = SP_OBJECT_NEXT(child); continue; } @@ -1645,17 +1771,30 @@ void sp_te_apply_style(SPItem *text, Inkscape::Text::Layout::iterator const &sta last = start; } Inkscape::Text::Layout const *layout = te_get_layout(text); - SPObject *start_item, *end_item; + SPObject *start_item = 0, *end_item = 0; + void *rawptr = 0; Glib::ustring::iterator start_text_iter, end_text_iter; - layout->getSourceOfCharacter(first, (void**)&start_item, &start_text_iter); - layout->getSourceOfCharacter(last, (void**)&end_item, &end_text_iter); - if (start_item == NULL) + layout->getSourceOfCharacter(first, &rawptr, &start_text_iter); + start_item = SP_OBJECT(rawptr); + layout->getSourceOfCharacter(last, &rawptr, &end_text_iter); + end_item = SP_OBJECT(rawptr); + if (start_item == 0) return; // start is at end of text if (is_line_break_object(start_item)) start_item = SP_OBJECT_NEXT(start_item); if (is_line_break_object(end_item)) end_item = SP_OBJECT_NEXT(end_item); - if (end_item == NULL) end_item = text; + if (end_item == 0) end_item = text; + + + /* Special case: With a tref, we only want to change its style when the whole + * string is selected, in which case the style can be applied directly to the + * tref node. If only part of the tref's string child is selected, just return. */ + + if (!sp_tref_fully_contained(start_item, start_text_iter, end_item, end_text_iter)) { + + return; + } /* stage 1: applying the style. Go up to the closest common ancestor of start and end and then semi-recursively apply the style to all the @@ -1666,9 +1805,24 @@ void sp_te_apply_style(SPItem *text, Inkscape::Text::Layout::iterator const &sta The recursion may involve creating new spans. */ SPObject *common_ancestor = get_common_ancestor(text, start_item, end_item); + + // bug #168370 (consider parent transform and viewBox) + // snipplet copied from desktop-style.cpp sp_desktop_apply_css_recursive(...) + SPCSSAttr *css_set = sp_repr_css_attr_new(); + sp_repr_css_merge(css_set, (SPCSSAttr*) css); + { + Geom::Matrix const local(sp_item_i2doc_affine(SP_ITEM(common_ancestor))); + double const ex(local.descrim()); + if ( ( ex != 0. ) + && ( ex != 1. ) ) { + sp_css_attr_scale(css_set, 1/ex); + } + } + start_item = ascend_while_first(start_item, start_text_iter, common_ancestor); end_item = ascend_while_first(end_item, end_text_iter, common_ancestor); - recursively_apply_style(common_ancestor, css, start_item, start_text_iter, end_item, end_text_iter, span_name_for_text_object(text)); + recursively_apply_style(common_ancestor, css_set, start_item, start_text_iter, end_item, end_text_iter, span_name_for_text_object(text)); + sp_repr_css_attr_unref(css_set); /* stage 2: cleanup the xml tree (of which there are multiple passes) */ /* discussion: this stage requires a certain level of inventiveness because @@ -1682,7 +1836,7 @@ void sp_te_apply_style(SPItem *text, Inkscape::Text::Layout::iterator const &sta and neither option can be made to work, a fallback could be to reduce everything to a single level of nesting and drop all pretence of roundtrippability. */ - while (tidy_xml_tree_recursively(common_ancestor)); + while (tidy_xml_tree_recursively(common_ancestor)){}; // if we only modified subobjects this won't have been automatically sent text->requestDisplayUpdate(SP_OBJECT_MODIFIED_FLAG | SP_OBJECT_STYLE_MODIFIED_FLAG);