-
Notifications
You must be signed in to change notification settings - Fork 258
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[NUI] Add WidgetComponent for ComponentBasedApplication #6396
Conversation
Internal API ChangedAdded: 10, Removed: 0, Changed: 0Added+ /// <since_tizen>none</since_tizen
+ [EditorBrowsable(EditorBrowsableState.Never)]
+ Tizen.NUI.NUIWidgetComponent
+ /// <since_tizen>none</since_tizen
+ [EditorBrowsable(EditorBrowsableState.Never)]
+ Tizen.NUI.Window Tizen.NUI.NUIWidgetComponent::Window()
+ /// <since_tizen>none</since_tizen
+ [EditorBrowsable(EditorBrowsableState.Never)]
+ System.Boolean Tizen.NUI.NUIWidgetComponent::OnCreate(System.Int32,System.Int32)
+ /// <since_tizen>none</since_tizen
+ [EditorBrowsable(EditorBrowsableState.Never)]
+ System.Void Tizen.NUI.NUIWidgetComponent::.ctor()
+ /// <since_tizen>none</since_tizen
+ [EditorBrowsable(EditorBrowsableState.Never)]
+ System.Void Tizen.NUI.NUIWidgetComponent::OnDestroy(System.Boolean)
+ /// <since_tizen>none</since_tizen
+ [EditorBrowsable(EditorBrowsableState.Never)]
+ System.Void Tizen.NUI.NUIWidgetComponent::OnPause()
+ /// <since_tizen>none</since_tizen
+ [EditorBrowsable(EditorBrowsableState.Never)]
+ System.Void Tizen.NUI.NUIWidgetComponent::OnResume()
+ /// <since_tizen>none</since_tizen
+ [EditorBrowsable(EditorBrowsableState.Never)]
+ System.Void Tizen.NUI.NUIWidgetComponent::OnStart(System.Boolean)
+ /// <since_tizen>none</since_tizen
+ [EditorBrowsable(EditorBrowsableState.Never)]
+ System.Void Tizen.NUI.NUIWidgetComponent::OnStop()
+ /// <since_tizen>none</since_tizen
+ [EditorBrowsable(EditorBrowsableState.Never)]
+ Tizen.Applications.ComponentBased.Common.IWindowProxy Tizen.NUI.NUIWidgetComponent::CreateWindowInfo(System.Int32,System.Int32)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 💯
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall Looks good because WidgeView is already specific for the owner ;)
And adding samples is also great/
@@ -0,0 +1,94 @@ | |||
/* | |||
* Copyright(c) 2021 Samsung Electronics Co., Ltd. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very minor
* Copyright(c) 2021 Samsung Electronics Co., Ltd. | |
* Copyright(c) 2024 Samsung Electronics Co., Ltd. |
} | ||
|
||
/// <summary> | ||
/// Overrides this method if want to handle behavior when the component receives the appcontrol message. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/// Overrides this method if want to handle behavior when the component receives the appcontrol message. | |
/// Overrides this method if you want to handle behavior when the component receives the appcontrol message. |
@@ -0,0 +1,101 @@ | |||
/* | |||
* Copyright (c) 2022 Samsung Electronics Co., Ltd. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* Copyright (c) 2022 Samsung Electronics Co., Ltd. | |
* Copyright (c) 2024 Samsung Electronics Co., Ltd. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your review. I'll fix it quickly 😊
Adds implementations and samples for WidgetComponent use For more information, please refer to the links below - Tizen.Applications.ComponentBased: https://github.com/Samsung/TizenFX/tree/master/src/Tizen.Applications.ComponentBased - Tizen.Applications.ComponentBased.Default: https://github.com/Samsung/TizenFX/tree/master/src/Tizen.Applications.ComponentBased.Default/Tizen.Applications.ComponentBased.Default - EFL Sample app: https://github.sec.samsung.net/appfw/csharp-component-based-app
Internal API ChangedAdded: 10, Removed: 0, Changed: 0Added+ /// <since_tizen>none</since_tizen
+ [EditorBrowsable(EditorBrowsableState.Never)]
+ Tizen.NUI.NUIWidgetComponent
+ /// <since_tizen>none</since_tizen
+ [EditorBrowsable(EditorBrowsableState.Never)]
+ Tizen.NUI.Window Tizen.NUI.NUIWidgetComponent::Window()
+ /// <since_tizen>none</since_tizen
+ [EditorBrowsable(EditorBrowsableState.Never)]
+ System.Boolean Tizen.NUI.NUIWidgetComponent::OnCreate(System.Int32,System.Int32)
+ /// <since_tizen>none</since_tizen
+ [EditorBrowsable(EditorBrowsableState.Never)]
+ System.Void Tizen.NUI.NUIWidgetComponent::.ctor()
+ /// <since_tizen>none</since_tizen
+ [EditorBrowsable(EditorBrowsableState.Never)]
+ System.Void Tizen.NUI.NUIWidgetComponent::OnDestroy(System.Boolean)
+ /// <since_tizen>none</since_tizen
+ [EditorBrowsable(EditorBrowsableState.Never)]
+ System.Void Tizen.NUI.NUIWidgetComponent::OnPause()
+ /// <since_tizen>none</since_tizen
+ [EditorBrowsable(EditorBrowsableState.Never)]
+ System.Void Tizen.NUI.NUIWidgetComponent::OnResume()
+ /// <since_tizen>none</since_tizen
+ [EditorBrowsable(EditorBrowsableState.Never)]
+ System.Void Tizen.NUI.NUIWidgetComponent::OnStart(System.Boolean)
+ /// <since_tizen>none</since_tizen
+ [EditorBrowsable(EditorBrowsableState.Never)]
+ System.Void Tizen.NUI.NUIWidgetComponent::OnStop()
+ /// <since_tizen>none</since_tizen
+ [EditorBrowsable(EditorBrowsableState.Never)]
+ Tizen.Applications.ComponentBased.Common.IWindowProxy Tizen.NUI.NUIWidgetComponent::CreateWindowInfo(System.Int32,System.Int32)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!!!!!!!!
리뷰 남겨주신 모든분들 감사합니다! 반영하겠습니다 👍 |
Adds implementations and samples for WidgetComponent use
For more information, please refer to the links below
Description of Change
API Changes