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

Issue 896515 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Oct 29
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 2
Type: Bug



Sign in to add a comment

Change "Themes & Wallpapers" string in Sync/Unity settings to "Themes" everywhere but ChromeOS

Project Member Reported by yyushkina@chromium.org, Oct 18

Issue description

If there are no wallpapers (e.g. everywhere but ChromeOS), the string shouldn't mention them.

Change "Themes & Wallpapers" string in Sync/Unity settings to "Themes" everywhere but ChromeOS.

Sabine/Marc - please triage.
 
Cc: droger@chromium.org msarda@chromium.org
Adding some Unity/signin folks - I'm curious if this string was a conscious decision?
Owner: ew...@chromium.org
Assigning to Eli to answer in comment #1
We just carried over the strings from the old sync settings. Nothing changed with Unity. So no, not a conscious decision.
Owner: tangltom@chromium.org
Status: Assigned (was: Available)
Assigning to Thomas to fix the string as this is Unity work.
Status: Started (was: Assigned)
Project Member

Comment 6 by bugdroid1@chromium.org, Oct 29

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

commit 2d98a61051050e6ea95e9724baa182eb6575b7ff
Author: Thomas Tangl <tangltom@chromium.org>
Date: Mon Oct 29 11:36:18 2018

Rename sync string for Themes on non-ChromeOS

Bug:  896515 
Change-Id: Idd45447bd092e934286f0b574782b92caa61a051
Reviewed-on: https://chromium-review.googlesource.com/c/1301517
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Commit-Queue: Thomas Tangl <tangltom@chromium.org>
Cr-Commit-Position: refs/heads/master@{#603458}
[modify] https://crrev.com/2d98a61051050e6ea95e9724baa182eb6575b7ff/chrome/app/settings_strings.grdp

Status: Fixed (was: Started)

Sign in to add a comment