Skip to content

Commit

Permalink
Merge pull request #27150 from brave/contents_margin_for_rounded_corners
Browse files Browse the repository at this point in the history
Don't need to have rounded contents margin in tab-fullscreen
  • Loading branch information
simonhong authored Jan 8, 2025
2 parents 6549441 + 1e52de0 commit 2a00b6d
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions browser/ui/views/frame/brave_browser_view_layout.cc
Original file line number Diff line number Diff line change
Expand Up @@ -266,6 +266,13 @@ void BraveBrowserViewLayout::UpdateContentsContainerInsets(
// Control contents's margin with sidebar & vertical tab state.
gfx::Insets contents_margins = GetContentsMargins();

// Don't need to have additional contents margin for rounded corners
// in tab-initiated fullscreen. Web contents occupies whole screen.
if (IsFullscreenForTab()) {
contents_container_bounds.Inset(contents_margins);
return;
}

// In rounded corners mode, we need to include a little margin so we have
// somewhere to draw the shadow.
int contents_margin_for_rounded_corners =
Expand Down

0 comments on commit 2a00b6d

Please sign in to comment.