The default bug view has changed. See this FAQ.

[SeaMonkey] Replace "emptytext" in abCardOverlay with "placeholder"

RESOLVED FIXED in seamonkey2.1b1

Status

SeaMonkey
UI Design
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: Nomis101, Assigned: Nomis101)

Tracking

Trunk
seamonkey2.1b1

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

5.23 KB, patch
neil@parkwaycc.co.uk
: review+
Details | Diff | Splinter Review
(Assignee)

Description

7 years ago
Created attachment 469465 [details] [diff] [review]
Fix

This patch will replace "emptytext" in abCardOverlay with "placeholder". It is a follow-up to Bug 550186 and Bug 551747.
Attachment #469465 - Flags: review?(neil)
Assignee: nobody → Nomis101
Status: NEW → ASSIGNED

Comment 1

7 years ago
Comment on attachment 469465 [details] [diff] [review]
Fix

Note that you seem to have overlooked the reference in abCardOverlay.js (both in SeaMonkey and in Thunderbird).
Attachment #469465 - Flags: review?(neil) → review+
(Assignee)

Comment 2

7 years ago
(In reply to comment #1)
> Note that you seem to have overlooked the reference in abCardOverlay.js (both
> in SeaMonkey and in Thunderbird).

Oops, you are right. Should I make a new patch with abCardOverlay.js included? Or a follow-up patch?

Comment 3

7 years ago
New patch please.
(Assignee)

Comment 4

7 years ago
Created attachment 469849 [details] [diff] [review]
Fix v2

Patch with abCardOverlay.js
Attachment #469465 - Attachment is obsolete: true
Attachment #469849 - Flags: review?(neil)

Updated

7 years ago
Attachment #469849 - Flags: review?(neil) → review+
(Assignee)

Updated

7 years ago
Keywords: checkin-needed
(Assignee)

Comment 5

7 years ago
I don't have the adequate rights to check this in. So if anyone can do this, that would be nice.
(In reply to comment #5)
> I don't have the adequate rights to check this in. So if anyone can do this,
> that would be nice.

Done http://hg.mozilla.org/comm-central/rev/2bcd6ba4d38e

But in the future, please use/read: 
http://blog.bonardo.net/2010/06/22/so-youre-about-to-use-checkin-needed
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Keywords: checkin-needed
Resolution: --- → FIXED

Updated

7 years ago
Target Milestone: --- → seamonkey2.1b1
You need to log in before you can comment on or make changes to this bug.