Commit 01a682b9 authored by Regan Hsu's avatar Regan Hsu Committed by Commit Bot

[CrOS Wallpaper] Fix Arabic 'Refresh' and 'Explore' button alignment.

The Arabic string for 'Refresh' changed, causing de-alignment issues.
Space on the right side is reduced to account for a longer string.

Initial -
screenshot: https://screenshot.googleplex.com/YTso9OyFquU

2nd patch -
screenshots:
  Normal Text:
  - https://screenshot.googleplex.com/auLs71d3Ops
  - https://screenshot.googleplex.com/YeMT1v0Wdgy
  Exaggerated long text:
  - https://screenshot.googleplex.com/BSJaTPBnGmb
  - https://screenshot.googleplex.com/Om5iuYQUfT1

Bug: 987195
Change-Id: Ib1de302b0eaab700270a5a1fd46abf9609fd7870
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1850412
Commit-Queue: Regan Hsu <hsuregan@chromium.org>
Reviewed-by: default avatarKyle Horimoto <khorimoto@chromium.org>
Reviewed-by: default avatarXiaoqian Dai <xdai@chromium.org>
Cr-Commit-Position: refs/heads/master@{#704480}
parent fe907800
...@@ -750,6 +750,10 @@ html[dir='rtl'] .top-header-contents .more-options { ...@@ -750,6 +750,10 @@ html[dir='rtl'] .top-header-contents .more-options {
visibility: hidden; /* Need this for correct positioning. */ visibility: hidden; /* Need this for correct positioning. */
} }
#current-wallpaper-info-bar .spacer {
flex: 1;
}
:not(.preview-mode) #current-wallpaper-info-bar.show-info-bar { :not(.preview-mode) #current-wallpaper-info-bar.show-info-bar {
visibility: visible; visibility: visible;
} }
...@@ -800,12 +804,6 @@ html[dir='rtl'] .top-header-contents .more-options { ...@@ -800,12 +804,6 @@ html[dir='rtl'] .top-header-contents .more-options {
display: flex; display: flex;
flex-direction: column; flex-direction: column;
font-size: 12px; font-size: 12px;
position: absolute;
right: 0;
}
html[dir='rtl'] #current-wallpaper-more-options {
right: 420px;
} }
#current-wallpaper-more-options > div { #current-wallpaper-more-options > div {
...@@ -825,7 +823,12 @@ html[dir='rtl'] #current-wallpaper-more-options { ...@@ -825,7 +823,12 @@ html[dir='rtl'] #current-wallpaper-more-options {
} }
#current-wallpaper-more-options .text { #current-wallpaper-more-options .text {
padding-top: 1px; height: 16px;
line-height: 16px;
max-width: 160px;
overflow: hidden;
text-overflow: ellipsis;
white-space: nowrap;
} }
#current-wallpaper-more-options .icon { #current-wallpaper-more-options .icon {
......
...@@ -104,6 +104,7 @@ found in the LICENSE file. ...@@ -104,6 +104,7 @@ found in the LICENSE file.
<div id="current-wallpaper-title"></div> <div id="current-wallpaper-title"></div>
<div id="current-wallpaper-description"></div> <div id="current-wallpaper-description"></div>
</div> </div>
<div class="spacer"></div>
<div id="current-wallpaper-more-options"> <div id="current-wallpaper-more-options">
<div id="refresh"> <div id="refresh">
<div class="icon"></div> <div class="icon"></div>
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment