Bug 93691 - Web Inspector: metrics, geolocation, orientation overrides do not belong to the settings panel
Summary: Web Inspector: metrics, geolocation, orientation overrides do not belong to t...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Alexander Pavlov (apavlov)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-08-09 23:32 PDT by Pavel Feldman
Modified: 2012-11-16 00:17 PST (History)
11 users (show)

See Also:


Attachments
[IMAGE] Screenshot of the suggested Overrides drawer UI (39.04 KB, image/png)
2012-10-15 07:44 PDT, Alexander Pavlov (apavlov)
no flags Details
[IMAGE] Updated screenshot with the new button glyph (96.84 KB, image/png)
2012-10-18 02:19 PDT, Alexander Pavlov (apavlov)
no flags Details
Patch (70.96 KB, patch)
2012-10-30 07:16 PDT, Alexander Pavlov (apavlov)
no flags Details | Formatted Diff | Diff
[IMAGE] Soft menu popup containing the "Overrides" and "Settings" items (in place of the long-press button) (80.18 KB, image/png)
2012-10-30 08:29 PDT, Alexander Pavlov (apavlov)
no flags Details
[IMAGE] Screenshot with the latest patch (67.72 KB, image/jpeg)
2012-10-30 09:26 PDT, Alexander Pavlov (apavlov)
no flags Details
[IMAGE] Settings redesign proposal (67.25 KB, image/jpeg)
2012-10-30 09:28 PDT, Alexander Pavlov (apavlov)
no flags Details
Patch (85.43 KB, patch)
2012-11-06 04:52 PST, Alexander Pavlov (apavlov)
no flags Details | Formatted Diff | Diff
[IMAGE] Screenshot of the Overrides drawer view with patch applied (101.15 KB, image/png)
2012-11-06 04:57 PST, Alexander Pavlov (apavlov)
no flags Details
[IMAGE] Settings dialog with patch applied (83.30 KB, image/png)
2012-11-06 04:58 PST, Alexander Pavlov (apavlov)
no flags Details
[PATCH] Fixed Settings/Overrides UI on the Mac platform (86.57 KB, patch)
2012-11-14 05:59 PST, Alexander Pavlov (apavlov)
no flags Details | Formatted Diff | Diff
[PATCH] Settings covers the entire DevTools area (86.52 KB, patch)
2012-11-15 04:33 PST, Alexander Pavlov (apavlov)
no flags Details | Formatted Diff | Diff
[IMAGE] Screenshot of the Settings dialog covering the entire DevTools contents (67.21 KB, image/png)
2012-11-15 04:34 PST, Alexander Pavlov (apavlov)
no flags Details
Mockup for a dialog-box style for settings (152.76 KB, image/png)
2012-11-15 05:08 PST, dubroy
no flags Details
Mockup for dialog-box style on top of a dark overlay (161.65 KB, image/png)
2012-11-15 05:09 PST, dubroy
no flags Details
[PATCH] Final variation with adjusted Settings layout (87.92 KB, patch)
2012-11-15 23:37 PST, Alexander Pavlov (apavlov)
vsevik: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Pavel Feldman 2012-08-09 23:32:57 PDT
It is very painful to go back and forth between settings and rest of the inspector for the metrics / orientation / location overrides. I think we should move these controls into the drawer (a-la local modifications). Gear icon could contain a submenu linking to Settings / Shortcuts / Overrides and the last one would open the overrides drawer.
Comment 1 Vivek Galatage 2012-08-13 12:58:32 PDT
As discussed over IRC, I thought of recording the discussion here in the bug.

I was exploring the device metrics override settings and found that the settings are applicable through all the pages. But this should be made per page/tab and should be restored upon inspector opening for a particular page only.

Would it make sense to have some of the inspector settings applicable on global scope and some to only local scope? 

Also I was thinking of design approach where it would allow the settings to be categorized as global vs local by providing some interface hooks which then can be integrated with browser vendors "Settings"(in case of chrome, it would be browser settings).
Comment 2 Alexander Pavlov (apavlov) 2012-10-12 08:30:01 PDT
Chromium bug: http://code.google.com/p/chromium/issues/detail?id=137873
Comment 3 Alexander Pavlov (apavlov) 2012-10-15 07:44:56 PDT
Created attachment 168710 [details]
[IMAGE] Screenshot of the suggested Overrides drawer UI
Comment 4 Alexander Pavlov (apavlov) 2012-10-18 02:19:00 PDT
Created attachment 169373 [details]
[IMAGE] Updated screenshot with the new button glyph
Comment 5 Alexander Pavlov (apavlov) 2012-10-30 07:16:15 PDT
Created attachment 171446 [details]
Patch
Comment 6 Alexander Pavlov (apavlov) 2012-10-30 08:29:30 PDT
Created attachment 171458 [details]
[IMAGE] Soft menu popup containing the "Overrides" and "Settings" items (in place of the long-press button)
Comment 7 Alexander Pavlov (apavlov) 2012-10-30 09:26:39 PDT
Created attachment 171466 [details]
[IMAGE] Screenshot with the latest patch
Comment 8 Alexander Pavlov (apavlov) 2012-10-30 09:28:07 PDT
Created attachment 171468 [details]
[IMAGE] Settings redesign proposal

This way we'll be able solve a number of issues and won't need the Settings popup menu
Comment 9 Vivek Galatage 2012-10-30 09:45:10 PDT
(In reply to comment #8)
> Created an attachment (id=171468) [details]
> [IMAGE] Settings redesign proposal
> 
> This way we'll be able solve a number of issues and won't need the Settings popup menu

+1 for the above screenshot. It looks very polished and everything is accessible in convenient manner without any popups.
Comment 10 Konrad Piascik 2012-10-30 10:34:25 PDT
Comment on attachment 171446 [details]
Patch

Agree with Vivek +1 on screenshot
Also I know that there's some changes here to the code moved out of SettingsScreen.js and it would be easier to review/see what was changed if you provided patch which only includes a diff of changes that were made to the override code.  This patch will be a merge of 2 diffs, the removal of code from SettingsScreen/creation of the new File + changes made to the moved code)
Comment 11 Alexander Pavlov (apavlov) 2012-10-30 11:56:52 PDT
(In reply to comment #10)
> (From update of attachment 171446 [details])
> Agree with Vivek +1 on screenshot
> Also I know that there's some changes here to the code moved out of SettingsScreen.js and it would be easier to review/see what was changed if you provided patch which only includes a diff of changes that were made to the override code.  This patch will be a merge of 2 diffs, the removal of code from SettingsScreen/creation of the new File + changes made to the moved code)

This screenshot implies that the extraction of the Overrides tab as a separate drawer view should be reverted. It will remain a "tab" in the Settings, which will be converted into a drawer view.
Comment 12 Paul Irish 2012-10-31 15:06:27 PDT
There's a UI inconsistency here that's bugging me though I'm not entirely sure what the right solution is.

* The console drawer has an icon on the left. It works on all panels (but Console)
* Settings and Overrides drawer have a menu from the same bar's settings cog trigger. They work on all panels.
* Search All Sources drawer has no UI trigger. (Starting it forces the sources tab visibility)
* Local Modifications drawer is unique to sources panel. There is no UI to reveal it other than a context menu item.

The drawer title and close icon placed at the end of the the status bar seems awkward, especially on Elements.
 I wonder if we're missing a more intuitive solution.
Comment 13 Pavel Feldman 2012-11-02 06:48:54 PDT
Comment on attachment 171446 [details]
Patch

Clearing r? as agreed offline.
Comment 14 Konrad Piascik 2012-11-02 06:52:07 PDT
(In reply to comment #13)
> (From update of attachment 171446 [details])
> Clearing r? as agreed offline.

Can you provide a summary of the offline discussion for clarification?
Comment 15 Alexander Pavlov (apavlov) 2012-11-06 04:52:42 PST
Created attachment 172554 [details]
Patch
Comment 16 Alexander Pavlov (apavlov) 2012-11-06 04:57:33 PST
Created attachment 172555 [details]
[IMAGE] Screenshot of the Overrides drawer view with patch applied
Comment 17 Alexander Pavlov (apavlov) 2012-11-06 04:58:46 PST
Created attachment 172556 [details]
[IMAGE] Settings dialog with patch applied
Comment 18 Alexander Pavlov (apavlov) 2012-11-14 05:59:25 PST
Created attachment 174141 [details]
[PATCH] Fixed Settings/Overrides UI on the Mac platform
Comment 19 Alexander Pavlov (apavlov) 2012-11-14 07:25:49 PST
(In reply to comment #14)
> (In reply to comment #13)
> > (From update of attachment 171446 [details] [details])
> > Clearing r? as agreed offline.
> 
> Can you provide a summary of the offline discussion for clarification?

No special offline discussion, just polishing the design and making sure it looks good on Mac.

So, we are aware of the drawer view statusbar element being close to the breadcrumbs in the Elements panel, but this issue is orthogonal to the separation of the Settings and Overrides, so this can be fixed in under a separate bug.
Comment 20 Alexander Pavlov (apavlov) 2012-11-15 04:33:55 PST
Created attachment 174401 [details]
[PATCH] Settings covers the entire DevTools area
Comment 21 Alexander Pavlov (apavlov) 2012-11-15 04:34:50 PST
Created attachment 174402 [details]
[IMAGE] Screenshot of the Settings dialog covering the entire DevTools contents
Comment 22 dubroy 2012-11-15 05:00:05 PST
This looks a bit weird IMO. I'm trying to think of other ideas...will post some mockups soon.
Comment 23 dubroy 2012-11-15 05:08:12 PST
Created attachment 174408 [details]
Mockup for a dialog-box style for settings
Comment 24 dubroy 2012-11-15 05:09:00 PST
Created attachment 174409 [details]
Mockup for dialog-box style on top of a dark overlay
Comment 25 Alexander Pavlov (apavlov) 2012-11-15 23:37:18 PST
Created attachment 174614 [details]
[PATCH] Final variation with adjusted Settings layout
Comment 26 Alexander Pavlov (apavlov) 2012-11-16 00:17:35 PST
Committed r134911: <http://trac.webkit.org/changeset/134911>