Skip to content

CA-5732: Update Tab Management Appearance#220

Merged
davertay-j merged 25 commits intomainfrom
tiffner/CA-5732
Mar 12, 2026
Merged

CA-5732: Update Tab Management Appearance#220
davertay-j merged 25 commits intomainfrom
tiffner/CA-5732

Conversation

@Phoenix7351
Copy link
Copy Markdown
Contributor

@Phoenix7351 Phoenix7351 commented Mar 10, 2026

ScreenRecording_03-10-2026.15-01-01_1.MP4

@Phoenix7351 Phoenix7351 added the minor Changes that should bump the MINOR version number label Mar 10, 2026
…/CA-5732

# Conflicts:
#	lib/Sources/App/WebContainerView.swift
#	lib/Sources/Settings/v2/SettingsViewV2.swift
@Phoenix7351 Phoenix7351 changed the title Tiffner/ca 5732 CA-5732: Update Tab Management Appearance Mar 10, 2026
@Phoenix7351 Phoenix7351 marked this pull request as ready for review March 10, 2026 20:09
Copy link
Copy Markdown
Collaborator

@davertay-j davertay-j left a comment

Choose a reason for hiding this comment

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

Functionally looks good, but some misalignment on the use of existing model behavior that could be improved.

Comment thread lib/Sources/Tabs/TabGridView.swift Outdated
Comment thread lib/Sources/Settings/SettingsModel.swift Outdated
Comment thread lib/Sources/Design/Extensions/View+EmbedInNavigationBackground.swift Outdated
Comment thread lib/Sources/App/FreshPageView.swift Outdated
Comment thread lib/Sources/App/WebContainerView.swift Outdated
Comment thread lib/Sources/Settings/v2/SettingsViewV2.swift Outdated
Comment thread lib/Sources/Settings/v2/SettingsViewV2.swift Outdated
@Phoenix7351 Phoenix7351 requested a review from davertay-j March 11, 2026 20:08
Copy link
Copy Markdown
Collaborator

@davertay-j davertay-j left a comment

Choose a reason for hiding this comment

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

Thanks for refactoring the background to be part of the setting view itself - much cleaner.

@davertay-j davertay-j merged commit d250052 into main Mar 12, 2026
3 checks passed
@davertay-j davertay-j deleted the tiffner/CA-5732 branch March 12, 2026 17:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

minor Changes that should bump the MINOR version number

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants