Commands in bookmarks sidebar panel are not working because the node that contains the commands is not being included from the bookmarksOverlay.xul overlay. Patch fixes that & adds a Find button to the bookmarks sidebar panel for easier access to find.
Created attachment 36287 [details] [diff] [review] while we're here, add some styles to classic to make the panel look prettier
r=fabian! for the two patches this rules Ben! Even drag and drop in sidebar works... excellent! I propose you mail the drivers for 0.9.1 checkin, because this is a high-visibility feature Patches tested on linux and win2k.
nominating for 0.9.1 - however, the .dtd are going to cause issues, l18n freeze and such (i got hit and had to create a 2nd patch for the trunk once we branch that introduces the .dtd changes). os all since this also works on my linux box.
Keywords: mozilla0.9.1, patch
OS: Windows 2000 → All
Oh I forgot a nit... you can't use "f" as "find.accesskey" because there is already a "f".
blake, sr= please? this is really important for 0.9.1
applying patch in attachment 36286 [details] [diff] [review] on linux i get: patching file mozilla/xpfe/components/bookmarks/resources/bm-panel.xul patch: **** malformed patch at line 15: <!-- XXX - would like to cut this dependency out -->
*** Bug 79952 has been marked as a duplicate of this bug. ***
copying keywords from identical 79952.
Status: NEW → ASSIGNED
Keywords: nsbeta1+, regression
sr=blake Good job fixing this nsbeta1+'er, Ben!
17 years ago
Target Milestone: --- → mozilla0.9.2
a= firstname.lastname@example.org for checkin to the trunk. (on behalf of drivers)
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
*** Bug 85423 has been marked as a duplicate of this bug. ***
*** Bug 85449 has been marked as a duplicate of this bug. ***
VERIFIED Fixed with 2001061404 builds
Status: RESOLVED → VERIFIED
*** Bug 88224 has been marked as a duplicate of this bug. ***
You need to log in before you can comment on or make changes to this bug.