Open Bug 357337 Opened 18 years ago Updated 2 years ago

Popups (tooltips) are not sized correctly for multiline tooltip

Categories

(Core :: Layout, defect)

Product:
Core
Component:
Layout
Platform:
x86
Windows XP
Type:
defect

Tracking

()

People

(Reporter: csthomas, Unassigned)

References

(Depends on 1 open bug,
URL
)

Details

(Keywords: testcase, Whiteboard: [reflow refactor])

Attachments

(2 files)

A nice screenshot of this bug
(deleted), image/png
Details
Testcase from bug 464440.
(deleted), text/html
Details
Firefox users: This bug is not *at all* related to Firefox. The code in question is not part of Firefox. <Neil> CTho: looks like reflow branch breaks your tooltips <Neil> CTho: in the reflow branch, I commented out all the .height/.width stuff but that just matches trunk brokenness I was told the multiline tooltip situation would be fixed on reflow branch. CCing a couple reflow devs based on what Neil told me. Anyway, the SeaMonkey workaround needs to be revisited for trunk.
Whiteboard: [reflow refactor]
URL field set to testcase. Behavior appears to be pretty much the same in current trunk builds as in gecko 1.8.0 (specifically, a tooltip that's tall enough for 1 line of text, but the text wrapped itself so the tooltip is too wide and not tall enough). Moving bug to Core since this isn't SeaMonkey-specific.
Assignee: cst → nobody
URL: http://ctho.ath.cx/tmp/tooltip_test.xul
Component: XP Apps: GUI Features → Layout
Keywords: testcase
Product: Mozilla Application Suite → Core
QA Contact: layout
Target Milestone: seamonkey1.5alpha → ---
So is t his a reflow branch regression? Or what?
No. It was something I was told the reflow branch would fix.
Depends on: 228673
Bug 279703 seems to fix this, if you use <tooltip orient="vertical">.
Depends on: 279703
Summary: Revisit multiline tooltip situation for reflow branch → Reflow branch is supposed to fix tooltip sizing issues, so remove the hacks when it lands
Sorry for the spam. <gavin_> CTho: can you give bug 357337 a better summary? <CTho> gavin_: ok, summary changed to say exactly what the bug is about. <CTho> of course, it's now obsolete <gavin_> well <gavin_> let's morph it into the bug the testcase shows, then? <CTho> ok
Summary: Reflow branch is supposed to fix tooltip sizing issues, so remove the hacks when it lands → Popups (tooltips) are not sized correctly
is width only remaining issue? height looks OK to me.
Summary: Popups (tooltips) are not sized correctly → Popups (tooltips) are not sized correctly for multiline tooltip
The testcase in the url field works for me (firefox 3.5 linux), as both height and width are correct. The testcase in bug 464440 is still wrong though.
Attached image A nice screenshot of this bug (deleted) —
I saw, there is still no screenshot of this bug, so I added one.
Attached file Testcase from bug 464440. (deleted) —
The testcase provided in the url is no longer available. Adding testcase from bug 464440. We could probably mark bug 464440 as a dupe.
Attachment #8563618 - Attachment mime type: text/plain → text/html
(In reply to Johan C from comment #12) > The testcase provided in the url is no longer available. s/url/url-field

Mass-removing myself from cc; search for 12b9dfe4-ece3-40dc-8d23-60e179f64ac1 or any reasonable part thereof, to mass-delete these notifications (and sorry!)

Severity: normal → S3

The severity field for this bug is relatively low, S3. However, the bug has 4 duplicates.
:dholbert, could you consider increasing the bug severity?

For more information, please visit auto_nag documentation.

Flags: needinfo?(dholbert)

The last needinfo from me was triggered in error by recent activity on the bug. I'm clearing the needinfo since this is a very old bug and I don't know if it's still relevant.

Flags: needinfo?(dholbert)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: