892937 - [responsive mode] default size should be the 1st preset

Status: RESOLVED FIXED

(DevTools :: Responsive Design Mode, defect)

Comment 1

[Paul Rouget [:paul]]

Attachment: Patch v1

Comment 2

[Heather Arthur [:harth]]

Comment on attachment 774603 [details] [diff] [review]
Patch v1

Review of attachment 774603 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/devtools/responsivedesign/responsivedesign.jsm
@@ +145,5 @@
>  
>      this.customPreset.width = bbox.width - 40; // horizontal padding of the container
>      this.customPreset.height = bbox.height - 80; // vertical padding + toolbar height
>  
> +    this.currentPresetKey = this.presets[1].key; // most common preset

r=me as long as you meant to say 1 instead of 0 (=

Comment 3

[Paul Rouget [:paul]]

(In reply to Heather Arthur [:harth] from comment #2)
> r=me as long as you meant to say 1 instead of 0 (=

I did. (0 is custom preset).

Comment 4

[Paul Rouget [:paul]]

https://hg.mozilla.org/integration/fx-team/rev/131854eced28

Comment 5

[Tim Taubert [:ttaubert] (inactive)]

https://hg.mozilla.org/mozilla-central/rev/131854eced28