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

Unified Diff: chrome/browser/resources/file_manager/js/file_manager.js

Issue 10258011: [File Manager] Better Google Drive promo layout in Open/Save dialogs (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 8 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/file_manager/js/file_manager.js
diff --git a/chrome/browser/resources/file_manager/js/file_manager.js b/chrome/browser/resources/file_manager/js/file_manager.js
index f1fdc9a7de18185fba384f275f9e0ecd40638db1..b3b04aa32abf0ea2d00d41ccee21dbffc48fa261 100644
--- a/chrome/browser/resources/file_manager/js/file_manager.js
+++ b/chrome/browser/resources/file_manager/js/file_manager.js
@@ -752,9 +752,8 @@ FileManager.prototype = {
var learnMore = this.document_.createElement('div');
learnMore.className = 'gdata learn-more plain-link';
learnMore.textContent = str('GDATA_LEARN_MORE');
- learnMore.addEventListener('click', function() {
- chrome.tabs.create({url: GOOGLE_DRIVE_ERROR_HELP_URL});
- });
+ learnMore.addEventListener('click',
+ this.onExternalLinkClick_.bind(this, GOOGLE_DRIVE_ERROR_HELP_URL));
this.unmountedPanel_.appendChild(learnMore);
};
@@ -2420,8 +2419,8 @@ FileManager.prototype = {
return chrome.extension.getURL('').split('/')[2];
};
- FileManager.prototype.onDownloadsWarningClick_ = function(event) {
- chrome.tabs.create({url: DOWNLOADS_FAQ_URL});
+ FileManager.prototype.onExternalLinkClick_ = function(url) {
+ chrome.tabs.create({url: url});
if (this.dialogType_ != FileManager.DialogType.FULL_PAGE) {
this.onCancel_();
}
@@ -3304,7 +3303,8 @@ FileManager.prototype = {
var html = util.htmlUnescape(str('DOWNLOADS_DIRECTORY_WARNING'));
box.lastElementChild.innerHTML = html;
var link = box.querySelector('a');
- link.addEventListener('click', this.onDownloadsWarningClick_.bind(this));
+ link.addEventListener('click',
+ this.onExternalLinkClick_.bind(this, DOWNLOADS_FAQ_URL));
} else {
box.lastElementChild.innerHTML = '';
}
@@ -3409,8 +3409,11 @@ FileManager.prototype = {
if (event.newDirEntry.unmounted)
this.dialogContainer_.setAttribute('unmounted', true);
- else
+ else {
this.dialogContainer_.removeAttribute('unmounted');
+ // Need to resize explicitly because the list container had display:none.
+ this.onResize_();
+ }
if (this.isOnGData()) {
this.dialogContainer_.setAttribute('gdata', true);
@@ -4318,9 +4321,8 @@ FileManager.prototype = {
var more = createDiv('gdrive-welcome-more plain-link', links);
more.textContent = str('GDATA_LEARN_MORE');
- more.addEventListener('click', function() {
- chrome.tabs.create({url: GOOGLE_DRIVE_FAQ_URL});
- });
+ more.addEventListener('click',
+ self.onExternalLinkClick_.bind(self, GOOGLE_DRIVE_FAQ_URL));
var dismiss = createDiv('gdrive-welcome-dismiss plain-link', links);
dismiss.textContent = str('GDATA_WELCOME_DISMISS');
« no previous file with comments | « chrome/browser/resources/file_manager/harness.html ('k') | chrome/browser/resources/file_manager/js/mock_chrome.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698