Open Bug 43180 Opened 24 years ago Updated 3 years ago

Back/Forward buttons should have the page title in the tooltip

Categories

(SeaMonkey :: UI Design, defect)

defect
Not set
normal

Tracking

(Not tracked)

People

(Reporter: bugzilla, Unassigned)

References

Details

Attachments

(1 file, 6 obsolete files)

The tooltip of the Back and Forward buttons should be the title of the pages 
that they are pointing to, like in Netscape 4.73.
methinks this should go to History...?
Assignee: don → radha
Component: XP Apps → History
QA Contact: sairuh → claudius
*** Bug 46376 has been marked as a duplicate of this bug. ***
not crucial for beta3
Status: NEW → ASSIGNED
Target Milestone: --- → M20
any headway with this?
4xp keyword
Keywords: 4xp
nav triage team: looked at this bug, it is not a beta stopper. bulk update of 
several such bugs. 
Keywords: nsbeta1-
Mass moving all Navigator bugs to the Nav team. 
Assignee: radha → vishy
Status: ASSIGNED → NEW
Marking nsbeta1- bugs as future to get off the radar
Target Milestone: --- → Future
*** Bug 71405 has been marked as a duplicate of this bug. ***
Depends on: 79240
Bug 79240 is related, but not depends surely?
It deals with putting seperate tooltips on the back button and the drop down
list, this is about changing the tooltip on the main back button. If 79240 is
not fixed, then this tooltip will simply show up on the drop down arrow (which
is no more incorrect than the current tooltip).
*** Bug 150442 has been marked as a duplicate of this bug. ***
Reassigning to a real person.
Assignee: vishy → radha
This is a browser feature. Session History provides all the required apis to
accomplish this. 
Assignee: radha → sgehani
Attached patch fix (obsolete) — Splinter Review
My first real patch..

I found diff for Windows at http://www.santafe.edu/~vince/alphatk/
Attached patch fix (obsolete) — Splinter Review
My first real patch..

I found diff for Windows at http://www.santafe.edu/~vince/alphatk/
Comment on attachment 111418 [details] [diff] [review]
fix

Oh, and thanks to Hixie and bug 105485!
Attachment #111418 - Attachment is obsolete: true
Attachment #111419 - Flags: review?(neil.parkwaycc.co.uk)
Comment on attachment 111419 [details] [diff] [review]
fix

Bookmarks have special handling because they have two rows of text. You don't,
in fact your "first" row only exists as a place to put a backup copy of your
tooltip text.

Could you try updating the tooltiptext when the back and forward buttons get
updated?
Attachment #111419 - Flags: review?(neil.parkwaycc.co.uk) → review-
> Bookmarks have special handling because they have two rows of text. You don't,
in fact your "first" row only exists as a place to put a backup copy of your
tooltip text.

The first labels are to store the "Go back/forward one page" strings. Should I
get rid of that and show the title and URL like the Personal Toolbar does?

> Could you try updating the tooltiptext when the back and forward buttons get
updated?

Yes, but I thought this would be slightly more efficient while browsing.
Attached patch Doesn't work (obsolete) — Splinter Review
But I don't have time to spend on it now.
Attached patch Almost works (obsolete) — Splinter Review
I'll see to the new history handling later.
Attachment #129997 - Attachment is obsolete: true
Attached patch Should work (obsolete) — Splinter Review
Slow or not..
Attachment #130051 - Attachment is obsolete: true
Attached patch Almost done (obsolete) — Splinter Review
This works. But I don't know where to get the url.
Attachment #130095 - Attachment is obsolete: true
Comment on attachment 130567 [details] [diff] [review]
Almost done

Aha! I see what you're doing now. But I suggest that you don't copy the default
value to the title element, instead just show the default element and hide the
title element. Also please use element.hidden = true; (or false).
Attached patch Updated patchSplinter Review
I assume I can use = instead of setAttribute because .hidden and .value are
default attributes?
Attachment #111419 - Attachment is obsolete: true
Attachment #130567 - Attachment is obsolete: true
Bug 210571 is related to this one, request to show URL in tooltip in addition to
title to avoid confusion when multiple sites with identical titles are in the
Back dropdown menu.
Blocks: 210571
Just my 2 cents, but I don't think that this patch is what we need. 

Ok, it might solve this bugs problem, but I think we need a more general way to
solve tooltip problems. Also, there's some, more or less, duplicated code. I'm
sure someone can fix this bug without duplicated code.

So, I'm for a more flexible solution. One that can be used in several places to
fix bugs or reduce code and to ease developers life a bit.

What about extending the tooltip binding in popup.xml with a new attribute
called 'multiline'? The default value should be set to 'false' but tooltips for
tabs, the bookmarks on the personal toolbar and the back and forward button
should use multiline="true". A new property could be used to pass in an array
with any data. We could make all tooltips multiline by default, andonly add new
lines when we need, after locating that new property (array).
It seems that IE has the ability to display tooltips using the html renderer.
Now user can show tooltip (like over the anchor tag) with full html tagged text.
Reassigning, per comment 13.
Assignee: samir_bugzilla → guifeatures
Component: History: Session → XP Apps: GUI Features
Priority: P3 → --
QA Contact: claudius
Target Milestone: Future → ---
Product: Core → Mozilla Application Suite
Additionally, the URL to be loaded should be displayed in the status bar.  This
is consistent with general expected link behaviour:

1: Hover over link
2: View status bar to see the address that will be loaded
3: Maybe see a tooltip with a description of the targeted page

The Back/Forward buttons should behave, from an end-user perspective, in the
same way as links within a web page, since they do the same thing -- take the
user someplace else.
*** Bug 328772 has been marked as a duplicate of this bug. ***
Filter "spam" on "guifeatures-nobody-20080610".
Assignee: guifeatures → nobody
QA Contact: guifeatures
Component: XP Apps: GUI Features → UI Design
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: