RESOLVED FIXED 218262
Web Inspector: truncate data URLs in the Resources sidebar and Headers panel
https://bugs.webkit.org/show_bug.cgi?id=218262
Summary Web Inspector: truncate data URLs in the Resources sidebar and Headers panel
Devin Rousso
Reported 2020-10-27 14:07:29 PDT
.
Attachments
Patch (3.64 KB, patch)
2020-10-27 14:15 PDT, Devin Rousso
no flags
[Image] after Patch is applied (155.68 KB, image/png)
2020-10-27 14:23 PDT, Devin Rousso
no flags
Devin Rousso
Comment 1 2020-10-27 14:15:36 PDT
Joseph Pecoraro
Comment 2 2020-10-27 14:18:01 PDT
Screenshot? I normally copy/paste from there, does copy still work?
Darin Adler
Comment 3 2020-10-27 14:18:25 PDT
Oops, didn’t see Joe’s comment.
Devin Rousso
Comment 4 2020-10-27 14:23:21 PDT
Created attachment 412464 [details] [Image] after Patch is applied (In reply to Joseph Pecoraro from comment #2) > Screenshot? I normally copy/paste from there, does copy still work? You can still right-click the resource in the navigation sidebar and Copy Link. For large data URLs (i.e. >10k), it doesn't make a lot of sense to try to copy it from the sidebar IMO. Not to mention, for large data URLs they likely scroll off the screen.
Devin Rousso
Comment 5 2020-10-27 14:25:21 PDT
(In reply to Devin Rousso from comment #4) > For large data URLs (i.e. >10k), it doesn't make a lot of sense to try to copy it from the sidebar IMO. err, sorry that wasn't entirely clear. By "sidebar" I meant "in the Initiated section of the Resource panel in the details sidebar"
Joseph Pecoraro
Comment 6 2020-10-27 14:41:41 PDT
(In reply to Devin Rousso from comment #5) > (In reply to Devin Rousso from comment #4) > > For large data URLs (i.e. >10k), it doesn't make a lot of sense to try to copy it from the sidebar IMO. > err, sorry that wasn't entirely clear. By "sidebar" I meant "in the > Initiated section of the Resource panel in the details sidebar" Sounds good. I'd still expect the full URL to show up in the "Location" section of the Resource Details sidebar. But I agree in the Initiated Section truncation is better (as long as the link takes you to the resource appropriately!).
EWS
Comment 7 2020-10-27 14:49:43 PDT
Committed r269075: <https://trac.webkit.org/changeset/269075> All reviewed patches have been landed. Closing bug and clearing flags on attachment 412463 [details].
Radar WebKit Bug Importer
Comment 8 2020-10-27 14:50:22 PDT
Note You need to log in before you can comment on or make changes to this bug.