Bug 14514

Summary: Add full URL view on mouseOver in the network page
Product: WebKit Reporter: Oncleben31 <oncleben31>
Component: Web Inspector (Deprecated)Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: aroben, timothy
Priority: P2 Keywords: InRadar
Version: 523.x (Safari 3)   
Hardware: All   
OS: OS X 10.4   
Attachments:
Description Flags
Patch to have the full URL on mouseOver
sam: review-
Update of Patch to add ChangeLog info
aroben: review-
patch aroben: review+

Description Oncleben31 2007-07-03 11:38:03 PDT
To be very  handy, we have to be able to read the full URL in the network page.
In the current version we are able to read the last part of the URL

A solution could be to add a roll-over effect when mouseOver event on the short URL.
Comment 1 Oncleben31 2007-07-03 11:40:56 PDT
Created attachment 15370 [details]
Patch to have the full URL on mouseOver

Patch to have the full URL on mouseOver just with CSS and HTML hack
Comment 2 Adam Roben (:aroben) 2007-07-04 20:17:20 PDT
Comment on attachment 15370 [details]
Patch to have the full URL on mouseOver

It would probably good to have a 1px border around the em.
Comment 3 Sam Weinig 2007-07-04 20:22:49 PDT
Comment on attachment 15370 [details]
Patch to have the full URL on mouseOver

Please make sure to include you set the EMAIL_ADDRESS environment variable before creating the ChangeLog and to fill in the ChangeLog with information such as the bugzilla url and title and summary of changes made.
Comment 4 Adam Roben (:aroben) 2007-07-04 20:28:29 PDT
If you just set a title attribute on the .network-file element, a tooltip will appear automatically. Then you don't need your "CSS and HTML hack"
Comment 5 Adam Roben (:aroben) 2007-07-04 20:29:52 PDT
This is what we do for resources in the left sidebar, so it'll have the added benefit of being consistent with other parts of the UI.
Comment 6 Timothy Hatcher 2007-07-04 20:39:07 PDT
Why not just use a title attribute on the link to get a tooltip for free? This will match what we already do for the sidebar.
Comment 7 Timothy Hatcher 2007-07-04 20:39:48 PDT
/me should read comments before he comments. 
Comment 8 Oncleben31 2007-07-05 00:21:42 PDT
> If you just set a title attribute on the .network-file element, a tooltip will
> appear automatically. Then you don't need your "CSS and HTML hack"

Using tooltip was my first idea but they doesn't appeared in my Safari  (on my mac and on Windows), I've search in the preference : nothing. I search on Internet and I found "defaults write com.apple.Safari WebKitShowsURLsInToolTips 1"
This trick doesn't seem to work on Safari for Windows, I've to test on a mac.

If user have to hack Webkit to have tooltip working, I think my solution is better. Moreover my solution allow to debate and find the best CSS design for the tool tip (color, position).

What is you point of view ?

Comment 9 Adam Roben (:aroben) 2007-07-05 00:26:25 PDT
(In reply to comment #8)
> Using tooltip was my first idea but they doesn't appeared in my Safari  (on my
> mac and on Windows)

Tooltips should work fine in Safari on OS X. Tooltips are not yet supported in Safari on Windows (see bug 14143). This will eventually be fixed, so I still believe the title attribute is the right way to go.
Comment 10 Oncleben31 2007-07-05 00:45:46 PDT
Created attachment 15393 [details]
Update of Patch to add ChangeLog info

In reponse of comment
Comment 11 Maciej Stachowiak 2007-07-05 22:59:57 PDT
I agree with Adam that this should use the title="" attribute. Please try a fix along those lines.
Comment 12 Adam Roben (:aroben) 2007-07-06 01:54:43 PDT
Comment on attachment 15393 [details]
Update of Patch to add ChangeLog info

Please change to using a title attribute for consistency with the existing UI.
Comment 13 Adam Roben (:aroben) 2008-01-29 11:02:30 PST
<rdar://problem/5712841>
Comment 14 Matt Lilek 2008-01-30 11:38:15 PST
Created attachment 18793 [details]
patch
Comment 15 Adam Roben (:aroben) 2008-01-30 11:49:48 PST
Comment on attachment 18793 [details]
patch

r=me
Comment 16 Matt Lilek 2008-01-31 16:50:28 PST
Committed revision 29901.