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

Disable NSToolbar Tabs #25137

Merged
merged 9 commits into from
Nov 5, 2024
Merged

Disable NSToolbar Tabs #25137

merged 9 commits into from
Nov 5, 2024

Conversation

PureWeen
Copy link
Member

@PureWeen PureWeen commented Oct 8, 2024

Description of Change

Backport of this code from main

internal static class TabbedViewExtensions

Issues Fixed

Fixes #23380

@PureWeen PureWeen requested a review from a team as a code owner October 8, 2024 18:23
@PureWeen PureWeen added this to the .NET 8 SR9.2 milestone Oct 8, 2024
@PureWeen PureWeen requested review from drasticactions and jfversluis and removed request for mattleibow and rmarinho October 8, 2024 18:24
@PureWeen
Copy link
Member Author

PureWeen commented Oct 8, 2024

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@PureWeen PureWeen changed the base branch from main to release/8.0.1xx-sr9 October 8, 2024 20:43
@PureWeen
Copy link
Member Author

PureWeen commented Oct 8, 2024

This one is on hold until we merge

#25140

@PureWeen PureWeen marked this pull request as draft October 8, 2024 20:55
@PureWeen
Copy link
Member Author

/backport to net9.0

Copy link
Contributor

Started backporting to net9.0: https://github.com/dotnet/maui/actions/runs/11277901831

@PureWeen PureWeen modified the milestones: .NET 8 SR9.2, .NET 8 SR10 Oct 14, 2024
@PureWeen PureWeen changed the base branch from release/8.0.1xx-sr9 to release/8.0.1xx-sr10 October 14, 2024 21:08
@PureWeen PureWeen self-assigned this Oct 31, 2024
@PureWeen
Copy link
Member Author

PureWeen commented Nov 4, 2024

/rebase

@PureWeen PureWeen marked this pull request as ready for review November 4, 2024 19:42
@jsuarezruiz
Copy link
Contributor

Fixing CI, the changes works as expected.

@PureWeen PureWeen merged commit f86576a into release/8.0.1xx-sr10 Nov 5, 2024
7 checks passed
@PureWeen PureWeen deleted the fix_25136 branch November 5, 2024 20:38
@PureWeen
Copy link
Member Author

PureWeen commented Nov 6, 2024

/backport to main

Copy link
Contributor

github-actions bot commented Nov 6, 2024

Started backporting to main: https://github.com/dotnet/maui/actions/runs/11694739135

Copy link
Contributor

github-actions bot commented Nov 6, 2024

@PureWeen backporting to main failed, the patch most likely resulted in conflicts:

$ git am --3way --empty=keep --ignore-whitespace --keep-non-patch changes.patch

Applying: Disable NSToolbar Tabs
.git/rebase-apply/patch:44: trailing whitespace.
			
warning: 1 line adds whitespace errors.
Using index info to reconstruct a base tree...
M	src/Controls/src/Core/Compatibility/Handlers/Shell/iOS/ShellItemRenderer.cs
M	src/Controls/src/Core/Compatibility/Handlers/TabbedPage/iOS/TabbedRenderer.cs
M	src/Core/src/Platform/iOS/TabbedViewExtensions.cs
Falling back to patching base and 3-way merge...
Auto-merging src/Core/src/Platform/iOS/TabbedViewExtensions.cs
CONFLICT (content): Merge conflict in src/Core/src/Platform/iOS/TabbedViewExtensions.cs
Auto-merging src/Controls/src/Core/Compatibility/Handlers/TabbedPage/iOS/TabbedRenderer.cs
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config advice.mergeConflict false"
Patch failed at 0001 Disable NSToolbar Tabs
Error: The process '/usr/bin/git' failed with exit code 128

Please backport manually!

Copy link
Contributor

github-actions bot commented Nov 6, 2024

@PureWeen an error occurred while backporting to main, please check the run log for details!

Error: git am failed, most likely due to a merge conflict.

PureWeen added a commit that referenced this pull request Nov 6, 2024
* Disable NSToolbar Tabs

* - fix OS check

* - fix tabbedview extentions PR

* - bump targets

* - add missing file

* - add if/def for NET18.0 target

* - add catalyst check

* - switch to newer APIs for iOS and catalyst 18+

* - if/def for iOS18
# Conflicts:
#	src/Core/src/Platform/iOS/TabbedViewExtensions.cs
PureWeen added a commit that referenced this pull request Nov 7, 2024
* Disable NSToolbar Tabs

* - fix OS check

* - fix tabbedview extentions PR

* - bump targets

* - add missing file

* - add if/def for NET18.0 target

* - add catalyst check

* - switch to newer APIs for iOS and catalyst 18+

* - if/def for iOS18
# Conflicts:
#	src/Core/src/Platform/iOS/TabbedViewExtensions.cs
@github-actions github-actions bot locked and limited conversation to collaborators Dec 6, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[BUG] iPadOS/iOS 18 Forced TabView/Tab Bar/Tabbar
7 participants