Unregist visual directly when IMAGE have empty map

Directly unregist visual when Property::IMAGE have empty map.
This code just follow up the action of control's Property::BACKGROUND doing.

When map is empty, the visual cannot be created. So it will always unregist visual before.
This patch just unregist the IMAGE's visual directly.
This will not print error message when NUI try to Unregist imagevisual

Change-Id: Ifdabbf3555a4471b0d3cf1da7b3f405a5e91553b
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "Make DoCreatePropertyMap use index key" into devel/master

[AT-SPI] Add DevelControl::Property::ACCESSIBILITY_HIDDEN

Change-Id: Ib9e67ae2bda4e38d9f5eed3cf0977948b3573eac

[AT-SPI] Derive ControlAccessible from ActorAccessible

A related dali-adaptor change adds the ActorAccessible class and moves
duplicated code (in AdaptorAccessible and ControlAccessible) from
AdaptorAccessible to ActorAccessible.

Change-Id: If1f62e9d832df61f95d814fde229859729855c45

Avoid integer overflow in cursor helper functions

Since index is uint, it can causes problems if totalNumberOfCharacters is 0.

Change-Id: I3a815474a42ddea9f02ab1e358f7edf8ec0f8dac
Signed-off-by: Bowon Ryu <bowon.ryu@samsung.com>

Make DoCreatePropertyMap use index key

Only image-visual's SynchronousLoading used string key.
It is related with TizenFX's implementation.

It is outlier. So we should change both of them

Change-Id: Iedd002fb24c945cc1fdac94c38ec04198536b4a2
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "Set focusable to true when touched in default." into devel/master

Set focusable to true when touched in default.

Change-Id: Ie9c333b5a0f760434a25d3519b8edee0c22bc6d1

[AT-SPI] Synchronize with AT-SPI cleanup in dali-adaptor

A related dali-adaptor change adds const-correctness and introduces a
new, more coherent naming convention for Accessible objects. The name
ControlAccessible now matches ApplicationAccessible, ProxyAccessible
etc. and the upcoming ActorAccessible.

Change-Id: I63a6887e1c291fbfb475fd8c2384126a31981d6b

Merge "Fix text cursor position issue" into devel/master

Fix text cursor position issue

When the size or direction of text is changed,
all decorator elements should be updated.

Change-Id: Ib2fb61f684801c5743762df67e8e6d53399c8098
Signed-off-by: Bowon Ryu <bowon.ryu@samsung.com>

Merge "Refactoring TextureUploadObserver." into devel/master

Merge "Remove .clang-format file from toolkit/automated-tests" into devel/master

Merge "[AT-SPI] Update #include paths to AT-SPI interfaces" into devel/master

DALi Version 2.1.3

Change-Id: I6c27effcb052b5f4d4732cd13330b2531dccb200

Refactoring TextureUploadObserver.

 - Use unified method for any request.

Change-Id: Ief4b744cd42b9184f83ba0ac0ba3436befd2aa61
Signed-off-by: seungho <sbsh.baek@samsung.com>

Remove .clang-format file from toolkit/automated-tests

Change-Id: Id90b45de110a60c6407e2994698be298aa55dae8
Signed-off-by: seungho <sbsh.baek@samsung.com>

[AT-SPI] Update #include paths to AT-SPI interfaces

A related dali-adaptor change splits accessibility-impl.h so that there
is one header file per AT-SPI interface.

Change-Id: Iffcbf190e8c7ed3098c0c5109c35ffbdd154e1d4

Merge "Fixing Bug: Caret is too big with LineSpacing" into devel/master

Merge "Changed GetNearestFocusableActor interface to receive rootActor" into devel/master

Fixing Bug: Caret is too big with LineSpacing

***********************************************************
Description

When using LineSpacing or MinLineSize the caret size is too big and does not look good.

The primary cursor height was obtaining its height from the line height (which includes the line spacing).
I've modified it to obtain the height from the default font height.

Please use the code below to test the issue and its fix.
***********************************************************

using namespace Dali;
using namespace Dali::Toolkit;

class SimpleApp : public ConnectionTracker
{
public:
  SimpleApp(Application& application)
  : mApplication(application)
  {
    mApplication.InitSignal().Connect(this, &SimpleApp::Create);
  }

  void Create(Application& application)
  {
    Window window = application.GetWindow();
    window.SetBackgroundColor(Vector4(0.04f, 0.345f, 0.392f, 1.0f));

    mEditor = TextEditor::New();
    mEditor.SetProperty(Actor::Property::PARENT_ORIGIN, ParentOrigin::TOP_CENTER);
    mEditor.SetProperty(Actor::Property::ANCHOR_POINT, AnchorPoint::TOP_CENTER);
    mEditor.SetProperty(Actor::Property::POSITION, Vector3(0.f, 0.0f, 0.f));
    mEditor.SetProperty(Actor::Property::SIZE, Vector2(200.f, 100.0f));
    mEditor.SetProperty(TextEditor::Property::TEXT, "Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.\n");
    mEditor.SetProperty(TextEditor::Property::ENABLE_MARKUP, true);
    mEditor.SetProperty(TextEditor::Property::POINT_SIZE, 40.0f);
    mEditor.SetBackgroundColor(Color::WHITE);
    mEditor.SetProperty( DevelTextEditor::Property::MIN_LINE_SIZE, 50.f );

    window.Add(mEditor);
  }

   bool OnButtonClicked(Button button)
  {
    if(button == mButton)
    {
      Vector3 originalSize = mEditor.GetNaturalSize();
    }
    return true;
  }

private:
  Application &mApplication;
  TextEditor mEditor;
  PushButton mButton;
  TextField mField;
};

int DALI_EXPORT_API main(int argc, char** argv)
{
  Application application = Application::New(&argc, &argv);
  SimpleApp test(application);
  application.MainLoop();

  return 0;
}

Change-Id: I8d8accbef03b69ba4e77f4f891bae08e37f93a00

[ATSPI] Fix for DefaultLabel

This patch:

* replaces usage of existing API AddPopup()/RemovePopup() by RegisterDefaultLabel()/UnregisterDefaultLabel() which has wider applicability (not only for popups)
* extends test utils to allow write tests that emit window:activate / window:deactivate events

Related patches:
https://review.tizen.org/gerrit/#/c/platform/core/uifw/dali-adaptor/+/267321/

Change-Id: Ibae793c10d2f29fdd4af32fc69d518e247b54633

Merge "Set FullRelayoutNeeded after GetNaturalSize/GetHeightForWidth to avoid the backup/restore of fields in visual-model." into devel/master

Fix svace issue on text-geometry (unsigned int < 0)

CharacterIndex type is unsigned int. So it cannot be negative

Change-Id: I9f9c6868f103ee45695a4fd87dc4cafd0819f471
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Changed GetNearestFocusableActor interface to receive rootActor

Change-Id: I7994d630cc9544ae048d3fbb4415e393d9870b2b

DoAction can chage CornerRadius/Borderline/BlurRadius

we can update properies by DoAction(DevelVisual::Action::UPDATE_PROPERTY).
It is not an usual case, but we may need to prepare of NUI code fix.
(There is some related issue at NUI ImageView)

Here, I update some properies that need to update shader.
We don't want to change the shader many times.
(ex : DoAction with CornerRadius = 1.0f, and DoAction 0.0f after that, and repeat.)
So if someone update properies by DoAction API, turn on
mAlwaysUseXXX = true so shader will not be previous shader anymore.
(This is same logic when these properies are animated.)

Change-Id: I963930f261982d50bd647e4ba7ed4e476dd8898d
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Fix const table size bug in button + Erase annoying message

Fix unmatched const static table value in button-impl.cpp
+
Remove annoying error message when we change button visual

Change-Id: I97acc25b73395799bef46a03a09739a6708359da
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "Add support for text geometry" into devel/master

Merge "Cache for the synchronous loading." into devel/master

DALi Version 2.1.2

Change-Id: Ie5ce75faddc5f04c4d1a13c326deb160281d6343

Cache for the synchronous loading.

Change-Id: Id793b3bcb72639d3c7a163196e29bdcf7d94718e
Signed-off-by: seungho <sbsh.baek@samsung.com>

Add support for text geometry

Added the following requested API's for text  geometry :

GetTextSize();
GetTextPosition();

above API's could return a list of values in case of :
1-multiline, each line in a separate value.
2-mixed text of LTR & RTL, each contiguous block of the same direction has a separate value.

note that above API's returns values based on the current rendered text position/size.

Change-Id: I1ee9d26954d529bb81c72647dbc9f09ccc8e35ac

Set FullRelayoutNeeded after GetNaturalSize/GetHeightForWidth to avoid the backup/restore of fields in visual-model.

This solved some issues happen after calling GetNaturalSize/GetHeightForWidth or after initialization when set WIDTH_RESIZE_POLICY to ResizePolicy::USE_NATURAL_SIZE

1) Fixing the issue where characters were being drawn at the same location whenever GetNaturalSize was called.
https://review.tizen.org/gerrit/c/platform/core/uifw/dali-toolkit/+/253286
2) Side effect on Scrolling, LineWrap and Invalid position of cursor in TextEditor after calling GetNaturalSize or GetHeightForWidth.
https://review.tizen.org/gerrit/c/platform/core/uifw/dali-toolkit/+/255379
3) Crash issue when set WIDTH_RESIZE_POLICY to ResizePolicy::USE_NATURAL_SIZE in TextEditor then insert/remove characters from 2nd line in it after initialization
4) Crash issue when set WIDTH_RESIZE_POLICY to ResizePolicy::USE_NATURAL_SIZE in TextEditor and select_text at initialization then cut it after initialization
5) Crash issue when set WIDTH_RESIZE_POLICY to ResizePolicy::USE_NATURAL_SIZE in TextEditor then double enter in it after initialization
6) Incorrect NaturalSize after the second call for GetNaturalSize. This issue in TextLabel when call GetNaturalSize, then set MULTI_LINE, then re-call GetNaturalSize with Text has break-line

Change-Id: Ieea984f41a218c62e016baa60156b51e985f2fc0

Merge "DALi Version 2.1.1" into devel/master

DALi Version 2.1.1

Change-Id: Ib2820c8760eba1bb69c38390c0850712b77bcff5

Merge "Fix svace issue at control/text-anchor-impl" into devel/master

Scaledown image when visual is bigger than n-patch fixed length

When n-patch image's fixed length is bigger than visual size,
the visual's geometry breakdown.

This patch calculate good scales in vertex shader.

NOTE : We scaledown same rates both width and height. This action follow up CSS current action.

Change-Id: Ief2c6e6e2deb04dac36a1b9575e7bd620697d81d
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Fix svace issue at control/text-anchor-impl

Initialize member value as zero.

Change-Id: If7b3073e6b75417bbb3b8c25764acb484ddefe2d
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

[AT-SPI] Follow-up on mAccessibilityRelations

The previous implementation used an `std::vector` (the outer one) as a
map, with the `RelationType` key working as the index to that vector.
However, relations are rare in practice, and every `Control::Impl`
having a 23-element vector or vectors, most, if not all of which are
empty, feels like a waste of memory.

This patch also changes the inner vector to a set, which resolves the
previously unaddressed issue of duplicate elements.

Change-Id: I639082fe49e0f6107213cda5b142a09882a3be39

Fix text relayouter update issue

When the size of the text control is 0,
there is a issue that the mRequestedNumberOfCharacters becomes 0
even if the actural text exists.

This patch avoids this issue by updating mNumberOfCharactersToAdd
in textUpdateInfo in that situation.

Change-Id: I440b39f846aa7c9c410a1d121dae7b8a63b811ae
Signed-off-by: Bowon Ryu <bowon.ryu@samsung.com>

Merge "[AT-SPI] Change the type of Control::Impl::mAccessibilityRelations" into devel/master

[AT-SPI] Change the type of Control::Impl::mAccessibilityRelations

This patch changes the relation's target type from
`Accessibility::Address` to `Accessibility::Accessible*`.
`AppendAccessibilityRelation()` does not allow to specify a remote
object anyway, because the parameter types are `Actor`s.

Moreover, the `Address` may be incomplete (specifically the
DBus-dependent part) if the above API is called too early. With this
change, the `Accessible::GetAddress()` call is postponed until the
relations are queried over DBus.

Change-Id: I881d7fb90eba79b6ba9752e913d113272485e29a

DALi Version 2.1.0

Change-Id: Iaa4df4a2e739ed89a6634c4943fdb98ce0ad2473

Removed some redundant methods from TextController & Moved some code to other files

Change-Id: Ib57da54aa520a15f346e24be93e35a18caa0315b

Merge "Fix svace issue at text-controller-impl.h" into devel/master

Merge "[ATSPI] Check parents to define SHOWING state" into devel/master

Fix svace issue at text-controller-impl.h

Minor fix

Change-Id: I25efe7abd36bc61303fe6ddbddd86a6b6c55a44a
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "Remove return; statement at fragment shader" into devel/master

Merge "[ATSPI] Implementation of Hypertext and Hyperlink in text controls" into devel/master

[ATSPI] Check parents to define SHOWING state

It will not be SHOWING if an accessible has invisible parent in its ascendant.

This is necessary for the following case.
The NUI navigator has more than 2 pages, and the below page's controls have
SHOWING state without this patch.

Change-Id: I27a7a998154267a14c2aa295a48b48b9e2896520

Merge "[ATSPI] do not send ROLE property changed event" into devel/master

Added appropriate @SINCE & @DEPRECATED aliases for DALi's 2.1 release & Tizen's 6.5 release

Change-Id: Icc5d8ce08b5967eb60afdc2654e17454876667d4

[ATSPI] Implementation of Hypertext and Hyperlink in text controls

This patch exposes on dbus ATSPI Hypertext interface
for the following text controls: TextEditor, TextField and TextLabel.
Also it brings new class TextAnchor inheriting from Control which marks
the geometry of an anchor inside the text controls mentioned above and
which exposes on dbus ATSPI Hyperlink interface.

Change-Id: Ic46bcf7a3ddfe49b1723ebf8025fba6779fda05d

Remove return; statement at fragment shader

Some graphic driver doesn't support return statement at fragment shader's main.
This patch just above that case with same logic

Change-Id: I8011c722946aeac2611908020cd73bc7b6f9cf56
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

[ATSPI] do not send ROLE property changed event

This patch is following the decision of the patch
https://review.tizen.org/gerrit/#/c/platform/upstream/elementary/+/79631/4/src/lib/elm_interface_atspi_accessible.c

To prevent unnecessary event.
No one is interest in this event.

Change-Id: I044ddeaf58db3010da827606f6fa83050050ec41

Merge "Add a callback to get textfitted font size." into devel/master

Merge "[AT-SPI] Support for hidden text added" into devel/master

(TextController/ScrollView) Reduced LOC

 - TextController - Moved some methods to the Impl class
 - TextControllerImpl - Moved a lot of functionality outside of the class
 - ScrollView - Added a class which handles the properties

Change-Id: I69127a9bc3dbd3e892f1fe0019cac9ad41fc2f3e

Refactoring ImageVisualShaderFactory::GetShader

There are two Intention this patch
1. Clean up GetShader params
2. Allow that native image also use visual features (a.k.a. RoundedCorner + Borderline)

Change-Id: I8b0cbd0a4c7c622268c90d3cbbbd345ae83dfa07
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "(Text Controller) Moved some input properties into a different class & some functionality into impl" into devel/master

Merge "DALi Version 2.0.53" into devel/master

DALi Version 2.0.53

Change-Id: I48308ed392eb944d51206b5b31307f0b99beb178

(Text Controller) Moved some input properties into a different class & some functionality into impl

Change-Id: I4d2cbd392d305ccd2ee31972df132713ff5cc19c

Merge "Fix alignment issues with field and editor." into devel/master

Merge "Support Emoji sequences" into devel/master

Fix alignment issues with field and editor.

Change-Id: I4bf0c7dbad30ff74b6c5ed455db2f6b6f75b108d

Support Emoji sequences

This is a common solution that handles emoji sequences:
  :: text presentation sequence and selector
  :: emoji presentation sequence and selector
  :: emoji modifier sequence like skin tone
  :: emoji keycap sequence
  :: emoji flag sequence
  :: emoji tag sequence like England flag
  :: more cases in emoji zwj sequence like Rainbow Flag

This patch handle display cases.
The editing cases will be handled in another patch.

This patch depends on the dali-adaptor patch:
https://review.tizen.org/gerrit/c/platform/core/uifw/dali-adaptor/+/266046

Change-Id: I1bc7aed8a73b1d396019b2ef0a7fab5ea187d39f

Add a callback to get textfitted font size.

Change-Id: I1aea1a5d5b03a27c89feb709d4877369c29290bb

[AT-SPI] Support for hidden text added

- Modified AT-SPI API to not disclose the hidden text in returned values.
- Values returned instead are either empty string or replacement characters.

Change-Id: I6736a30ad70a45b77b0b4ac4b54c2a2c0072df25

Merge "Fix: The last line of the text overlaps with the text-editor's border/edge" into devel/master

fixed ellipsis position with multi line

Change-Id: I2b2e47c307dab38770717bf483e302097ddbcf38

Merge "[ATSPI] make ScrollToChild use descendant" into devel/master

DALi Version 2.0.52

Change-Id: I263b610f85720c5f22ce490a17d4b05081f494df

[ATSPI] make ScrollToChild use descendant

It is not make a child visible if the child is 1st generation child.
Usually the 1st generation child is content container.
So we need to use descendant.

This patch makes the CollectionView work with following change;
https://github.com/Samsung/TizenFX/pull/3726

We need to enhance this part and related places later.

Change-Id: I9e88312b7b1936909dc350cc1154f2a18e900748

Merge "[ATSPI] Update accessibility-test-utils files" into devel/master

Merge "Adding theme for Raspberry Pi" into devel/master

Adding theme for Raspberry Pi

Change-Id: I9abb4298532e3184e976fe4869598b5fd05c5d69

Fix alignment issues in backgroundActor.

Change-Id: I7fd58d1437a1118ab60568567e3613adcf0ab0c4

Merge "CanvasView: Do ApplyNativeFragmentShader for NativeImage buffer" into devel/master

Merge "Support multiple transitions for a Control." into devel/master

[ATSPI] Update accessibility-test-utils files

- Set 'true' to Accessibility::Accessible::Get() for the first time

- This patch should be merged together with dali-adaptor one :
  https://review.tizen.org/gerrit/#/c/platform/core/uifw/dali-adaptor/+/264762/

Change-Id: I95034fce30d391942d24d3a0be10d21f04bb64b6
Signed-off-by: Seoyeon Kim <seoyeon2.kim@samsung.com>

Support multiple transitions for a Control.

 - If a Control has multiple transitions, the Control must be hidden durring only the minimum delay of the transitions.
 - During delay time, the transitioned property of the Control will be set to the source value.

Change-Id: If158d01511b373edaad59b5e288bc33e5adde9bb
Signed-off-by: seungho <sbsh.baek@samsung.com>

Merge "DALi Version 2.0.51" into devel/master

DALi Version 2.0.51

Change-Id: I657554cccea1b9c262bdd8dd60a04a459a523947

Merge "Move ModelUpdating method outside of Text::Controller::Impl class to reduce LOC" into devel/master

Move ModelUpdating method outside of Text::Controller::Impl class to reduce LOC

Change-Id: Id4e4c47793b3e37ef999b0014403d3d30f2b77e8

Fix text visual issue

uTextColorAnimatable property should always be registered.

Change-Id: I3ffbccc896f6abe4665a6f133b5be22df64df043

Merge "Fix text constraint issue" into devel/master

CanvasView: Do ApplyNativeFragmentShader for NativeImage buffer

Change-Id: I916af0c85333a53d8c22063522c53c010d503b21

Fix text constraint issue

Not to create new Constraints when the visual is added to the Scene again

Change-Id: I19d30064d9f8cd08ca500a5ad2dee17b4b775b5a

Refactored text editor/field to reduce loc

Change-Id: I408f8931071c6c4464ba3dafb00e36311d01379c
Signed-off-by: David Steele <david.steele@samsung.com>

Refactored text field/editor property getters/setters

Change-Id: Iab27acf2f198af060d3eee398b9874116ff7e562

DALi Version 2.0.50

Change-Id: I7a4514833e041b15e4f638009762739f913e187a

CanvasView: Refactoring to get rasterized buffer

When the rasterized buffer is fetched after the task is rasterized,
it is received as a texture, not as pixel data.
This makes the buffer in dali-adaptor not dependent on whether
to use NativeImageQueue's tbm_surface or PixelData.

Change-Id: I80338ea2454847ce88a8f79fa67e38a842cf67f2

Merge "(TextController) Reformatted to reduce LOC" into devel/master

Merge "Fix numeric error on BLUR_RADIUS" into devel/master

(TextController) Reformatted to reduce LOC

Change-Id: Idaf7745fb13fa60864fa6f1dd39c1558a8d942f1

Fix numeric error on BLUR_RADIUS

Some device cannot calculate 1.0 / (sqrt(D) + B);
So we make mediump to highp.

Change-Id: I5c50a70fdef748daca6452333ced8e21c7e66e19
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Make to use Size of source for transition.

 - currently GetCurrentProperty is used to get size of source.
 - but, when source is a control to be transitioned, it could be 0, 0, 0.

Change-Id: Ic91c83afa7ee08a9acfb152b5078af115667ae9b
Signed-off-by: seungho <sbsh.baek@samsung.com>

Make transition select target control during pair transition

Change-Id: I87c868572b67dab4ad3289299599ebb3d25104da
Signed-off-by: Seungho Baek <sbsh.baek@samsung.com>