always add labels to buffers in MapMode::Still

src/mbgl/layer/symbol_layer.cpp

@@ -114,7 +114,8 @@ bool SymbolLayer::recalculate(const StyleCalculationParameters& parameters) {
 
 std::unique_ptr<Bucket> SymbolLayer::createBucket(StyleBucketParameters& parameters) const {
     auto bucket = std::make_unique<SymbolBucket>(parameters.tileID.overscaling,
-                                                 parameters.tileID.z);
+                                                 parameters.tileID.z,
+                                                 parameters.mode);
 
     bucket->layout = layout;
 

src/mbgl/map/source.cpp

@@ -330,6 +330,7 @@ TileData::State Source::addTile(const TileID& id, const StyleUpdateParameters& p
                                                              std::move(monitor),
                                                              info.source_id,
                                                              parameters.style,
+                                                             parameters.mode,
                                                              callback);
         }
 

src/mbgl/map/tile_worker.cpp

@@ -21,13 +21,15 @@ TileWorker::TileWorker(TileID id_,
                        SpriteStore& spriteStore_,
                        GlyphAtlas& glyphAtlas_,
                        GlyphStore& glyphStore_,
-                       const std::atomic<TileData::State>& state_)
+                       const std::atomic<TileData::State>& state_,
+                       const MapMode mode_)
     : id(id_),
       sourceID(std::move(sourceID_)),
       spriteStore(spriteStore_),
       glyphAtlas(glyphAtlas_),
       glyphStore(glyphStore_),
-      state(state_) {
+      state(state_),
+      mode(mode_) {
 }
 
 TileWorker::~TileWorker() {
@@ -140,7 +142,8 @@ void TileWorker::parseLayer(const StyleLayer* layer, const GeometryTile& geometr
                                      spriteStore,
                                      glyphAtlas,
                                      glyphStore,
-                                     *collisionTile);
+                                     *collisionTile,
+                                     mode);
 
     std::unique_ptr<Bucket> bucket = layer->createBucket(parameters);
 

src/mbgl/map/tile_worker.hpp

@@ -3,6 +3,7 @@
 
 #include <mapbox/variant.hpp>
 
+#include <mbgl/map/mode.hpp>
 #include <mbgl/map/tile_data.hpp>
 #include <mbgl/util/noncopyable.hpp>
 #include <mbgl/util/ptr.hpp>
@@ -44,7 +45,8 @@ class TileWorker : public util::noncopyable {
                SpriteStore&,
                GlyphAtlas&,
                GlyphStore&,
-               const std::atomic<TileData::State>&);
+               const std::atomic<TileData::State>&,
+               const MapMode);
     ~TileWorker();
 
     TileParseResult parseAllLayers(std::vector<std::unique_ptr<StyleLayer>>,
@@ -67,6 +69,7 @@ class TileWorker : public util::noncopyable {
     GlyphAtlas& glyphAtlas;
     GlyphStore& glyphStore;
     const std::atomic<TileData::State>& state;
+    const MapMode mode;
 
     bool partialParse = false;
 

src/mbgl/map/vector_tile_data.cpp

@@ -12,6 +12,7 @@ VectorTileData::VectorTileData(const TileID& id_,
                                std::unique_ptr<GeometryTileMonitor> monitor_,
                                std::string sourceID,
                                Style& style_,
+                               const MapMode mode_,
                                const std::function<void()>& callback)
     : TileData(id_),
       style(style_),
@@ -21,7 +22,8 @@ VectorTileData::VectorTileData(const TileID& id_,
                  *style_.spriteStore,
                  *style_.glyphAtlas,
                  *style_.glyphStore,
-                 state),
+                 state,
+                 mode_),
       monitor(std::move(monitor_))
 {
     state = State::loading;

src/mbgl/map/vector_tile_data.hpp

@@ -22,6 +22,7 @@ class VectorTileData : public TileData {
                    std::unique_ptr<GeometryTileMonitor> monitor,
                    std::string sourceID,
                    Style&,
+                   const MapMode,
                    const std::function<void()>& callback);
 
     ~VectorTileData();

src/mbgl/renderer/painter_symbol.cpp

@@ -153,8 +153,8 @@ void Painter::renderSymbol(SymbolBucket& bucket, const SymbolLayer& layer, const
     }
 
     // TODO remove the `true ||` when #1673 is implemented
-    const bool drawAcrossEdges = true || !(layout.text.allowOverlap || layout.icon.allowOverlap ||
-          layout.text.ignorePlacement || layout.icon.ignorePlacement);
+    const bool drawAcrossEdges = (data.mode == MapMode::Continuous) && (true || !(layout.text.allowOverlap || layout.icon.allowOverlap ||
+          layout.text.ignorePlacement || layout.icon.ignorePlacement));
 
     // Disable the stencil test so that labels aren't clipped to tile boundaries.
     //

src/mbgl/renderer/symbol_bucket.cpp

@@ -53,8 +53,8 @@ SymbolInstance::SymbolInstance(Anchor& anchor, const std::vector<Coordinate>& li
     iconCollisionFeature(line, anchor, shapedIcon, iconBoxScale, iconPadding, iconAlongLine) {};
 
 
-SymbolBucket::SymbolBucket(float overscaling_, float zoom_)
-    : overscaling(overscaling_), zoom(zoom_), tileSize(512 * overscaling_), tilePixelRatio(tileExtent / tileSize) {
+SymbolBucket::SymbolBucket(float overscaling_, float zoom_, const MapMode mode_)
+    : overscaling(overscaling_), zoom(zoom_), tileSize(512 * overscaling_), tilePixelRatio(tileExtent / tileSize), mode(mode_) {
 }
 
 SymbolBucket::~SymbolBucket() {
@@ -329,7 +329,7 @@ void SymbolBucket::addFeature(const std::vector<std::vector<Coordinate>> &lines,
             // the buffers for both tiles and clipped to tile boundaries at draw time.
             //
             // TODO remove the `&& false` when is #1673 implemented
-            const bool addToBuffers = inside || (mayOverlap && false);
+            const bool addToBuffers = (mode == MapMode::Still) || inside || (mayOverlap && false);
 
             symbolInstances.emplace_back(anchor, line, shapedText, shapedIcon, layout, addToBuffers,
                     textBoxScale, textPadding, textAlongLine,

src/mbgl/renderer/symbol_bucket.hpp

@@ -3,6 +3,7 @@
 
 #include <mbgl/renderer/bucket.hpp>
 #include <mbgl/map/geometry_tile.hpp>
+#include <mbgl/map/mode.hpp>
 #include <mbgl/geometry/vao.hpp>
 #include <mbgl/geometry/elements_buffer.hpp>
 #include <mbgl/geometry/text_buffer.hpp>
@@ -65,7 +66,7 @@ class SymbolBucket : public Bucket {
     typedef ElementGroup<1> CollisionBoxElementGroup;
 
 public:
-    SymbolBucket(float overscaling, float zoom);
+    SymbolBucket(float overscaling, float zoom, const MapMode);
     ~SymbolBucket() override;
 
     void upload() override;
@@ -119,6 +120,7 @@ class SymbolBucket : public Bucket {
     const float tileSize;
     const float tileExtent = 4096.0f;
     const float tilePixelRatio;
+    const MapMode mode;
 
     std::set<GlyphRange> ranges;
     std::vector<SymbolInstance> symbolInstances;

src/mbgl/style/style.cpp

@@ -132,6 +132,7 @@ void Style::update(const TransformState& transform,
                                      workers,
                                      texturePool,
                                      shouldReparsePartialTiles,
+                                     data.mode,
                                      data,
                                      *this);
 

src/mbgl/style/style_bucket_parameters.hpp

@@ -1,6 +1,7 @@
 #ifndef STYLE_BUCKET_PARAMETERS
 #define STYLE_BUCKET_PARAMETERS
 
+#include <mbgl/map/mode.hpp>
 #include <mbgl/style/filter_expression.hpp>
 #include <mbgl/map/tile_data.hpp>
 
@@ -26,7 +27,8 @@ class StyleBucketParameters {
                           SpriteStore& spriteStore_,
                           GlyphAtlas& glyphAtlas_,
                           GlyphStore& glyphStore_,
-                          CollisionTile& collisionTile_)
+                          CollisionTile& collisionTile_,
+                          const MapMode mode_)
         : tileID(tileID_),
           layer(layer_),
           state(state_),
@@ -35,7 +37,8 @@ class StyleBucketParameters {
           spriteStore(spriteStore_),
           glyphAtlas(glyphAtlas_),
           glyphStore(glyphStore_),
-          collisionTile(collisionTile_) {}
+          collisionTile(collisionTile_),
+          mode(mode_) {}
 
     bool cancelled() const {
         return state == TileData::State::obsolete;
@@ -52,6 +55,7 @@ class StyleBucketParameters {
     GlyphAtlas& glyphAtlas;
     GlyphStore& glyphStore;
     CollisionTile& collisionTile;
+    const MapMode mode;
 };
 
 } // namespace mbgl

src/mbgl/style/style_update_parameters.hpp

@@ -20,6 +20,7 @@ class StyleUpdateParameters {
                           Worker& worker_,
                           TexturePool& texturePool_,
                           bool shouldReparsePartialTiles_,
+                          const MapMode mode_,
                           MapData& data_,
                           Style& style_)
         : pixelRatio(pixelRatio_),
@@ -29,6 +30,7 @@ class StyleUpdateParameters {
           worker(worker_),
           texturePool(texturePool_),
           shouldReparsePartialTiles(shouldReparsePartialTiles_),
+          mode(mode_),
           data(data_),
           style(style_) {}
 
@@ -39,6 +41,7 @@ class StyleUpdateParameters {
     Worker& worker;
     TexturePool& texturePool;
     bool shouldReparsePartialTiles;
+    const MapMode mode;
 
     // TODO: remove
     MapData& data;