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

Unified Diff: chrome/browser/resources/print_preview/margin_settings.js

Issue 8395006: Print Preview: Hiding header/footer options when 'no margins' is selected (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebasing Created 9 years, 1 month 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/print_preview/margin_settings.js
diff --git a/chrome/browser/resources/print_preview/margin_settings.js b/chrome/browser/resources/print_preview/margin_settings.js
index 7fb82096bf2a02e164552166f9af354ae94b5a77..3e9063986ab8db1c1a27369f3e707982682890c2 100644
--- a/chrome/browser/resources/print_preview/margin_settings.js
+++ b/chrome/browser/resources/print_preview/margin_settings.js
@@ -240,6 +240,13 @@ cr.define('print_preview', function() {
this.customMargins_ = new Margins(-1, -1, -1 , -1);
this.customMargins = lastUsedMarginsSettings;
}
+ this.dispatchMarginsChangedEvent_();
+ },
+
+ dispatchMarginsChangedEvent_: function() {
+ var customEvent = cr.Event(customEvents.MARGINS_SELECTION_CHANGED);
+ customEvent.selectedMargins = this.selectedMarginsValue;
+ document.dispatchEvent(customEvent);
},
/**
@@ -560,6 +567,8 @@ cr.define('print_preview', function() {
* @private
*/
onMarginsChanged_: function() {
+ this.dispatchMarginsChangedEvent_();
+
if (this.isDefaultMarginsSelected() || this.isMinimumMarginsSelected() ||
this.isNoMarginsSelected())
this.onDefaultMinimumNoMarginsSelected_();

Powered by Google App Engine
This is Rietveld 408576698