Skip to content

doc(Tab): update documentation #5789

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.

Sign up for GitHub

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 7 commits into from
Apr 6, 2025
Merged

doc(Tab): update documentation #5789

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
c10e573
refactor: 重构 Tab 组件增加 TabHeader 参数代替 Layout
ArgoZhang Apr 6, 2025
e3bc9ab
refactor: 移除 _tab 变量
ArgoZhang Apr 6, 2025
3ae7950
test: 更新单元测试
ArgoZhang Apr 6, 2025
fdb2a70
test: 更新单元测试
ArgoZhang Apr 6, 2025
04b8f91
doc: 更新注释
ArgoZhang Apr 6, 2025
982f9aa
refactor: 移除全屏按钮
ArgoZhang Apr 6, 2025
de2a06d
Merge branch 'main' into refactor-tab
ArgoZhang Apr 6, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions src/BootstrapBlazor.Server/Components/Layout/ComponentLayout.razor
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@
</div>

<div class="tabs-coms">
<Tab IsBorderCard="true" IsLazyLoadTabItem="true" ShowFullScreen="true" TabStyle="TabStyle.Chrome" ShowToolbar="true" @ref="Tab">
<Tab IsBorderCard="true" IsLazyLoadTabItem="true" TabStyle="TabStyle.Chrome" ShowToolbar="true" @ref="Tab">
<TabItem Text="Example" Icon="fa-solid fa-desktop">
<CascadingValue Value="@RazorFileName" Name="RazorFileName">
@Body
Expand All @@ -42,7 +42,7 @@
<TabItem Text="C#" Icon="fa-regular fa-file-code">
<Pre @key="@CSharpFileName" CodeFile="@CSharpFileName" class="code-cs"></Pre>
</TabItem>
<TabItem Text="@Video" Icon="fa-regular fa-file-video" ShowFullScreen="false">
<TabItem Text="@Video" Icon="fa-regular fa-file-video">
<Video @key="@VideoFileName" Name="@VideoFileName"></Video>
<p><b>交流群</b></p>
<QQGroup />
Expand Down
2 changes: 1 addition & 1 deletion src/BootstrapBlazor/Components/Layout/Layout.razor.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -262,7 +262,7 @@ public partial class Layout : IHandlerException, ITabHeader
public Func<bool, Task>? OnCollapsed { get; set; }

/// <summary>
/// 获得/设置 默认标签页 关闭所以标签页时自动打开此地址 默认 null 未设置
/// 获得/设置 默认标签页 关闭所有标签页时自动打开此地址 默认 null 未设置
/// </summary>
Comment on lines 264 to 266
Copy link
Contributor

@sourcery-ai sourcery-ai bot Apr 6, 2025

Choose a reason for hiding this comment

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

nitpick (typo): Corrected comment typo improves documentation consistency.

The correction from "关闭所以标签页" to "关闭所有标签页" makes the comment clearer. This change improves readability and avoids potential confusion.

Suggested change
/// <summary>
/// 获得/设置 默认标签页 关闭所以标签页时自动打开此地址 默认 null 未设置
/// 获得/设置 默认标签页 关闭所有标签页时自动打开此地址 默认 null 未设置
/// </summary>
/// <summary>
/// 获得/设置 默认标签页，关闭所有标签页时自动打开此地址。默认为 null（未设置）。
/// </summary>

[Parameter]
public string TabDefaultUrl { get; set; } = "";
Expand Down