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

Issue 6802026: Move profile menu button to leave room for full screen button (Closed)

Created:
9 years, 8 months ago by sail
Modified:
9 years, 7 months ago
Reviewers:
Avi (use Gerrit)
CC:
chromium-reviews
Visibility:
Public.

Description

Move profile menu button to leave room for full screen button With this change we now move the profile menu button to the left if a full screen button exists. Here's a screenshot of what this looks like: https://docs.google.com/leaf?id=0B0Odde3V7EhWMmEzOGFhYzktOWE1Ny00NjU2LWE2NDYtZTcwMTRhNmUzMjRi&hl=en when collapsed: https://docs.google.com/leaf?id=0B0Odde3V7EhWOGUxMDlmYzgtZjEyMi00NjdiLTg1YTUtNDE3MmZjZDdhMTFm&hl=en BUG=None TEST=Ran and verified that the profile menu button didn't overlap the fullscreen button. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=80849

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+15 lines, -0 lines) Patch
M chrome/browser/ui/cocoa/tabs/tab_strip_controller.h View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/ui/cocoa/tabs/tab_strip_controller.mm View 1 chunk +14 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
sail
9 years, 8 months ago (2011-04-06 22:33:35 UTC) #1
Avi (use Gerrit)
9 years, 8 months ago (2011-04-06 22:59:23 UTC) #2
LG

Powered by Google App Engine
This is Rietveld 408576698