fix selection issue in OnCursorKeyEvent

Solved the problem when selecting with Shift + Left, Right.

1. Deselect the left == right case.
Previously, the selection handle was drawn even though nothing was selected, which looked weird.

2. For SelectWholeText and SelectText, set PrimaryCursorPosition to RightSelectionPosition.
Selection processing in OnCursorKeyEvent is implemented assuming that the cursor position is RightSelectionPosition.

mPrimaryCursorPosition = mRightSelectionPosition;
This ensures the above behavior.

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

Fix svace issue in TextField, TextEditor

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

Merge "Add KEYBOARD_FOCUSABLE_CHILDREN property. Whether the children of this actor can be focusable by keyboard navigation." into devel/master

Merge "Fix VD svace problem." into devel/master

Merge "The field and editor receive all tap events" into devel/master

The field and editor receive all tap events

Change-Id: I5ae41eeb8f6b6840fb8570ac4b5ac3f9f81e3ef2

Merge "Refactoring transition and fadeTransition" into devel/master

Fix SVACE errors in Graphics Backend

Change-Id: I919ed0f473697c8f14c1feb2d66981f052f942f5

Merge "DALi Version 2.0.41" into devel/master

DALi Version 2.0.41

Change-Id: Id7887461b0d94e6aee80bef9e4dca39884f1776c

Refactoring transition and fadeTransition

 - changes fade to fadeTransition
 - changes to use world transition and color for the all transition
   to make it indepenent of the parent Control and its transitions

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

Fix VD svace problem.

 - The return type of fread is size_t has been converted to ssize_t.
 - At the previous implementation, because fread always returns positive value,
   the case ssize_t can be negative exists only when the fseek returns non-zero value.
 - This patch changes ReadBinFile to return false if it read nothing from binary.

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

Add KEYBOARD_FOCUSABLE_CHILDREN property.
Whether the children of this actor can be focusable by keyboard navigation.

Change-Id: I2b650cc485ead5805a019763d0897094c5bd2f9a

Merge "(Vector) Replace std::chrono::system_clock with chrono::steady_clock" into devel/master

Merge "fix wrong behaviour of SelectNone()" into devel/master

Merge "Add SelectionCleared signal" into devel/master

Revert "Add RelayoutRequest on Control"

This reverts commit 71c03dad71835226d820f790dbc53f5c538ec65c.

Add RelayoutRequest on Control

currently RelayoutRequest is on ControlWrapper,
but Control should request relayout by itself.

Change-Id: I2f40c54998d8b78c4a7b746a5fb9e68d8341d291

(Vector) Replace std::chrono::system_clock with chrono::steady_clock

Change-Id: If2cfb975c8796963d38141ae3c1868ee385b40c1

fix wrong behaviour of SelectNone()

1. It is wrong behaviour for the cursor position to be 0 when selection is canceled.
Keeping the previous cursor position is the proper behavior.

2. SelectNone() only works if selection is enabled.
In this case, if selection is canceled, EDITING state is more appropriate than INACTIVE.
In INACTIVE, it is difficult to continue editing work, such as the cursor being deactivated.

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

Merge "Implement more request interceptor APIs." into devel/master

Merge "Remove unused custom shader prefix method" into devel/master

Merge "(TextController) Fixed SVACE errors in EventHandler" into devel/master

(TextController) Fixed SVACE errors in EventHandler

Change-Id: I71b617bca81576d478f2eaae794d45ca9fd3f1b8

Support Public APIs - CopyText, CutText & PasteText

with the following prototypes and descriptions:

string CopyText(TextEditor textEditor) or string CopyText(TextField textField)

This function will copy the previously selected string into the clipboard and will return it.

string CutText(TextEditor textEditor) or string CutText(TextField textField)

This function will cut the previously selected string into the clipboard and will return it.

void PasteText(TextEditor textEditor) or void PasteText(TextField textField

This function will paste the most recent string in the clipboard stack into the text control.

Change-Id: I7dc8156e636f95b8acf8c350bfc7827419e239ca

Implement more request interceptor APIs.

Change-Id: Ie8b2d5247431d428c28d17153e98fa6812c2f93b

Remove unused custom shader prefix method

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

Add SelectionCleared signal

added signal which will be called when selection has been cleared

void OnSelectionCleared(TextEditor textEditor)

Change-Id: Icce352f1709549b86e9697d28588c32ba89affc4

Merge "Add SelectionChanged signal" into devel/master

Add SelectionChanged signal

added signal which will be called when selection has been changed

void OnSelectionChanged(TextEditor textEditor, uint32_t oldStart, uint32_t oldEnd)

Change-Id: I2f769fe11f487462309d8a474fd8ac5aa5c1f2d8

Merge "CanvasView: Change Process to be called as PostProcessor" into devel/master

Merge "DALi Version 2.0.40" into devel/master

DALi Version 2.0.40

Change-Id: I5898f5f1e8cfb8ffaf5ca4e9ed405552184b2727

CanvasView: Change Process to be called as PostProcessor

Size is set first and then Process Rasterizes.

Change-Id: I11f8806d63acd55dc611114fdfbacee0dc72aa1f

Merge "Add CursorPositionChanged Signal" into devel/master

(Vector) Reduce unnecessray loop

Change-Id: I8b39d4d21e8a397bed8f06dbf03fd85ac2cec6fd

Add CursorPositionChanged Signal

added signal which will be called when cursor(caret) position has been changed.

void OnCursorPositionChanged(TextEditor textEditor, uint32_t oldPosition)

Change-Id: I5cb768e1a7ae55840fed1d5c320b48ad8731f30d

Merge "Support line spacing & min line size in texteditor" into devel/master

If parent is hidden, the child cannot be focused.

Change-Id: I4b8fc1d81099583805c3bfc7ab4fd33231bc9f16

Disconnect relationship with VisualUrl and VisualFactory

When VisualFactory destory,
VisualFactoryCache also destroy, and TextureManager also destory.
At this time, mTextureInfoContainer clear, and TextureInfo's VisualUrl destroy.

When VisualUrl destuctor called, we call VisaulFactory::Get() and try to get TextureManager.
But We are now on VisualFactory's destroy called...

This is why image-view-encoded-image-buffer.example make Segfault.

This patch move previous VisualUrl's job
to TextureManager and ImageVisual.

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

Merge "Fixed IsCandidate bug" into devel/master

Merge "Change raw pointer to unique_ptr" into devel/master

Merge "DALi Version 2.0.39" into devel/master

DALi Version 2.0.39

Change-Id: Ia024d79423aef61658cba1f8d9cb1162f472c2e4

Change raw pointer to unique_ptr

The mPlaceholderFont pointer is allocated memory once only
when the text-controller uses a placeholder.
This will persist until mEventData is destroyed.
Changed to a smart pointer suitable for maintenance.

Additionally,
In SetPlaceholderTextFontWidth() and SetPlaceholderTextFontSlant(), mPlaceholderFont can not be NULL.

Even if there is no value in SetFontStyleProperty, it is always set to None,
This is because an instance of mPlaceholderFont is created through SetPlaceholderTextFontWeight(), which is called first.

So this causes coverage issues.
Replace the duplicate code with CreatePlaceholderFont() solves this problem.

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

Merge "CanvasView: Change CanvasRenderer::GetSize() method to const" into devel/master

Merge "Fix comments about CornerRadiusPolicy and BorderlineColor" into devel/master

Support line spacing & min line size in texteditor

support texteditor to have properties like textlabel.

line spacing set the space between lines.
min line size set the total line size (we internally take the max between line size vs font line size)

Change-Id: I2cccc910406db181561ff5e7274b997b60b44737

Fix comments about CornerRadiusPolicy and BorderlineColor

When CornerRadiusPolicy is Relative, the maximum value is 0.5f.
If value is bigger than 0.5f, it will be clampled internally.
But we cannot notificate that informations only by documents.
Now fix it.
+
We add some more informations about borderline color;
It will be affected by visual opacity

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

Merge "Add TextSelect()" into devel/master

Merge "fix wrong behaviour of SelectWholeText()" into devel/master

Merge "Fix SetPrimaryCursorPosition() behaviour when no focus" into devel/master

Fixed IsCandidate bug

Change-Id: Ibaa8a6412f28acf60011173487123f0d9dc6ec08

Merge "DALi Version 2.0.38" into devel/master

DALi Version 2.0.38

Change-Id: Id8fc11eea89453a28282de08ba8635d0df8006e6

Merge "[ATSPI] Fix for SCREEN coordinate type in GetExtents" into devel/master

Merge "[AT-SPI] enhance SHOWING state decision logic" into devel/master

Fixed bug

Change-Id: Ib1f9d1ebbab42399dd8437247adea2072061b30c

Fix to build incrementally with gbs

Incremental build with gbs calls the prep section in the rpm spec once.
It doesn't execute the section from second.

Change-Id: I5eaa375566013bd91de206a34fd0acee937a4e6b

Check if actor can be focused.

Change-Id: I7066fae12059dbdb29e333dd2aac653bf7656ba7

Add TextSelect()

This api can replace the SELECTED_TEXT_START and SELECTED_TEXT_END properties.
The feature to select a range of text should be provided as a method like SelectWholeText(), SelectNone().

Currently, the SELECTED_TEXT_START and SELECTED_TEXT_END properties are causing several problems.
1. User can not set start and end at the same time.
2. Since it is not an event method, it does not work if it is called before text is rendered.

This patch adds SelectText().
And I will make a separate patch to remove the property.

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

CanvasView: Change CanvasRenderer::GetSize() method to const

Change-Id: Ia75ec89ae37ef457102e124eb4ad186dcee480b3

fix wrong behaviour of SelectWholeText()

When there are multiple text controls,
there is a problem that the selection is keeped even if the text control loses focus
due to another text control's SelectWholeText().

This patch prevents wrong selection updates via flags.

// example
DevelTextField::SelectWholeText(field1);
DevelTextField::SelectWholeText(field2);
DevelTextField::SelectWholeText(field3);
DevelTextField::SelectWholeText(field4);

In the case of above example, all TextFields display a selection.

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

Fix SetPrimaryCursorPosition() behaviour when no focus

If TextField does SetPrimaryCursorPosition() before CONNECTED_TO_SCENE,
there is an issue that the caret renders regardless of focus.

Also, SetPrimaryCursorPosition() always sets focus to the Text Control regardless of the user's intent.
This creates unintended bugs in the app.

This patch only updates the value of PrimaryCursorPosition when there is no focus.
When there is focus, the behaviour is the same.

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

[ATSPI] Fix for SCREEN coordinate type in GetExtents

Change-Id: I9c030b9cd7d153e19de13d381778ab1750c71cb3

Merge "PositionSize API at dali-toolkit-test-utils/toolkit-window.cpp" into devel/master

PositionSize API at dali-toolkit-test-utils/toolkit-window.cpp

Previous version don't use Window.RenderSurface's PositionSize information
I add some interface that toolkit UTC can use it.

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

Remove VisualFactory::Get() code at AsyncImageLoader

Make to send EncodedImageBuffer from TextureManager to AsyncImageLoader directly.
Now we don't need to get standalone factory at AsyncImageLoader.
So we don't need fearful exception check

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

[AT-SPI] enhance SHOWING state decision logic

To define SHOWING state, using CULLED and VISIBLE property is not enough.
If a child is clipped out by its parent, then CULLED property is not TRUE.
We need to use similar logic as EFL does.

./src/lib/elementary/efl_ui_widget.c > _elm_widget_onscreen_is
is used for calculating SHOWING state using parents geometry information.

So this patch set is using parents extent information for SHOWING state.

Change-Id: I5eecf93c66ce199bd0ca88f3ad3ff471c1ff79a5

Remove Unnecessary Doxygen PREDEFINED

This typoerror exist since 2014-11-14. And today, It fixed.

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

Merge "Set TapGesture only if there is an anchor." into devel/master

Set TapGesture only if there is an anchor.

Change-Id: I3787911cbedda12b5d781e8de6b16902c23186e7

DALi Version 2.0.37

Change-Id: Ifb9bf35a343400030e3731d3d2506ed30d8b0cad

Merge "Add transition headers in dali-toolkit.h" into devel/master

Add transition headers in dali-toolkit.h

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

CanvasView: Add RemoveAllDrawables() API

Add an API that removes the all drawables registered with AddDrawable().

Change-Id: I6a47c4ce980cdebb374acfe8dfb87b0a815de138

CanvasView: Add RemoveDrawable() API

Add an API that removes the Drawable registered with AddDrawable().

Change-Id: Icf5796ca03093d20f13e5fe12db8c75aa5965a7e

CanvasView: Add Property::VIEW_BOX

Add VIEW_BOX property to change canvas's viewbox value.

Change-Id: I4b23049f390158b232d1055703968637ebcbc3fa

Add CanvasView::New() API that can be created without viewbox.

This is provided in case the viewBox argument is not absolutely necessary.

Change-Id: Ic955b4cacb93620c1f48b62f82ef5cc16c335f59

Merge "Add CanvasView thread rasterization" into devel/master

Fix SVACE error caused by recent ATSPI change

Change-Id: I150b9d4a0afd74081baa92fddf153824dd44faa6

[ATSPI] Blocking unwanted emission of "MoveOuted" signal

This patch is blocking unwanted emission of "MoveOuted" signal to dbus
when auto-scroll feature moves highlighted object into the screen.

This patch requires following dali-adaptor patch:
https://review.tizen.org/gerrit/#/c/platform/core/uifw/dali-adaptor/+/261237/

Change-Id: I43115faf0e9a061436d6751069ca1541ff0d4017

Generate url and load texture from encoded image buffer

Make some way to load image from raw-buffer stream.
This raw-buffer will alive when some actor contain this buffer.

Limitation : animated-image, vector-image, animated-vector-image not working now. Will support another patch

1. Generate url from encoded raw buffer like this
  ImageUrl url = Toolkit::Image::GenerateUrl(const EncodedImageBuffer &);
2. When we use URL as this string, we can load it like normal LOCAL file.
  ImageView imageView = Toolkit::ImageView::New(url.GetUrl());
3. Buffer will automatically removed when ImageUrl scope is out-of-date, and no actor use it.

NOTE : even ImageUrl lifecycle is done, cached texture can use internally.

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

Add CanvasView thread rasterization

When a rasterize request occurs, create a task and rasterize on thread.

Change-Id: I8f9117490fb242c91d0124759e0a1f7df9ed701a

Merge "Reduce the number of conversion between std::string <-> VisaulUrl at interal" into devel/master

Fix SVACE errors in accessibility code in control-data-impl

Change-Id: I08d2e79b19cb269ed48db198ebd04a45f9f89d18

Merge "DALi Version 2.0.36" into devel/master

DALi Version 2.0.36

Change-Id: I4aad1dc9a06436b74dd92396c2e0cf7bf3436ef6

Merge "Implement WebView::GetPlainTextAsynchronously" into devel/master

Merge "[ATSPI] Check higlight area using PropertyNotification" into devel/master

Implement WebView::GetPlainTextAsynchronously

Change-Id: I6691a877ec987dc9971216a43a34bb6daa2ba95d
Signed-off-by: Jiyun Yang <ji.yang@samsung.com>

Added missing initialization in text layout-engine

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

Fix coverity issue in text layout-engine

middleLineIndex is always bigger than 0u by the while condition (0u < middleLineIndex).
therefore execution cannot reach the expression 0u inside the statement below.
: ellipsisLineIndex = middleLineIndex > 0u ? middleLineIndex - 1u : 0u;

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

Merge "Fix to do not remove entity inside loop.(Transition)" into devel/master

Merge "Add ApplyCustomFragmentPrefix" into devel/master

[ATSPI] Check higlight area using PropertyNotification

Move highlight to next or prev if highlighted object goes out of screen.
This patch is using PropertyNotification only for highlighted object.

Change-Id: Idaea158fa9e9a1eb569ab619210d2aa916f28f22

Add Timer at TapGestureRecognizer

Change-Id: Iea2b71b2eb9020cd71067554cdea621e86ae8f7b

Merge "Users can implement focus movement by SetCustomAlgorithm." into devel/master

Fix SVACE error in bidirectional-support.cpp

Change-Id: I732e088e83a76642a4e4bce1bd3f54a04adcd992

Merge "DALi Version 2.0.35" into devel/master