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

Issue 101933006: [DevTools] Inspector should allow resetting zoom level using the numpad0 key. (Closed)

Created:
7 years ago by vivekg_samsung
Modified:
7 years ago
Reviewers:
vivekg, apavlov, pfeldman
CC:
blink-reviews, caseq+blink_chromium.org, loislo+blink_chromium.org, eustas+blink_chromium.org, alph+blink_chromium.org, yurys+blink_chromium.org, lushnikov+blink_chromium.org, vsevik+blink_chromium.org, pfeldman+blink_chromium.org, paulirish+reviews_chromium.org, apavlov+blink_chromium.org, devtools-reviews_chromium.org, aandrey+blink_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/blink.git@master
Visibility:
Public.

Description

[DevTools] Inspector should allow resetting zoom level using the numpad0 key. Inspector supports having multiple zoom levels on the frontend. It also supports resetting the zoom level to default using the key: ctrl/cmd+'0'. At the same time the developers can use the numpad '+' and '-' along with ctrl/cmd to increase/decrease zoom level. Currently resetting zoom level to default is not supported for the key: ctrl/cmd+'numpad0'. This patch adds the support for using the numpad0 to reset the zoom level. BUG=326414 Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=163336

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -0 lines) Patch
M Source/devtools/front_end/inspector.js View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
vivekg
Please take a look, thank you!
7 years ago (2013-12-06 04:46:36 UTC) #1
apavlov
lgtm
7 years ago (2013-12-06 06:42:46 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/vivek.vg@samsung.com/101933006/1
7 years ago (2013-12-06 06:43:01 UTC) #3
commit-bot: I haz the power
7 years ago (2013-12-06 13:47:30 UTC) #4
Message was sent while issue was closed.
Change committed as 163336

Powered by Google App Engine
This is Rietveld 408576698