Skip to content

docs(zh): updated #2903

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

Merged
merged 4 commits into from
Sep 2, 2019
Merged

docs(zh): updated #2903

merged 4 commits into from
Sep 2, 2019

Conversation

Jinjiang
Copy link
Member

Ref: e05631d...e0d4dc4
/ping @Justineo
Thanks.

<router-link tag="li" to="/foo">
<a>/foo</a>
- `href`:解析后的 URL。将会作为一个 `a` 元素的 `href` attribute。
- `route`:解析后的正常化的地址。
Copy link
Member

Choose a reason for hiding this comment

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

normalize 目前约定的是“规范化”,这里要用吗?

- `href`:解析后的 URL。将会作为一个 `a` 元素的 `href` attribute。
- `route`:解析后的正常化的地址。
- `navigate`:触发导航的函数。**会在必要时自动阻止事件**,和 `router-link` 同理。
- `isActive`:如果需要应用 [active class](#active-class) 则为 `true`。允许应用一个任意的 class。
Copy link
Member

Choose a reason for hiding this comment

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

后文用的是“激活 class”,这里保持一致?

Copy link
Member Author

Choose a reason for hiding this comment

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

统一成了“激活的 class”和“精确激活的 class”如何?

- `route`:解析后的正常化的地址。
- `navigate`:触发导航的函数。**会在必要时自动阻止事件**,和 `router-link` 同理。
- `isActive`:如果需要应用 [active class](#active-class) 则为 `true`。允许应用一个任意的 class。
- `isExactActive`::如果需要应用 [exact active class](#exact-active-class) 则为 `true`。允许应用一个任意的 class。
Copy link
Member

Choose a reason for hiding this comment

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

同上

@posva posva added the docs label Aug 25, 2019
@posva posva merged commit 039bd8f into vuejs:dev Sep 2, 2019
@posva
Copy link
Member

posva commented Sep 2, 2019

Thanks to the both of you! 💚

@Jinjiang Jinjiang deleted the zh branch September 3, 2019 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants