Bug 14514 - Add full URL view on mouseOver in the network page
Summary: Add full URL view on mouseOver in the network page
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 523.x (Safari 3)
Hardware: All OS X 10.4
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2007-07-03 11:38 PDT by Oncleben31
Modified: 2008-01-31 16:50 PST (History)
2 users (show)

See Also:


Attachments
Patch to have the full URL on mouseOver (2.00 KB, patch)
2007-07-03 11:40 PDT, Oncleben31
sam: review-
Details | Formatted Diff | Diff
Update of Patch to add ChangeLog info (2.16 KB, patch)
2007-07-05 00:45 PDT, Oncleben31
aroben: review-
Details | Formatted Diff | Diff
patch (1.21 KB, patch)
2008-01-30 11:38 PST, Matt Lilek
aroben: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.