diff --git a/CHANGELOG.md b/CHANGELOG.md index 33a7f8803..25c7cc37e 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -11,6 +11,8 @@ For a complete change history, see the git log. Not yet released +- Added style-level 'opacity' (#314) + - PostGIS: Added 'simplify_geometries' option - will trigger ST_Simplify on geometries before returning to Mapnik (#1179) - Improved error feedback for invalid values passed to map.query_point diff --git a/include/mapnik/feature_type_style.hpp b/include/mapnik/feature_type_style.hpp index 0230c0517..45707fbdc 100644 --- a/include/mapnik/feature_type_style.hpp +++ b/include/mapnik/feature_type_style.hpp @@ -51,7 +51,7 @@ typedef std::vector rule_ptrs; class MAPNIK_DECL feature_type_style { private: - rules rules_; + rules rules_; filter_mode_e filter_mode_; // image_filters std::vector filters_; @@ -63,6 +63,7 @@ private: rule_ptrs if_rules_; rule_ptrs else_rules_; rule_ptrs also_rules_; + float opacity_; public: feature_type_style(); @@ -90,7 +91,9 @@ public: // compositing void set_comp_op(composite_mode_e comp_op); boost::optional comp_op() const; - + void set_opacity(float opacity); + float get_opacity() const; + ~feature_type_style() {} private: diff --git a/src/agg/agg_renderer.cpp b/src/agg/agg_renderer.cpp index c58b5b292..2ec85cd2f 100644 --- a/src/agg/agg_renderer.cpp +++ b/src/agg/agg_renderer.cpp @@ -197,15 +197,25 @@ template void agg_renderer::start_style_processing(feature_type_style const& st) { MAPNIK_LOG_DEBUG(agg_renderer) << "agg_renderer: Start processing style"; - if (st.comp_op()) style_level_compositing_ = true; - else style_level_compositing_ = false; + if (st.comp_op() || st.image_filters().size() > 0 || st.get_opacity() < 1) + { + style_level_compositing_ = true; + } + else + { + style_level_compositing_ = false; + } - if (style_level_compositing_ || st.image_filters().size() > 0) + if (style_level_compositing_) { if (!internal_buffer_) + { internal_buffer_ = boost::make_shared(pixmap_.width(),pixmap_.height()); + } else + { internal_buffer_->set_background(color(0,0,0,0)); // fill with transparent colour + } current_buffer_ = internal_buffer_.get(); } else @@ -217,33 +227,35 @@ void agg_renderer::start_style_processing(feature_type_style const& st) template void agg_renderer::end_style_processing(feature_type_style const& st) { - bool blend_from = false; - if (st.image_filters().size() > 0) + if (style_level_compositing_) { - blend_from = true; - mapnik::filter::filter_visitor visitor(*current_buffer_); - BOOST_FOREACH(mapnik::filter::filter_type filter_tag, st.image_filters()) + bool blend_from = false; + if (st.image_filters().size() > 0) + { + blend_from = true; + mapnik::filter::filter_visitor visitor(*current_buffer_); + BOOST_FOREACH(mapnik::filter::filter_type filter_tag, st.image_filters()) + { + boost::apply_visitor(visitor, filter_tag); + } + } + + if (st.comp_op()) + { + composite(pixmap_.data(),current_buffer_->data(), *st.comp_op(), st.get_opacity(), 0, 0, false); + } + else if (blend_from || st.get_opacity() < 1) + { + composite(pixmap_.data(),current_buffer_->data(), src_over, st.get_opacity(), 0, 0, false); + } + + // apply any 'direct' image filters + mapnik::filter::filter_visitor visitor(pixmap_); + BOOST_FOREACH(mapnik::filter::filter_type filter_tag, st.direct_image_filters()) { boost::apply_visitor(visitor, filter_tag); - } + } } - - if (st.comp_op()) - { - composite(pixmap_.data(),current_buffer_->data(), *st.comp_op(), 1.0f, 0, 0, false); - } - else if (blend_from) - { - composite(pixmap_.data(),current_buffer_->data(), src_over, 1.0f, 0, 0, false); - } - - // apply any 'direct' image filters - mapnik::filter::filter_visitor visitor(pixmap_); - BOOST_FOREACH(mapnik::filter::filter_type filter_tag, st.direct_image_filters()) - { - boost::apply_visitor(visitor, filter_tag); - } - MAPNIK_LOG_DEBUG(agg_renderer) << "agg_renderer: End processing style"; } diff --git a/src/feature_type_style.cpp b/src/feature_type_style.cpp index c14413e6e..2dc820895 100644 --- a/src/feature_type_style.cpp +++ b/src/feature_type_style.cpp @@ -38,7 +38,8 @@ feature_type_style::feature_type_style() : filter_mode_(FILTER_ALL), filters_(), direct_filters_(), - scale_denom_validity_(-1) + scale_denom_validity_(-1), + opacity_(1.0f) {} feature_type_style::feature_type_style(feature_type_style const& rhs, bool deep_copy) @@ -46,7 +47,8 @@ feature_type_style::feature_type_style(feature_type_style const& rhs, bool deep_ filters_(rhs.filters_), direct_filters_(rhs.direct_filters_), comp_op_(rhs.comp_op_), - scale_denom_validity_(-1) + scale_denom_validity_(-1), + opacity_(rhs.opacity_) { if (!deep_copy) { rules_ = rhs.rules_; @@ -67,6 +69,7 @@ feature_type_style& feature_type_style::operator=(feature_type_style const& rhs) direct_filters_ = rhs.direct_filters_; comp_op_ = rhs.comp_op_; scale_denom_validity_ = -1; + opacity_= rhs.opacity_; return *this; } @@ -126,6 +129,16 @@ boost::optional feature_type_style::comp_op() const return comp_op_; } +void feature_type_style::set_opacity(float opacity) +{ + opacity_ = opacity; +} + +float feature_type_style::get_opacity() const +{ + return opacity_; +} + void feature_type_style::update_rule_cache(double scale_denom) { if_rules_.clear(); diff --git a/src/load_map.cpp b/src/load_map.cpp index c1fbe21e2..a73bb3853 100644 --- a/src/load_map.cpp +++ b/src/load_map.cpp @@ -433,8 +433,13 @@ void map_parser::parse_style(Map & map, xml_node const& sty) } } - // image filters + optional opacity = sty.get_opt_attr("opacity"); + if (opacity) + { + style.set_opacity(*opacity); + } + // image filters mapnik::image_filter_grammar > filter_grammar; diff --git a/src/save_map.cpp b/src/save_map.cpp index 15e9cf127..ff0219ce5 100644 --- a/src/save_map.cpp +++ b/src/save_map.cpp @@ -518,7 +518,6 @@ void serialize_style( ptree & map_node, Map::const_style_iterator style_it, bool { feature_type_style const& style = style_it->second; std::string const& name = style_it->first; - filter_mode_e filter_mode = style.get_filter_mode(); ptree & style_node = map_node.push_back( ptree::value_type("Style", ptree()))->second; @@ -526,11 +525,18 @@ void serialize_style( ptree & map_node, Map::const_style_iterator style_it, bool set_attr(style_node, "name", name); feature_type_style dfl; + filter_mode_e filter_mode = style.get_filter_mode(); if (filter_mode != dfl.get_filter_mode() || explicit_defaults) { set_attr(style_node, "filter-mode", filter_mode); } + double opacity = style.get_opacity(); + if (opacity != dfl.get_opacity() || explicit_defaults) + { + set_attr(style_node, "opacity", opacity); + } + rules::const_iterator it = style.get_rules().begin(); rules::const_iterator end = style.get_rules().end(); for (; it != end; ++it)