New issue
Advanced search Search tips

Issue 747052 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Closed: Jul 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Ash "Exiting session" dialog spacing is too cramped

Project Member Reported by jamescook@chromium.org, Jul 20 2017

Issue description

chrome fc6bc93f0cb43c4d78b176a14297cfb51b2ed166 r488266

* go/dmstug to set up policy server
* enable ShowLogoutButtonInTray pref
* Sign in, then click "Sign out" button in shelf

Dialog spacing is vertically cramped. Maybe harmony dialog related? Maybe always broken?

Code is in ash/system/session/logout_confirmation_dialog.cc. Probably it would be easier to hack the code to show the button or dialog than to set up the policy server.

 
exiting-session-dialog.png
106 KB View Download

Comment 1 by est...@chromium.org, Jul 20 2017

Owner: est...@chromium.org
Status: Started (was: Untriaged)
easy hack to make this show:

void LogoutConfirmationController::OnLastWindowClosed() {/*
  if (Shell::Get()->session_controller()->login_status() !=
      LoginStatus::PUBLIC) {
    return;
  }
*/

(then close all windows)
Project Member

Comment 2 by bugdroid1@chromium.org, Jul 21 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/05fa15942cc9d70da0e54cce3fcad809fce7b930

commit 05fa15942cc9d70da0e54cce3fcad809fce7b930
Author: Evan Stade <estade@chromium.org>
Date: Fri Jul 21 23:42:25 2017

Fix layout of ash logout confirmation dialog.

Bug:  747052 
Change-Id: I4312c56fc27209167437ba8188d1e2cbd5930565
Reviewed-on: https://chromium-review.googlesource.com/580359
Commit-Queue: Evan Stade <estade@chromium.org>
Reviewed-by: James Cook <jamescook@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488802}
[modify] https://crrev.com/05fa15942cc9d70da0e54cce3fcad809fce7b930/ash/system/session/logout_confirmation_dialog.cc

Comment 3 by est...@chromium.org, Jul 22 2017

Status: Fixed (was: Started)

Comment 4 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment