Closed
Bug 196756
Opened 22 years ago
Closed 15 years ago
Meta bug for major architectural changes in bookmarks code
Categories
(SeaMonkey :: Bookmarks & History, defect)
Tracking
(Not tracked)
RESOLVED
INCOMPLETE
People
(Reporter: janv, Unassigned)
References
Details
(Keywords: meta)
Attachments
(3 files, 2 obsolete files)
121.53 KB,
image/png
|
Details | |
41.56 KB,
image/png
|
Details | |
115.29 KB,
application/x-gzip
|
jag+mozilla
:
review+
bryner
:
superreview+
|
Details |
I know that there is bug 160019, but I want to keep this separate. There are many issues which need major changes in the code, so it's not possible to fix all those bugs separately. I created a new branch for this purpose. The branch name is BOOKMARKS_20030310_BRANCH. I'll add dependencies for bugs we plan to fix on the branch.
Reporter | ||
Updated•22 years ago
|
Reporter | ||
Updated•22 years ago
|
Status: NEW → ASSIGNED
OS: Linux → All
Priority: -- → P1
Target Milestone: --- → mozilla1.4beta
Reporter | ||
Updated•22 years ago
|
QA Contact: kasumi → petersen
Reporter | ||
Comment 2•22 years ago
|
||
I also included some cleanup bugs and minor issues, sorry for the SPAM.
Depends on: 119796
Comment 3•22 years ago
|
||
Bug #25909, bug #34059, and bug #36828 would all probably require some changes to the bookmarks file format. However, they're all RFEs, so I'm not sure if they really go here.
Reporter | ||
Comment 5•22 years ago
|
||
Comment 6•22 years ago
|
||
Comment on attachment 117763 [details]
Screenshot
I don't see a PT site icon in that screenshot :-P
Reporter | ||
Comment 8•22 years ago
|
||
Reporter | ||
Comment 9•22 years ago
|
||
Comment 13•22 years ago
|
||
imo, if 'address' is chosen instead of location, then all occurrences of 'location' should be replaced. but in another bug. Let's keep consistency for now.
Reporter | ||
Updated•22 years ago
|
Attachment #118114 -
Attachment is obsolete: true
Reporter | ||
Updated•22 years ago
|
Attachment #118115 -
Attachment is obsolete: true
Reporter | ||
Comment 14•22 years ago
|
||
Reporter | ||
Updated•22 years ago
|
Attachment #118339 -
Attachment mime type: application/x-tar → application/x-gzip
Attachment #118339 -
Flags: superreview?(bryner)
Attachment #118339 -
Flags: review?(jaggernaut)
Reporter | ||
Comment 15•22 years ago
|
||
Changes done on BOOKMARKS_20030310_BRANCH: http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=all&branch=BOOKMARKS_20030310_BRANCH&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=all&mindate=&maxdate=&cvsroot=%2Fcvsroot Changes done on BOOKMARKS_20030320_BRANCH: http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=all&branch=BOOKMARKS_20030320_BRANCH&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=all&mindate=&maxdate=&cvsroot=%2Fcvsroot
Comment 16•22 years ago
|
||
Comment on attachment 118339 [details]
latest diff
I don't think I'm going to be able to read through every line of this patch,
it's too big. It looks good though. One comment:
- in nsTreeBodyFrame.cpp:
+ if (primaryX1 > rowRect.x) {
+ }
Jan tells me that he was planning to add code there to deal with the mac native
theme code drawing a vertical line instead of a horizontal separator. Could
you replace the useless if check with a comment about the problem?
sr=bryner with that change.
Attachment #118339 -
Flags: superreview?(bryner) → superreview+
Comment 17•22 years ago
|
||
Comment on attachment 118339 [details]
latest diff
r=jag. I went over this with Jan last Friday and Saturday.
Attachment #118339 -
Flags: review?(jaggernaut) → review+
It looks like the branch landing broke bookmarks in Phoenix.
Comment 21•22 years ago
|
||
This has existed since pre 1.0, and 0 attention has been given: http://bugzilla.mozilla.org/show_bug.cgi?id=110313
Comment 22•22 years ago
|
||
I'm not sure about the zero attention, however, maybe Bug #110313 should be part of this Meta tracking bug?
Reporter | ||
Comment 23•22 years ago
|
||
I filed this bug to track bugs/features we plan to fix on the branch. The branch landed on the trunk, so other bugs shouldn't be added here. Leaving this open, because some of the bugs are not verified yet.
Comment 24•22 years ago
|
||
This bug should probably also depend on: Bug 59173 (Need better/advanced bookmark/history search) Bug 93058 ("Type Ahead Find"-like bookmark keyword search) Prog.
Updated•22 years ago
|
Blocks: bookmark-loss
Comment 25•22 years ago
|
||
Comment on attachment 118339 [details]
latest diff
+ var iconNode = document.getAnonymousElementByAttribute(target,
"class", "toolbarbutton-icon");
Did it not occur to you that this can return null?
+ try {
+ return node.QueryInterface(kRDFRSCIID).Value;
+ }
+ catch (e) {
+ return node? node.QueryInterface(kRDFLITIID).Value : "";
+ }
instanceof has been around for at least two years...
Comment 26•21 years ago
|
||
Were these lines in bookmarks.js added as part of this landing? QueryInterface : function (aUID) {return this}, This is just so lame it's unbelievable... it royally confuses XPConnect which subsequently tries to QI your object to all sorts of unsuitable interfaces including unscriptable ones!
Comment 27•21 years ago
|
||
Neil: yes, that's part of this landing, and that's also what kin (who wrote the transaction manager) suggested in the js implementation he sent me via email. But feel free to patch it.
Updated•20 years ago
|
Product: Browser → Seamonkey
Updated•17 years ago
|
Assignee: Jan.Varga → nobody
Status: ASSIGNED → NEW
Priority: P1 → --
QA Contact: chrispetersen → bookmarks
Target Milestone: mozilla1.4beta → ---
![]() |
||
Comment 28•15 years ago
|
||
MASS-CHANGE: This bug report is registered in the SeaMonkey product, but has been without a comment since the inception of the SeaMonkey project. This means that it was logged against the old Mozilla suite and we cannot determine that it's still valid for the current SeaMonkey suite. Because of this, we are setting it to an UNCONFIRMED state. If you can confirm that this report still applies to current SeaMonkey 2.x nightly builds, please set it back to the NEW state along with a comment on how you reproduced it on what Build ID, or if it's an enhancement request, why it's still worth implementing and in what way. If you can confirm that the report doesn't apply to current SeaMonkey 2.x nightly builds, please set it to the appropriate RESOLVED state (WORKSFORME, INVALID, WONTFIX, or similar). If no action happens within the next few months, we move this bug report to an EXPIRED state. Query tag for this change: mass-UNCONFIRM-20090614
Status: NEW → UNCONFIRMED
Updated•15 years ago
|
Status: UNCONFIRMED → RESOLVED
Closed: 15 years ago
Resolution: --- → INCOMPLETE
You need to log in
before you can comment on or make changes to this bug.
Description
•