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: do not close tab item when OnCloseTabItemAsync return false #488

Merged
merged 17 commits into from
Feb 6, 2023

Conversation

ArgoZhang
Copy link
Collaborator

do not close tab item when OnCloseTabItemAsync return false

Description

close #487

@bb-auto bb-auto bot added the enhancement New feature or request label Feb 6, 2023
@bb-auto bb-auto bot added this to the V7.4.0 milestone Feb 6, 2023
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ ArgoZhang
❌ flash123456789
You have signed the CLA already but the status is still pending? Let us recheck it.

@codecov
Copy link

codecov bot commented Feb 6, 2023

Codecov Report

Merging #488 (4c5d28d) into main (dc6a4c0) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main      #488   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          471       471           
  Lines        14836     14832    -4     
=========================================
- Hits         14836     14832    -4     
Impacted Files Coverage Δ
src/BootstrapBlazor/Components/Tab/Tab.razor.cs 100.00% <100.00%> (ø)
...rc/BootstrapBlazor/Components/Tab/TabLink.razor.cs 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@ArgoZhang ArgoZhang merged commit a29aed8 into main Feb 6, 2023
@ArgoZhang ArgoZhang deleted the pr_3795 branch February 6, 2023 03:07
ArgoZhang added a commit that referenced this pull request Feb 6, 2023
* Tab add  Fun:BeforeCloseTabItemAsync

* Tab update OnCloseTabItemAsync

* refactor: update parameter name

* doc: update xml

* refactor: 更新判断语法

* refactor: 更新判断语法

* refactor: 重构代码

* doc: 删除冗余行

* refactor: 消除警告信息

* refactor: 移除可为空约束

* refactor: 移除 index 自增长逻辑

* refactor: 重构代码

* !3795 feat(#I6CY5T): update OnCloseTabItemAsync parameter on Tab
* Tab update OnCloseTabItemAsync
* Tab add  Fun:BeforeCloseTabItemAsync

* test: add unit test for OnCloseTabItemAsync

* doc: 更新文档注释

---------

Co-authored-by: flash123456789 <918127890@qq.com>
Co-authored-by: 张峻凡 <918127890@qq.com>
ArgoZhang added a commit that referenced this pull request Feb 6, 2023
* Tab add  Fun:BeforeCloseTabItemAsync

* Tab update OnCloseTabItemAsync

* refactor: update parameter name

* doc: update xml

* refactor: 更新判断语法

* refactor: 更新判断语法

* refactor: 重构代码

* doc: 删除冗余行

* refactor: 消除警告信息

* refactor: 移除可为空约束

* refactor: 移除 index 自增长逻辑

* refactor: 重构代码

* !3795 feat(#I6CY5T): update OnCloseTabItemAsync parameter on Tab
* Tab update OnCloseTabItemAsync
* Tab add  Fun:BeforeCloseTabItemAsync

* test: add unit test for OnCloseTabItemAsync

* doc: 更新文档注释

---------

Co-authored-by: flash123456789 <918127890@qq.com>
Co-authored-by: 张峻凡 <918127890@qq.com>
ArgoZhang added a commit that referenced this pull request Feb 6, 2023
* Tab add  Fun:BeforeCloseTabItemAsync

* Tab update OnCloseTabItemAsync

* refactor: update parameter name

* doc: update xml

* refactor: 更新判断语法

* refactor: 更新判断语法

* refactor: 重构代码

* doc: 删除冗余行

* refactor: 消除警告信息

* refactor: 移除可为空约束

* refactor: 移除 index 自增长逻辑

* refactor: 重构代码

* !3795 feat(#I6CY5T): update OnCloseTabItemAsync parameter on Tab
* Tab update OnCloseTabItemAsync
* Tab add  Fun:BeforeCloseTabItemAsync

* test: add unit test for OnCloseTabItemAsync

* doc: 更新文档注释

---------

Co-authored-by: flash123456789 <918127890@qq.com>
Co-authored-by: 张峻凡 <918127890@qq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: do not close tab item when OnCloseTabItemAsync return false
3 participants