Skip to content
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

feat: Add topic for igx-drop-down component. #402

Merged
merged 15 commits into from
Jun 1, 2018

Conversation

IvayloG
Copy link
Contributor

@IvayloG IvayloG commented May 10, 2018

closes #401
Related to: IgniteUI/igniteui-angular#984

@IvayloG IvayloG requested review from bazal4o and dafo May 10, 2018 07:20
dafo
dafo previously approved these changes May 10, 2018
...
@ViewChild(IgxDropDownComponent) public igxDropDown: IgxDropDownComponent;

public items: any[] = [
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's create one more group/header

Copy link
Contributor

@bazal4o bazal4o left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have single comment

bazal4o
bazal4o previously approved these changes May 10, 2018

| Name | Cancelable | Description | Parameters
| :--- | :--- | :--- | :--- |
| `onSelection` | false | Emitted when item selection is changing, before the selection completes. | `{ oldSelection: string, newSelection: string, event?: Event }` |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The parameters here are of ISelectionEventArgs type.

| Name | Type | Description |
| :--- | :--- | :--- |
| `selectedItem` | `number` | Gets the selected item. |
| `items` | `QueryList of IgxDropDownItemComponent` | Gets all of the items but headers. |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Items, as Headers, are of IgxDropDownItemComponent[] type.

Copy link
Contributor

@ig-ealbert ig-ealbert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yoshimis I left comments inline.
I will email about problems with the TM / SDLXLIFF file.
Additionally, there were changes made to the toggle.md file, which should be addressed here too, so we keep everything in sync.

title: Drop Down コンポーネント
_description: Ignite UI for Angular Drop Down は、単一項目の選択が可能なグループ化された項目のリストにスクロールを表示します。
_keywords: Ignite UI for Angular, UI controls, Angular widgets, web widgets, UI widgets, Angular, Native Angular Components Suite, Native Angular Controls, Native Angular Components Library, Angular Drop Down components, Angular Drop Down controls
---
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yoshimis There needs to be an extra metadata field added to all new Japanese files.
_language: ja

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added.

---
title: Drop Down コンポーネント
_description: Ignite UI for Angular Drop Down は、単一項目の選択が可能なグループ化された項目のリストにスクロールを表示します。
_keywords: Ignite UI for Angular, UI controls, Angular widgets, web widgets, UI widgets, Angular, Native Angular Components Suite, Native Angular Controls, Native Angular Components Library, Angular Drop Down components, Angular Drop Down controls
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This needs to be translated.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Translated

<iframe id="dropdown-sample-3-iframe" src='{environment:demosBaseUrl}/dropdown-sample-3' width="100%" height="100%" seamless frameBorder="0" onload="onSampleIframeContentLoaded(this);"></iframe>
</div>
<div>
<button data-localize="stackblitz" class="stackblitz-btn" data-iframe-id="dropdown-sample-3-iframe" data-demos-base-url="{environment:demosBaseUrl}">stackblitz で開く</button>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In all of the other topics, we've been casing this as StackBlitz.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Replaced stackblitz with StackBlitz.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just noticed that stackblitz appears all capital in production. So case does not matter.

})
export class AppModule {}
``` 
構成の後、すべてのオプションを [**IgxDropDownComponent**](https://www.infragistics.com/products/ignite-ui-angular/angular/components/drop_down.html) コンポーネントに表示します。ドロップダウンは、ドロップダウン項目として複数のオプションを含みます。
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please update the URL. It can actually be changed to a relative URL like drop_down.html. I forget if they went and changed all of them to .html or .md, so please reference the other topics to see how it's done. Otherwise, this will always go to the EN production site. Even if you just change it to the JP site, then it won't work on staging.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated '.md'.


<div class="divider--half"></div>

選択済みの項目をあらかじめ定義する場合、[**IgxDropDownComponent**](https://www.infragistics.com/products/ignite-ui-angular/angular/components/drop_down.html) **igx-drop-down** `onOpening` イベントを処理します。
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please update the URL here too. Additionally, there is a の missing between the link and the control name.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated URL and adding の.

#### ゲッター
以下のゲッターは **igx-drop-down** コンポーネントで利用できます。

| 名前 | タイプ | 説明 |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same "type" feedback.

| 名前 | タイプ | 説明 |
| :--- | :--- | :--- |
| `selectedItem` | `number` | 選択項目を取得します。|
| `items` | `QueryList of IgxDropDownItemComponent` | Gヘッダー以外のすべての項目を取得します。 |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Extra G here - please remove.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed.

#### 入力
以下の入力は **igx-drop-down-item** コンポーネントで利用できます。

| 名前 | タイプ | 説明 |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same type feedback.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Replaced タイプ with 型.

| `isFocused` | ブール値| 指定した項目がフォーカスされるかどうかを定義します。|

#### ゲッター
以下のゲッターは **igx-drop-down** コンポーネントで利用できます。
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be **igx-drop-down-item**.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added.

#### ゲッター
以下のゲッターは **igx-drop-down** コンポーネントで利用できます。

| 名前 | タイプ | 説明 |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same type feedback.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Replaced タイプ with 型.

@ig-ealbert
Copy link
Contributor

@yoshimis Can you please make the changes to the toggle.md file?

ig-ealbert
ig-ealbert previously approved these changes May 16, 2018
@bazal4o bazal4o merged commit 0977c43 into master Jun 1, 2018
@bazal4o bazal4o deleted the IGanchev/add-dropdown-component branch June 1, 2018 08:13
@bazal4o bazal4o restored the IGanchev/add-dropdown-component branch June 1, 2018 12:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add IgxDropDown topic.
6 participants