Skip to content

chromium: Reduce minimum browser window width to 480px #698

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.

Sign up for GitHub

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

chromium: Reduce minimum browser window width to 480px #698

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions meta-chromium/recipes-browser/chromium/chromium-gn.inc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ SRC_URI += " \
file://0013-Fix-html_minifier-script-for-node-v12.patch \
file://0015-drop-gsimple-template-names.patch \
file://0016-cast-to-bool-to-avoid-constexpr-error.patch \
file://0017-Reduce-minimum-browser-window-width-to-480px.patch \
"

SRC_URI:append:libc-musl = "\
Expand Down
37 changes: 37 additions & 0 deletions ...um/recipes-browser/chromium/files/0017-Reduce-minimum-browser-window-width-to-480px.patch
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,37 @@
From 567930b5887c80f666433ba78810cab37cc1f680 Mon Sep 17 00:00:00 2001
From: Marek Vasut <[email protected]>
Date: Wed, 1 Mar 2023 23:54:47 +0100
Subject: [PATCH] Reduce minimum browser window width to 480px

The TST043015CNHX panel is a 480x272 panel. Chromium browser limits the
minimum window width to 500px since commits:

6b0ecc1e4532c ("Set lower bound on contents pane size.")
580d6900dc7fc ("In tab dragging, the window size should be larger than its minimum size.")

To make chromium browser usable on those tiny panels as well, reduce
the minimum window width to 480px. The upstreaming of this change is
in progress.

Upstream-Status: Pending
Signed-off-by: Marek Vasut <[email protected]>
---
chrome/browser/ui/views/frame/browser_view_layout.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/chrome/browser/ui/views/frame/browser_view_layout.h b/chrome/browser/ui/views/frame/browser_view_layout.h
index 9fc925e06e5e9..9346f321476c3 100644
--- a/chrome/browser/ui/views/frame/browser_view_layout.h
+++ b/chrome/browser/ui/views/frame/browser_view_layout.h
@@ -48,7 +48,7 @@ class BrowserViewLayout : public views::LayoutManager {
// very small window, even on large monitors (which is why a minimum height is
// not specified). This value is used for the main browser window only, not
// for popups.
- static constexpr int kMainBrowserContentsMinimumWidth = 500;
+ static constexpr int kMainBrowserContentsMinimumWidth = 480;

// |browser_view| may be null in tests.
BrowserViewLayout(std::unique_ptr<BrowserViewLayoutDelegate> delegate,
--
2.39.2