Closed
Bug 81076
Opened 24 years ago
Closed 23 years ago
The search window's default columns fix has regressed.
Categories
(SeaMonkey :: MailNews: Message Display, defect, P3)
SeaMonkey
MailNews: Message Display
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla0.9.7
People
(Reporter: stephend, Assigned: skasinathan)
References
()
Details
(Keywords: regression, Whiteboard: [nsbeta1+]Have Fix)
Attachments
(1 file)
(deleted),
patch
|
Details | Diff | Splinter Review |
Build ID: 2001051504, all OSs.
Summary: The search window's default columns fix has regressed.
Rather than opening a bug that *was* fixed and verified, I'm filing this.
As a regression from bug 76216:
"The default columns (and order) in the Results area should be: Thread, Subject,
Sender, Date, Location, Column Widget."
Something has broken, and my functions either aren't getting called, or are useless.
http://lxr.mozilla.org/seamonkey/source/mailnews/base/search/resources/content/SearchDialog.js
I will add dump statements and build debug, but I bet this is beyond my scope.
Reporter | ||
Comment 1•24 years ago
|
||
Currently, we are showing all of the columns again (so this is a total
regression). Since bug 76216 was marked nsbeta1+, I'm nominating this, too.
Keywords: nsbeta1,
regression
QA Contact: esther → laurel
Comment 2•24 years ago
|
||
the thread column should not be in the search dialog.
if the spec says that, can log a bug on jglick to fix it?
weren't there a bunch of dialog problems today?
I'm not seeing this in my linux build.
Reporter | ||
Comment 3•24 years ago
|
||
Sorry, I just cut and pasted from the other bug.
http://www.mozilla.org/mailnews/specs/search/ is correct, I believe. My fault.
And I'm not sure about bunch of dialog problems (although I heard something
broke in nav, too).
Comment 4•24 years ago
|
||
reassigning to suresh. Moving to 0.9.2, but if this got fixed before Friday that
would be great.
Assignee: naving → suresh
Priority: -- → P3
Whiteboard: [nsbeta1+]
Target Milestone: --- → mozilla0.9.2
Comment 5•24 years ago
|
||
stephend, did you see this problem on a new profile or on your existing profile?
Reporter | ||
Comment 6•24 years ago
|
||
On both, and you'll also notice this isn't totally broken. If it was, we
wouldn't be hiding by default the other columns. Really, what has regressed
here is the attribute setting of "ignoreincolumnpicker".
Comment 7•24 years ago
|
||
I see this too, now.
outlinerBindings.xml became outliner.xml on 5/11
it doesn't look like all the changes to outlinerBindings.xml made it into the
new file.
hewitt?
Comment 9•23 years ago
|
||
sr=hewitt on the patch, but, you might want to replace those tabs with spaces
Comment 10•23 years ago
|
||
r=sspitzer
make sure to fix the tabs problem before checking in.
Comment 11•23 years ago
|
||
Sorry to jump in late...
Suresh, how about this:
if (firstTime) {
+ if (currCol.getAttribute("hidden") != "true") {
becomes this:
if (firstTime && currCol.getAttribute("hidden") != "true") {
after that. r=hwaara
Updated•23 years ago
|
Whiteboard: [nsbeta1+] → [nsbeta1+]Have Fix
Comment 12•23 years ago
|
||
moving to 0.9.3. Please hold onto the fix in case we can get it in later.
Target Milestone: mozilla0.9.2 → mozilla0.9.3
Assignee | ||
Comment 13•23 years ago
|
||
-> 0.9.4
Status: NEW → ASSIGNED
Target Milestone: mozilla0.9.3 → mozilla0.9.4
Comment 15•23 years ago
|
||
so this is ready to go and all we're waiting for is a green light?
Assignee | ||
Comment 16•23 years ago
|
||
Looks like this is fixed...:)
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Comment 17•23 years ago
|
||
Looks OK to me using dec13 commercial trunk builds.
Label column is not displayed by default but is in the available choices in the
search results's column widget, covered by bug 107878.
Marking this bug verified.
Status: RESOLVED → VERIFIED
Updated•20 years ago
|
Product: Browser → Seamonkey
Component: MailNews: Search → MailNews: Message Display
QA Contact: laurel → search
You need to log in
before you can comment on or make changes to this bug.
Description
•