Open
Bug 1186281
Opened 9 years ago
Updated 2 years ago
Missing max width for search suggestions on about:home and about:newtab
Categories
(Firefox :: New Tab Page, defect, P5)
Tracking
()
NEW
Tracking | Status | |
---|---|---|
firefox42 | --- | affected |
People
(Reporter: alex, Unassigned)
References
(Blocks 1 open bug)
Details
Attachments
(3 files)
(deleted),
image/png
|
Details | |
(deleted),
image/png
|
Details | |
(deleted),
patch
|
Details | Diff | Splinter Review |
Steps to reproduce:
1. Open about:home or about:newtab in Firefox Nightly (here: 2015-07-21).
2. Type a very long search term into the search input that is longer than the input field itself.
3. Press the search button or the enter key so that the input gets added to the search suggestions.
4. Go back to about:home/about:newtab and type the beginning of the previously entered search term so that it appears within the suggestions below.
Expected result:
Because the suggested search term is longer than the input field, the suggestion should be cut off.
Actual result:
The suggestions box grows according to the suggestions length. This breaks the design and does not correspond to the search bars behavior.
(It could be intended behavior, but the visual design looks like it wasn't.)
Comment 1•9 years ago
|
||
Thanks for filing this.
Just wanted to note here that this bug exists in the old UI as well. We probably want to truncate long suggestions with an ellipsis. Stephen, can you confirm?
Flags: needinfo?(shorlander)
Comment 3•9 years ago
|
||
(In reply to Nihanth Subramanya [:nhnt11] from comment #1)
> We probably want to truncate long suggestions with an ellipsis.
Yes, that's what the search bar suggestion panel does.
Actually, we probably don't want to have the panel expend at all based on the size of the text content. Having it expand dynamically could cause interesting complications with the layout of the one-off engine icons when these were previously displayed on more than one line.
Comment 5•9 years ago
|
||
This WIP mostly fixes the bug, but the width of the suggestions list sub-table is off by 2px for some reason, and I don't yet know the reason for needing to subtract 22px from the input box's width when setting the width of the table.
Assignee: nobody → nhnt11
Status: NEW → ASSIGNED
Flags: needinfo?(shorlander)
Updated•6 years ago
|
Comment 6•6 years ago
|
||
nhnt11, what's the status of this bug? Is it an activity stream bug now? The recently filed bug 1478048 looks a lot like a duplicate.
Flags: needinfo?(nhnt11)
Comment 7•6 years ago
|
||
Pfff, forgot about this one. This is a content search bug, the fix should go in browser/base/content/contentSearchUI.css. As for bug 1478048, the description in that bug suggests that the dropdown isn't being shown at all, which I can't reproduce. Not sure if it's the same bug. It does sound related though. Let me see if I can resurrect this patch, thanks for bringing it to my attention.
Comment 8•6 years ago
|
||
Update: I just saw the screencast in bug 1478048, and it seems like it is indeed a dupe of this bug. I will update that bug accordingly.
Comment 9•6 years ago
|
||
Another update: bug 1478048 also highlights another issue that is separate from the missing max-width issue. I'm going to leave things the way they are. The summary on that bug still makes sense.
Comment 10•6 years ago
|
||
(In reply to Florian Quèze [:florian] from comment #6)
> Is it an activity stream bug now?
I realized I didn't answer this. I believe Activity Stream is the only consumer of content search at this point, so maybe they can allocate time for this.
Flags: needinfo?(nhnt11)
Updated•6 years ago
|
Priority: -- → P5
Updated•5 years ago
|
Component: Search → New Tab Page
Priority: P5 → --
Updated•5 years ago
|
Priority: -- → P5
Updated•2 years ago
|
Severity: normal → S3
Comment 14•2 years ago
|
||
The severity field for this bug is relatively low, S3. However, the bug has 3 duplicates.
:daleharvey, could you consider increasing the bug severity?
For more information, please visit auto_nag documentation.
Flags: needinfo?(dharvey)
Comment 15•2 years ago
|
||
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?(dharvey)
You need to log in
before you can comment on or make changes to this bug.
Description
•