Add null check for mTextureSet. 22/244222/1
authorJoogab Yun <joogab.yun@samsung.com>
Wed, 16 Sep 2020 06:22:08 +0000 (15:22 +0900)
committerJoogab Yun <joogab.yun@samsung.com>
Wed, 16 Sep 2020 06:22:08 +0000 (15:22 +0900)
For empty text, text-visual doesn't create a textureSet.
So it happens that mTextureSet is null.

(sample)

 void Create(Application& application)
  {
    // Get a handle to the window
    Window window = application.GetWindow();
    window.SetBackgroundColor(Color::WHITE);

    textLabel = TextLabel::New();
    textLabel.SetProperty( Actor::Property::SIZE, Vector2(300.f, 200.f));
    textLabel.SetProperty( Toolkit::TextLabel::Property::TEXT, ""  );
    textLabel.SetProperty( Toolkit::TextLabel::Property::ENABLE_AUTO_SCROLL, true  );

    window.Add(textLabel);

    // Respond to a touch anywhere on the window
    window.GetRootLayer().TouchedSignal().Connect(this, &HelloWorldController::OnTouch);

  }

  bool OnTouch(Actor actor, const TouchEvent& touch)
  {
    textLabel.SetProperty( Toolkit::TextLabel::Property::TEXT, ""  );
    textLabel.SetProperty( Toolkit::TextLabel::Property::ENABLE_AUTO_SCROLL, false  );
    return true;
  }

Change-Id: Iff50abb0278c7d2f650a8c92cf8d5c5eaec6f36a

dali-toolkit/internal/text/text-scroller.cpp [changed mode: 0644->0755]

old mode 100644 (file)
new mode 100755 (executable)
index 22f1156..6d0930b
@@ -271,7 +271,10 @@ void TextScroller::SetParameters( Actor scrollingTextActor, Renderer renderer, T
 
     // Reset to the original shader and texture before scrolling
     mRenderer.SetShader(mShader);
 
     // Reset to the original shader and texture before scrolling
     mRenderer.SetShader(mShader);
-    mRenderer.SetTextures( mTextureSet );
+    if( mTextureSet )
+    {
+      mRenderer.SetTextures( mTextureSet );
+    }
   }
 
   mShader = mRenderer.GetShader();
   }
 
   mShader = mRenderer.GetShader();