Skip to content

feat(Tab): reduce toolbar spacing #5782

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
ArgoZhang merged 11 commits into from
Apr 4, 2025
Merged

feat(Tab): reduce toolbar spacing #5782

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
42f022b
refactor: 调整间隙
ArgoZhang Apr 4, 2025
1bcdfe0
refactor: 调整拐角
ArgoZhang Apr 4, 2025
9f4991f
refactor: 调整左间隙
ArgoZhang Apr 4, 2025
afcfc42
refactor: 增加左侧偏移量
ArgoZhang Apr 4, 2025
94af160
refactor: 调整不同模式左侧偏移量
ArgoZhang Apr 4, 2025
f57a37f
chore: bump version 9.5.3
ArgoZhang Apr 4, 2025
0ebb61d
refactor: 兼容阿拉伯兄弟
ArgoZhang Apr 4, 2025
d9c2433
refactor: 调整工具按钮防止被遮挡
ArgoZhang Apr 4, 2025
67589bd
refactor: 左侧间隙支持响应式
ArgoZhang Apr 4, 2025
a18f33e
refactor: 更新工具按钮大小
ArgoZhang Apr 4, 2025
b6460b7
refactor: 更新 DevUI 样式
ArgoZhang Apr 4, 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
20 changes: 14 additions & 6 deletions src/BootstrapBlazor.Server/wwwroot/css/devui.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -168,13 +168,21 @@ pre code.hljs {
--bb-tabs-item-close-button-height: 16px;
}

.tabs-chrome > .tabs-header,
.tabs-capsule > .tabs-header {
--bb-tabs-item-body-padding: 2px 8px;
--bb-tabs-header-height: 26px;
}

.tabs-chrome {
--bb-tabs-item-wrap-padding-x: 0.5rem;
--bb-tabs-item-body-margin-bottom: 2px;
}

.tabs-chrome > .tabs-header,
.tabs-capsule > .tabs-header {
--bb-tabs-item-body-padding: 2px 8px;
--bb-tabs-chrome-item-height: 26px;
}

.tabs-chrome > .tabs-header .nav-link-bar .nav-link-bar-button, .tabs-capsule > .tabs-header .nav-link-bar .nav-link-bar-button {
padding: 0 .5rem;
}

.tabs-chrome > .tabs-header .of .tabs-nav-scroll, .tabs-chrome > .tabs-header .extend .tabs-nav-scroll {
margin-inline-start: -0.75rem;
}
2 changes: 1 addition & 1 deletion src/BootstrapBlazor/BootstrapBlazor.csproj
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
<Project Sdk="Microsoft.NET.Sdk.Razor">

<PropertyGroup>
<Version>9.5.3-beta01</Version>
<Version>9.5.3</Version>
</PropertyGroup>

<ItemGroup>
Expand Down
30 changes: 27 additions & 3 deletions src/BootstrapBlazor/Components/Tab/Tab.razor.scss
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@
.tabs-nav-wrap {
> .nav-link-bar {
cursor: pointer;
font-size: 1rem;
font-size: var(--bb-font-size);
Copy link
Copy Markdown
Contributor

@sourcery-ai sourcery-ai Bot Apr 4, 2025

Choose a reason for hiding this comment

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

suggestion: Consider adding a fallback for --bb-font-size.

Using a CSS variable improves configurability, but adding a fallback like var(--bb-font-size, 1rem) can prevent potential issues if the variable is not defined.

Suggested change
font-size: var(--bb-font-size);
font-size: var(--bb-font-size, 1rem);

color: var(--bs-body-color);
width: var(--bb-tabs-bar-width);
height: 100%;
Expand Down Expand Up @@ -582,6 +582,16 @@
.tabs-chrome > .tabs-header {
padding: var(--bb-tabs-header-padding);

.nav-link-bar-button:hover {
z-index: 2;
}

.of, .extend {
.tabs-nav-scroll {
margin-inline-start: -1rem;
}
}

.tabs-item-wrap {
overflow: hidden;
position: relative;
Expand All @@ -603,7 +613,7 @@
}

&:not(:first-child) {
margin-left: calc(0px - 2 * var(--bb-tabs-item-wrap-padding-x));
margin-inline-start: calc(0px - 2 * var(--bb-tabs-item-wrap-padding-x));
}

&:not(.active) {
Expand Down Expand Up @@ -633,6 +643,14 @@
}
}

&:hover:not(.active) {
z-index: 5;

.tab-corner {
display: none;
}
}

.tabs-item {
border: none !important;
border-top-left-radius: 10px;
Expand Down Expand Up @@ -682,8 +700,14 @@
}

.tabs-capsule > .tabs-header {
.of, .extend {
.tabs-nav-scroll {
margin-inline-start: -0.5rem;
}
}

.tabs-item-wrap {
margin-left: calc(var(--bb-tabs-item-wrap-padding-x) / 2);
margin-inline-start: calc(var(--bb-tabs-item-wrap-padding-x) / 2);

&.active {
.tabs-item {
Expand Down
2 changes: 1 addition & 1 deletion src/BootstrapBlazor/wwwroot/scss/theme/bootstrapblazor.scss
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -603,7 +603,7 @@ $bb-switch-inner-off-text-color: #606266;
$bb-switch-label-margin-left: 10px;

// Tab
$bb-tabs-item-padding: 0 1rem;
$bb-tabs-item-padding: 0 .5rem;
$bb-tabs-item-height: 40px;
$bb-tabs-item-active-color: #409eff;
$bb-tabs-item-hover-color: #409eff;
Expand Down