Summary: | Web Inspector: [regression] Cookies view does not allow columns resize. | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Pavel Feldman <pfeldman> | ||||
Component: | Web Inspector (Deprecated) | Assignee: | Pavel Feldman <pfeldman> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | apavlov, bweinstein, commit-queue, joepeck, keishi, loislo, pfeldman, pmuellr, rik, timothy, yurys | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Attachments: |
|
Description
Pavel Feldman
2011-01-04 09:36:01 PST
Created attachment 77895 [details]
[PATCH] Proposed change
Comment on attachment 77895 [details]
[PATCH] Proposed change
When did this regress?
Comment on attachment 77895 [details]
[PATCH] Proposed change
All updates go through this function, so this might work on updates work, What problem is this fixing? Shouldn't the resize event on the datagrid already handle this? I see one defined on the simple cookie table, but not the advanced. I see the datagrid fires a "width changed" event on resize changes, that might be good to tap into. I'm in a car so I can't quite fully verify :(.
> All updates go through this function, so this might work on updates work, What problem is this fixing? Shouldn't the resize event on the datagrid already handle this? I see one defined on the simple cookie table, but not the advanced. I see the datagrid fires a "width changed" event on resize changes, that might be good to tap into. I'm in a car so I can't quite fully verify :(.
Ok, to repro:
1. pull up
2. stop engine
3. run webkit nightly
4. try resizing cookies table columns :)
Comment on attachment 77895 [details] [PATCH] Proposed change Clearing flags on attachment: 77895 Committed r75489: <http://trac.webkit.org/changeset/75489> All reviewed patches have been landed. Closing bug. |