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

Issue 659189 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
User never visited
Closed: Jan 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

External link icons incongruous in context of Get Help app

Project Member Reported by morsed@google.com, Oct 25 2016

Issue description

Links with Off-Site Icons

- Triggered by Target=Blank, set in gKMS article links and assigned icon within Get Help app CSS. 

- For Online help, usage makes sense such that off-site icon appears with links for content outside of online help.  But, for Get Help app, meaning of off-site icon is confusing when some links that have it point to online resources outside of Get Help app and some that don’t have it also point to online resources.  

e.g. For Getting online > Set up virtual private networks (VPNs) > VPN apps, see "force install" and "upload a config file" links.  Both are off-site, relative to Get Help, but they don't both have external link icons. 

 

Comment 1 by morsed@google.com, Oct 25 2016

Description: Show this description

Comment 2 by morsed@google.com, Oct 25 2016

Description: Show this description

Comment 3 by morsed@google.com, Nov 15 2016

Status: Started (was: Assigned)
Project Member

Comment 4 by bugdroid1@chromium.org, Nov 18 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/chromeos-assets/+/8dbfd4f980a372545f4900f194342c2cd8fce7d3

commit 8dbfd4f980a372545f4900f194342c2cd8fce7d3
Author: David Morse <morsed@google.com>
Date: Wed Nov 16 00:56:49 2016

Comment 5 by morsed@google.com, Nov 19 2016

Labels: Merge-Request-56
This landed with 9001.0.0/56.0.2923.0/3502314 and I believe it missed the M56 release branch, so now requesting a mergeback to M56.

Comment 6 by dimu@chromium.org, Nov 19 2016

Labels: -Merge-Request-56 Merge-Approved-56 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M56 (branch: 2924)
Project Member

Comment 7 by sheriffbot@chromium.org, Nov 30 2016

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 8 by morsed@google.com, Nov 30 2016

Labels: -Hotlist-Merge-Approved -Merge-Approved-56

Comment 9 by morsed@google.com, Jan 18 2017

Status: Fixed (was: Started)
Status: Verified (was: Fixed)

Sign in to add a comment