Closed Bug 228844 Opened 21 years ago Closed 21 years ago

[Bookmarks Sidebar] Quick search fails to find non-ASCII

Categories

(Firefox :: Bookmarks & History, defect)

Product:
Firefox
Component:
Bookmarks & History
Type:
defect
Priority:
Not set
Severity:
major

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: noririty, Assigned: noririty)

References

Details

Attachments

(1 file)

fix
(deleted), patch
Details | Diff | Splinter Review
Attached patch fix (deleted) — — Splinter Review
forgotten line to mass change escape() to encodeURIComponent()
*** Bug 228843 has been marked as a duplicate of this bug. ***
Sorry I missed this in my patch for bug 225695. Why don't you ask for r/sr either here or in bug 225695 (after making this as a dupe of bug 225695 and uploading your patch there)?
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Comment on attachment 137644 [details] [diff] [review] fix Reopening as I only landed on trunk. We need approval to FIREBIRD_0_8_BRANCH. pch, could you take a look at this?
Attachment #137644 - Flags: review?(p_ch)
-> Reopen
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
When was FB 0.8 branch cut? Was it after my patch for bug 44272 got landed? Was it before or after my patch for bug 225695 got in (December 3rd)? If it's before my treewide sweep got in. My treewide sweep patch in bug 225695 also needs a0.8.
jshin: after. so, your patch is already (automatically) on 0.8 branch.
Status: REOPENED → ASSIGNED
sure, go ahead
Comment on attachment 137644 [details] [diff] [review] fix thanks
Attachment #137644 - Flags: review?(p_ch)
landed on 0.8 branch -> FIXED
Status: ASSIGNED → RESOLVED
Closed: 21 years ago → 21 years ago
Keywords: intl, regression
Resolution: --- → FIXED
sorry for bugspam, long-overdue mass reassign of ancient QA contact bugs, filter on "beltznerLovesGoats" to get rid of this mass change
QA Contact: mconnor → bookmarks
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: