Skip to content

Commit

Permalink
Merge branch 'dominik/ntp-search-bar' into dominik/ntp-search-bar-shi…
Browse files Browse the repository at this point in the history
…p-review
  • Loading branch information
ayoy committed Oct 21, 2024
2 parents d70c72b + f0f60a5 commit b9bd82c
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 5 deletions.
4 changes: 2 additions & 2 deletions DuckDuckGo/HomePage/View/FavoritesView.swift
Original file line number Diff line number Diff line change
Expand Up @@ -34,15 +34,15 @@ struct Favorites: View {
var body: some View {

if #available(macOS 12.0, *) {
LazyVStack(spacing: 4) {
LazyVStack(spacing: 0) {
FavoritesGrid(isHovering: $isHovering)
}
.frame(maxWidth: .infinity)
.onHover { isHovering in
self.isHovering = isHovering
}
} else {
VStack(spacing: 4) {
VStack(spacing: 0) {
FavoritesGrid(isHovering: $isHovering)
}
.frame(maxWidth: .infinity)
Expand Down
6 changes: 3 additions & 3 deletions DuckDuckGo/HomePage/View/HomePageView.swift
Original file line number Diff line number Diff line change
Expand Up @@ -368,7 +368,7 @@ fileprivate extension HomePage.Views.RootView {
.visibility(model.isRecentActivityVisible ? .visible : .gone)
}
.padding(.vertical, Const.itemSpacing)
.frame(height: totalHeight(with: geometry))
.frame(width: Self.targetWidth, height: totalHeight(with: geometry))
.offset(y: -centeredViewVerticalOffset(with: geometry))

VStack(spacing: 0) {
Expand All @@ -379,7 +379,7 @@ fileprivate extension HomePage.Views.RootView {
}
.frame(width: Self.targetWidth)
}
.frame(width: Self.targetWidth, height: max(geometry.size.height, totalHeight(with: geometry)))
.frame(height: max(geometry.size.height, totalHeight(with: geometry)))
.frame(maxWidth: .infinity)
.onPreferenceChange(RemoteMessageHeightPreferenceKey.self) { value in
remoteMessageHeight = value
Expand Down Expand Up @@ -423,7 +423,7 @@ fileprivate extension HomePage.Views.RootView {
height += Const.itemSpacing + continueSetUpCardsTopPadding
}
if model.isFavoriteVisible {
height += 126 + Const.itemSpacing
height += 122 + Const.itemSpacing
}
if model.isRecentActivityVisible {
height += 90 + Const.itemSpacing
Expand Down
1 change: 1 addition & 0 deletions DuckDuckGo/HomePage/View/MoreOrLessView.swift
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@ extension HomePage.Views {
.foregroundColor((.homePageMoreOrLessText))
}.frame(maxWidth: .infinity)
}
.padding(.horizontal, 2)
.padding(.top, 8)
.font(.system(size: 11))
.link {
Expand Down
1 change: 1 addition & 0 deletions DuckDuckGo/HomePage/View/RecentlyVisitedView.swift
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,7 @@ struct RecentlyVisited: View {
}
.padding(.vertical, 24)
}
.padding(.horizontal, 2)
.padding(.bottom, 24)

}
Expand Down

0 comments on commit b9bd82c

Please sign in to comment.