Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2298)

Unified Diff: chrome/browser/resources/extensions/extensions.html

Issue 462273003: Show extension name for dependent extensions (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: use template Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/extensions/extensions.html
diff --git a/chrome/browser/resources/extensions/extensions.html b/chrome/browser/resources/extensions/extensions.html
index 25dc245a26ebf406fb01bac7b314ff7e6fed4d3f..f34e7399c8ce267985ffb260113c2834d0defb82 100644
--- a/chrome/browser/resources/extensions/extensions.html
+++ b/chrome/browser/resources/extensions/extensions.html
@@ -252,6 +252,20 @@
</div>
+<div id="template-dependent" hidden>
not at google - send to devlin 2014/08/13 19:43:47 I'm not at all familiar with the idioms of this fi
elijahtaylor1 2014/08/13 21:12:46 Done.
+ <li class="dependent-list-item">
elijahtaylor1 2014/08/13 18:02:59 I wasn't sure if I should continue to use <li> or
not at google - send to devlin 2014/08/13 19:43:47 <li> looks right, but having <divs> inside <li> is
elijahtaylor1 2014/08/13 21:12:46 I got rid of the divs and used an embedded <ul> to
+ <div>
+ <span class="extension-title"></span>
+ </div>
+ <div class="developer-extras">
+ <div>
+ <span i18n-content="extensionSettingsExtensionId"></span>
+ <span class="extension-id"></span>
+ </div>
+ </div>
+ </li>
+</div>
+
<script src="chrome://extensions-frame/strings.js"></script>
<script src="chrome://resources/js/i18n_template2.js"></script>

Powered by Google App Engine
This is Rietveld 408576698