Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(139)

Issue 6350071: Added stubs for zoom info panel. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years ago by chudy
Modified:
11 years, 12 months ago
Reviewers:
reed1
CC:
skia-review_googlegroups.com
Base URL:
https://skia.googlecode.com/svn/trunk
Visibility:
Public.

Description

Added stubs for zoom info panel.

Patch Set 1 #

Patch Set 2 : Connected Scale Factor to info box #

Patch Set 3 : Finishing up SkSettings Widget #

Unified diffs Side-by-side diffs Delta from patch set Stats (+445 lines, -250 lines) Patch
M debugger/QT/SkCanvasWidget.h View 1 2 2 chunks +5 lines, -1 line 0 comments Download
M debugger/QT/SkCanvasWidget.cpp View 1 2 6 chunks +10 lines, -6 lines 0 comments Download
M debugger/QT/SkDebuggerGUI.h View 1 2 4 chunks +15 lines, -3 lines 0 comments Download
M debugger/QT/SkDebuggerGUI.cpp View 1 2 21 chunks +134 lines, -111 lines 0 comments Download
M debugger/QT/SkSettingsWidget.h View 1 2 2 chunks +34 lines, -15 lines 0 comments Download
M debugger/QT/SkSettingsWidget.cpp View 1 2 1 chunk +105 lines, -60 lines 0 comments Download
M debugger/QT/moc_SkCanvasWidget.cpp View 1 2 3 chunks +32 lines, -4 lines 0 comments Download
M debugger/QT/moc_SkDebuggerGUI.cpp View 1 2 3 chunks +62 lines, -45 lines 0 comments Download
M debugger/QT/moc_SkInspectorWidget.cpp View 1 2 1 chunk +1 line, -1 line 0 comments Download
M debugger/QT/moc_SkSettingsWidget.cpp View 1 2 3 chunks +47 lines, -4 lines 0 comments Download

Messages

Total messages: 4
chudy
12 years ago (2012-07-03 20:53:31 UTC) #1
chudy
Sending to Mike until Derek comes back
12 years ago (2012-07-03 22:40:29 UTC) #2
reed1
fine in general (like I know anything about Qt). Does the scaleFactor need to be ...
11 years, 12 months ago (2012-07-09 14:01:08 UTC) #3
chudy
11 years, 12 months ago (2012-07-09 17:54:41 UTC) #4
Changed the int to a float for future potential. Hooked up the rest of the
widgets in SkSettings. 

Am now branching off into multiple CL's for features and bug fixes. Had been
working on this branch before.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b