Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(51)

Issue 113977: linux: wrap prefs checkbox in options window. (Closed)

Created:
11 years, 7 months ago by Evan Martin
Modified:
9 years, 7 months ago
Reviewers:
Evan Stade
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

linux: wrap prefs checkbox in options window. This is a quick hack, that will be replaced once we have the rest of prefs implemented. BUG=12736

Patch Set 1 #

Total comments: 5

Patch Set 2 : rename #

Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -2 lines) Patch
M chrome/browser/gtk/options/options_window_gtk.cc View 1 2 chunks +10 lines, -2 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Evan Martin
11 years, 7 months ago (2009-05-28 23:16:41 UTC) #1
Evan Stade
http://codereview.chromium.org/113977/diff/1/2 File chrome/browser/gtk/options/options_window_gtk.cc (right): http://codereview.chromium.org/113977/diff/1/2#newcode125 Line 125: GtkWidget* label = gtk_label_new( |metrics_label| perhaps http://codereview.chromium.org/113977/diff/1/2#newcode129 Line ...
11 years, 7 months ago (2009-05-28 23:58:38 UTC) #2
Evan Martin
http://codereview.chromium.org/113977/diff/1/2 File chrome/browser/gtk/options/options_window_gtk.cc (right): http://codereview.chromium.org/113977/diff/1/2#newcode125 Line 125: GtkWidget* label = gtk_label_new( On 2009/05/28 23:58:39, Evan ...
11 years, 7 months ago (2009-05-29 00:02:56 UTC) #3
Evan Stade
11 years, 7 months ago (2009-05-29 00:15:23 UTC) #4
We can revisit this later, when we fill out the rest of the options input. For
now this LGTM.

http://codereview.chromium.org/113977/diff/1/2
File chrome/browser/gtk/options/options_window_gtk.cc (right):

http://codereview.chromium.org/113977/diff/1/2#newcode129
Line 129: gtk_widget_set_size_request(label, 475, -1);
> > won't this make it unreadable if you have a WM that ignores the minimum
size?

sorry, I shouldn't have posed this as a question. I tested, and it does make it
unreadable.

> GTK labels don't wrap unless you explicitly set a max width on them.

In my test, it did wrap without setting an explicit max length, but it wrapped
way early and ended up with 2 approximately even length lines.

Powered by Google App Engine
This is Rietveld 408576698