New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 618232 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Jul 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Regression: Button text is misaligned when Font size is Very large.

Project Member Reported by sc00335...@techmahindra.com, Jun 8 2016

Issue description

Version: 53.0.2760.0/8426.0.0 (Official Build) dev-channel peach_pit,gnawty,wolf
OS: Chrome OS

What steps will reproduce the problem?
(1) Change Font size to very large in chrome://settings >> Open files app/gallery and try deleting any folder/image >> Observe Delete and cancel buttons

Expected: Even on changing font size button font should be center aligned.
Actual: Instead button text is misaligned after changing font size.

This is a regression issue as it is working fine in 44.0.2403.125/7077.111.0 (Official Build) stable-channel daisy.

Issue is also seen in M50 stable.

@fukino: Please confirm the behavior.
 
Actual_buttons font.png
91.5 KB View Download
Expected_delete buttons.png
54.7 KB View Download
Able to reproduce the issue on Gnawty using chrome version 53.0.2760.0/8426.0.0
Cc: oka@chromium.org
Comfirmed. Thanks!
Project Member

Comment 3 by sheriffbot@chromium.org, Jul 6 2016

Labels: -M-53 M-54 MovedFrom-53
Moving this nonessential bug to the next milestone.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Owner: yamaguchi@chromium.org
Status: Started (was: Assigned)
Project Member

Comment 5 by bugdroid1@chromium.org, Jul 13 2016

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

commit 0bf9254e08db3e93c5a619da6d93743f147366ee
Author: yamaguchi <yamaguchi@chromium.org>
Date: Wed Jul 13 08:03:00 2016

Allow resizing dialog button to prevent a label get out of it.

BUG= 618232 
TEST=manuall confirmed that the view renders as intended

Review-Url: https://codereview.chromium.org/2143873002
Cr-Commit-Position: refs/heads/master@{#405074}

[modify] https://crrev.com/0bf9254e08db3e93c5a619da6d93743f147366ee/ui/file_manager/file_manager/foreground/css/common.css

Status: Fixed (was: Started)
What had happened was that the button text size was changed but the size of the button was fixed (32px). The change above allows resizing the button box when the font is too large, so as to have enough size to align the text to its center.

The setting menu item affecting this issue is entitled "[Web content] - [Font size]". So I am not sure if we should resize the dialog box text and all the labels inside the Files.app view (like filename string beside file icons). In other words, should the dialog box buttons be considered "web contents"?

Project Member

Comment 7 by bugdroid1@chromium.org, Jul 13 2016

Labels: merge-merged-2795
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/0bf9254e08db3e93c5a619da6d93743f147366ee

commit 0bf9254e08db3e93c5a619da6d93743f147366ee
Author: yamaguchi <yamaguchi@chromium.org>
Date: Wed Jul 13 08:03:00 2016

Allow resizing dialog button to prevent a label get out of it.

BUG= 618232 
TEST=manuall confirmed that the view renders as intended

Review-Url: https://codereview.chromium.org/2143873002
Cr-Commit-Position: refs/heads/master@{#405074}

[modify] https://crrev.com/0bf9254e08db3e93c5a619da6d93743f147366ee/ui/file_manager/file_manager/foreground/css/common.css

Status: Verified (was: Fixed)
Chrome OS version 54.0.2824.0/8696.0 samus

Sign in to add a comment