After PR Samsung#4892 `HasBody()` can be false in `ReleaseSwigCPtr`.
So, some class who check double-released might not release
native memory. It might occure some leak.

This patch make them to check `Disposed` instead of `HasBody()`.

+

Clean up some codes of `ReleaseSwigCPtr`.

TODO : Also need to check `Disposable.Dispose` function

TODO : Need to check third-party library implementation who use
their custom `ReleaseSwigCPtr`

Signed-off-by: Eunki Hong <[email protected]>

…amically changed.

GetInputMethodContext() in ReleaseSwigCPtr crashes due to current changes in BaseHandle.
Move DestroyContext() to Dispose to avoid this.

Signed-off-by: Bowon Ryu <[email protected]>

…e thread

Reorder the `ReleaseSwigCPtr` call time and `SwigCPtr` value setup.
It will be matched as `BaseHandle.cs` doing now.

Plus

Make we don't print any fatal log when we try to dispose `Disposable` class.
For example, `Vector2` or `Uint16Pair` don't need to be used only for main thread.
Only some UI releative things has some dependency issue of Main / UI thread.

Signed-off-by: Eunki, Hong <[email protected]>

Revert patch of PR Samsung#4897

This reverts commit 0e6528a.

- Adds descriptions
- Adds throwing an exception

Signed-off-by: Hwankyu Jhun <[email protected]>

Signed-off-by: Hwankyu Jhun <[email protected]>

Signed-off-by: Hwankyu Jhun <[email protected]>

Signed-off-by: Hwankyu Jhun <[email protected]>

To provide the screen size, NUIApplication.GetScreenSize is added.

Signed-off-by: Eunki Hong <[email protected]>

Make recent action match with dali

Signed-off-by: Eunki Hong <[email protected]>

* [NUI] Add Tizen.NUI.Scene3D simple demo

Hello 3d world!

Signed-off-by: Eunki, Hong <[email protected]>

* Update Scene3DSample.cs

2022 -> 2023

Signed-off-by: Eunki, Hong <[email protected]>
Co-authored-by: Eunki, Hong <[email protected]>

- AppControl.GetDefaultApplicationIds()

Signed-off-by: Hwankyu Jhun <[email protected]>

Signed-off-by: Hwankyu Jhun <[email protected]>

* Implement NUIGadget

Signed-off-by: Hwankyu Jhun <[email protected]>

* Adjust project file

Signed-off-by: Hwankyu Jhun <[email protected]>

* Update csproj

Signed-off-by: Hwankyu Jhun <[email protected]>

* Update implementation

Signed-off-by: Hwankyu Jhun <[email protected]>

* Remove dll unload

Signed-off-by: Hwankyu Jhun <[email protected]>

* Remove unnecessary file

Signed-off-by: Hwankyu Jhun <[email protected]>

* Update sln file

Signed-off-by: Hwankyu Jhun <[email protected]>

* Add missing dependencies

Signed-off-by: Hwankyu Jhun <[email protected]>

* Add a missing property

The type is added.

Signed-off-by: Hwankyu Jhun <[email protected]>

* Update implementation

Signed-off-by: Hwankyu Jhun <[email protected]>

* Fix typo

Signed-off-by: Hwankyu Jhun <[email protected]>

* Check return type of Assembly.CreateInstance()

This patch uses 'as' keyword to check the instance type.
If it's not NUIGadget, the return value will be null.
In that case, the Add() method throws an exception.

Signed-off-by: Hwankyu Jhun <[email protected]>

* Add abstract keyword to NUIGadget class

Signed-off-by: Hwankyu Jhun <[email protected]>

* Modify Create() method of NUIGadget

If the Create() method returns 'false', the Add() method of NUIGadgetManager
throws an exception.

Signed-off-by: Hwankyu Jhun <[email protected]>

* Use CoreApplication.Post() instead of TizenUISynchronzationContext.Post()

Signed-off-by: Hwankyu Jhun <[email protected]>

Signed-off-by: Hwankyu Jhun <[email protected]>

This patch is to prevent the delegate garbage collected.

Signed-off-by: Hwankyu Jhun <[email protected]>

1. Collect all private values in upward of source code.
2. Formatting C#
3. MilliSeconds --> Milliseconds
4. Add some comments

Signed-off-by: Eunki Hong <[email protected]>

Signed-off-by: Jiyun Yang <[email protected]>

* [NUI] Remove build warning messages in NUI Samples

 - Replace `PreFocusChangeEventArgs` from `FocusChangingEventArgs`.
 - Use `HorizontalAlignment` or `VerticalAlignment` properties
   instead of `LinearAlignment`/
 - Use new Vector2(...) constructor instead of using `X` or `Y` setter.
 - Remove Unreachable code.

Signed-off-by: Seoyeon Kim <[email protected]>

* [NUI] Remove build warning messages

 - Remove not used variables : `axis` and `degree`.
 - Clean code according to NUI Coding style
 - Use `HorizontalAlignment` or `VerticalAlignment` properties
   instead of `LinearAlignment`.
 - Use `StyleName` property instead of `SetStyleName(string)`.
 - Use `ChildCount` property instead of `GetChildCount()`.
 - Use new Vector2(...) constructor instead of `Width` or `Height` setter.

Signed-off-by: Seoyeon Kim <[email protected]>

* [NUI] Remove build warning messages part.3

 - Remove not used variables : mClippedImagePropertyIndex,
mLogoTapDetector, FocusEffect.actor, oldPageCount, FrameUpdateCallback.text, etc.
 - Use `HorizontalAlignment` or `VerticalAlignment` properties
  instead of `LinearAlignment`.
 - Use `ChildCount` property instead of `GetChildCount()`.
 - Use new Vector2(...) constructor instead of `Width` or `Height` setter.
 - Clean code according to NUI Coding style

Signed-off-by: Seoyeon Kim <[email protected]>

Signed-off-by: Seoyeon Kim <[email protected]>

Signed-off-by: Eunki Hong <[email protected]>

Since we create new handle at csharp binder, we should call DeleteHandle function.

Note : Extensions.GetInstanceSafely delete the getted cPtr's reference inside of
that function. So we cannot use it.

Signed-off-by: Eunki Hong <[email protected]>

Add sample to use EncodedImageBuffer.

Signed-off-by: Eunki Hong <[email protected]>

Since we update ImageView lazy, ImageUrl's lifecycle might not matched
what user think.

```
ImageUrl imageUrl = ~~~.GenerateUrl();
ImageView imageView = new ImageView(imageUrl.ToString());
imageUrl.Dispose(); ///< We should allow this situation.
```

To make scene, We allow to call UpdateImage(); synchoronously
if we use GeneratedUrl().

Signed-off-by: Eunki Hong <[email protected]>

This PR is FrontPorting of Samsung#4827

Signed-off-by: Eunki, Hong <[email protected]>

Signed-off-by: Eunki, Hong <[email protected]>

There was a flag that prevent state updating while applying style.
But the flag seemed to have a bug : when style is disabled, the state updating is never going to be done.

This patch fixes the bug and changes the flag's name to be more contextual: styleApplied -> styleApplying.

Signed-off-by: Jiyun Yang <[email protected]>

Previously, although users sets size with width 0, the width is updated
by dali's size calculation (e.g. text calculation) instead of width 0
set by user.

The following patch fixed the above bug.
8033b0e

By applying the above patch, some size assignment in theme should be
fixed to set size height instead of size with width 0.

Signed-off-by: Hwankyu Jhun <[email protected]>

Signed-off-by: Hwankyu Jhun <[email protected]>

Checker:
 - DEREF_OF_NULL

Signed-off-by: Hwankyu Jhun <[email protected]>

And hide PROFILE_TV flags to make clear code

Signed-off-by: Jiyun Yang <[email protected]>

…file

Signed-off-by: Jiyun Yang <[email protected]>

* [Applications.Common] Modify Parcel constructor

The withoutHeader parameter is added to the Parcel constructor.
If it's true, the parcel object does not have the header.

Signed-off-by: Hwankyu Jhun <[email protected]>

* [MediaContent] Remove deprecated APIs

Previously, HeightSpecifications of Picker, DatePicker, TimePicker are
MatchParent.
So the Pickers' SizeHeights are calculated based on their parent's
SizeHeight.
This causes the recursive size calculation problem if Picker's parent
has WrapContent HeightSpecification. (e.g. Dialog)

Now, the Pickers' SizeHeights are set in DefaultThemeCommon as follows.
Picker's SizeHeight is set in DefaultThemeCommon.
DatePicker's Pickers SizeHeights are set in DefaultThemeCommon.
TimePicker's Pickers SizeHeights are set in DefaultThemeCommon.

Therefore, Picker does not need to have MatchParent HeightSpecification.

SceneView's camera is not an instance object anymore.
It can be holded by app side.
So, if we dispose it inside of CameraTransition Function, It become sudden dead.

Actually, Dali side camera object still alive, but the NUI.Camera handle is dereferenced.
So, If app try to access that camera, crash occured.

Signed-off-by: Eunki, Hong <[email protected]>

Tested by `dotnet build /p:BuildWithAnalyzer=True` and checked there is no warning.

Signed-off-by: Eunki, Hong <[email protected]>

When the size of thumb or track changes, the it should update the position of the thumb properly.

And to prevent unnecessary switch state updating while applying style, upgrade styleApplying flag to int type.

Signed-off-by: Jiyun Yang <[email protected]>

To support TabView in xaml, TabItem has been added.

TabView provides Add(TabItem) to add a tab from the given TabItem.
TabItem contains Title, IconURL of a new TabButton in TabView's TabBar
and Content of a new View in TabView's TabContent.

To increase Button width based on the Button's text, Button Theme sets
MinimumSize instead of fixed Size.

To follow system font size scale, TextLabel Theme sets FontSizeScale
with UseSystemSetting.

To apply string style during constructing controls, constructors with
string style are added.

* ThemeManager.AppendTheme: While ApplyTheme() clears previous applied user theme, AppendTheme() appends it to the previous one.
* ThemeChanging event: An event called right after the theme changed and before applying it to the view.
* Open Theme.indexer public: This is for C_XAML.
* Theme implements IResourcesProvider: to support XAMLResources in xaml code

Signed-off-by: Jiyun Yang <[email protected]>

Signed-off-by: Jiyun Yang <[email protected]>

- Update Slider ValueIndicatorImage / Text to change its size depending
on the content.
- Thumb size uses natural size of its image size.
- Update Slider style guide sample for ValueIndicator

Signed-off-by: Seoyeon Kim <[email protected]>

Change default camera generate API as 3D engine style.

Signed-off-by: Eunki, Hong <[email protected]>

* Button's IconOrientation bug
In the previous code, when the IconOrientation property changed,
the LayoutItems() was called and it created a new layout.
But in case the button was inside another layout, the behaviour
described above caused unexpect look. Therefore I fixed it
by changing orientation related properties to the existing layout
instead of creating a new layout.

* Improve switch and extension relationship
The switch extension does not register any event handler to the switch components anymore.

Signed-off-by: Jiyun Yang <[email protected]>

* Progress
Previously, the Progress did not update the distance of the indeterminate animation on resize.

* Loading
Add a property that can help to set lottie animation image url.
This is for using Theme.

Signed-off-by: Jiyun Yang <[email protected]>

To support button size set by user, Button's min size is removed.

* Bug: When user changes indicator image url, all indicators turn into unselected look.

Signed-off-by: Jiyun Yang <[email protected]>

Previously, slider thumb size for normal and pressed were different.
This caused that the thumb position was calculated incorrectly.

Now, slider thumb for normal is resized to be the same with the thumb
for pressed.
By the above, the thumb position is calculated correctly.

Signed-off-by: Eunki, Hong <[email protected]>

Menu.Content is a container of menu items.

Previously, Menu.Content was displayed even if there is no menu item.
Now, Menu.Content is displayed only if menu item exists.

Make API that we can use gltf model file defined camera.

Signed-off-by: Eunki Hong <[email protected]>

To make we build by Visual Studio in Windows OS,
Add some sln files that we can build it.

Signed-off-by: Eunki, Hong <[email protected]>

Fix SetInputRegion binding function  with correct internal function.
Because previous binded with includedRegion.

…Ptr.

When if we set View.Size, It will call WidthPolicy / HeightPolicy internal.
And then it will call SizeWidth and SizeHeight.

This patch ignore some duplicated size setting each other.

Signed-off-by: Eunki, Hong <[email protected]>

Signed-off-by: seungho baek <[email protected]>

…owSizeWithBorder

In the case of a border window, the size including the border area is required.

When we create ImageView with url, It will create image visual twice.
To avoid this situation, make we replace internal url value for
creation time.

And also, setup some missing values when we inputed desired size.

Signed-off-by: Eunki, Hong <[email protected]>

Previous MoveAbove / MoveBelow API change only native side layer order.
It didn't apply to the window who hold this layer.

Signed-off-by: Eunki, Hong <[email protected]>

When calling the CreateInstance() method, this patch modifies the set
the ignoreCase to true that the case of the className is no checked.

Signed-off-by: Hwankyu Jhun <[email protected]>
Co-authored-by: Jay Cho <[email protected]>

* [NUI.Gadget] Add new internal APIs

If the UI gadget developer sets the class name, the application developer
can adds a new NUIGadget instance using the NUIGadgetInfo object.

The following APIs are added:
 + NUIGadgetManager.Add(NUIGadgetInfo gadgetInfo); [method]
 + NUIGadgetManager.GetGadgetInfos(); [method]
 + NUIGadgetInfo.ClassName; [property]

Signed-off-by: Hwankyu Jhun <[email protected]>

* Change the visibility of the ClasaName property

The visibility of the NUIGadgetInfo.ClassName property is changed to internal.

Signed-off-by: Hwankyu Jhun <[email protected]>

* Modify description of GetGadgets()

Signed-off-by: Hwankyu Jhun <[email protected]>

* Remove Add() method

Signed-off-by: Hwankyu Jhun <[email protected]>

---------

Signed-off-by: Hwankyu Jhun <[email protected]>
Co-authored-by: Jay Cho <[email protected]>

Signed-off-by: Jihoon Kim <[email protected]>

Signed-off-by: Jiyun Yang <[email protected]>

Since Window.ViewAdded signal required to all common view's creation time,
It will reduce each view's creation time.
And more, ViewAdded signal only works for default windows previously.

Now we make the signal ownership from window.

And additionaly, register custom HitTest signal only if subclass required.

TODO : The role of signal conflict with View.AddedToWindow.
Can we just remove this ViewAdded signal?

TODO : Can we just skip default view constructor's signal connection?

Signed-off-by: Eunki, Hong <[email protected]>

* [NUI.Scene3D] Add model to test glb format

Check glb format file load well
+
Let this demo allow to load multiple camera

Signed-off-by: Eunki Hong <[email protected]>

* Update test/Tizen.NUI.Scene3D.Sample/Scene3DSample.cs

Co-authored-by: Seoyeon2Kim <[email protected]>

* Update Scene3DSample.cs

typo fix

---------

Signed-off-by: Eunki Hong <[email protected]>
Co-authored-by: Eunki Hong <[email protected]>
Co-authored-by: Seoyeon2Kim <[email protected]>

* Update postback pr

* update nightly release

* [NUI.Gadget] Add NUIGadgetResourceManager class

To support multilanguage, a new class is added.

Adds:
 - NUIGadgetResourceManager

Signed-off-by: Hwankyu Jhun <[email protected]>

* [NUI.Gadget] Modify property & metadata name

Signed-off-by: Hwankyu Jhun <[email protected]>

---------

Signed-off-by: Hwankyu Jhun <[email protected]>

Load facial blendshape animation from json format.

Signed-off-by: Eunki, Hong <[email protected]>

* Remove dependency: ExternalThemeManager -> ThemeManager

The NUI theme modules (ThemeManager and ExternalThemeManager) had dependencies
to each other and that did not meet "Seperation of concerns" principle.
This patch makes dependency direction one way: ThemeManager -> ExtenalThemeManager

* Add a sample application to verify functional regressions.

Signed-off-by: Jiyun Yang <[email protected]>

Signed-off-by: Jiyun Yang <[email protected]>

Add RotationCompletedAcknowledgement APIs in Window.
The APIs are SetNeedsRotationCompletedAcknowledgement and SendRotationCompletedAcknowledgement