Closed
Bug 17920
Opened 25 years ago
Closed 24 years ago
personal toolbar items need context menus (for editing, deleting, etc.)
Categories
(SeaMonkey :: Bookmarks & History, enhancement, P2)
SeaMonkey
Bookmarks & History
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla0.9
People
(Reporter: ezh, Assigned: bugs)
References
Details
(Keywords: helpwanted)
Attachments
(7 files)
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review |
Many people wish to able edit (erase, etc.) bookmarks right from Personal
Toolbar (right click menu?), not from bookmark menu.
Tnx!
Assignee: leger → don
Component: Browser-General → XPApps
QA Contact: leger → claudius
Putting on XP Apps radar for an enhancement. don you can set the milestone. :-)
Updated•25 years ago
|
Target Milestone: M14
Comment 2•25 years ago
|
||
post-beta, m14.
Reporter | ||
Comment 5•24 years ago
|
||
It is very critical for about 150 computers we have in my company. We are using
The Bat! (http://www.ritlabs.com/the_bat/index.html) with NN4.x with utility
which allows to run The Bat! when clicked on mail link.
Is it so hard to implement "mail to:" links opens default mail programm???
Reporter | ||
Comment 7•24 years ago
|
||
Any news for this enh?
Comment 8•24 years ago
|
||
This Request For Enhancement is currently targeted for the 'future', meaning it won't make it
into the current release as we try to concentrate on fixing critical bugs.
I've added the 'helpwanted' keyword which may attract a mozilla contributor willing to
implement this feature in a shorter time frame.
Keywords: helpwanted
Reporter | ||
Comment 9•24 years ago
|
||
May be now, after NS6 release, we can set the right milestone for this?
Comment 10•24 years ago
|
||
Updating summary to include "context menus" so I find this bug next time I
search for it. Cc'ing blake in case he's interested.
Summary: Editing bookmarks in personal toolbar. → personal toolbar items need context menus (for editing, deleting, etc.)
Comment 11•24 years ago
|
||
Since Don has left, Vishy is taking his bugs in bulk, pending reassignment.
thanks,
Vishy
Assignee: don → vishy
Comment 12•24 years ago
|
||
nav triage team:
Let's look into this, marking nsbeta1, mozilla0.9, reassigning to ben
Comment 13•24 years ago
|
||
Recommend wontfix. People will wanting to turn the Personal Toolbar on and off
much more frequently than they will be wanting to edit the items on it. Since
the UI for this is relatively inaccessible (a submenu of the View menu), it
would make more sense for Ctrl+/right-clicking the toolbar to bring up a menu
for showing/hiding toolbars (bug 45532), like Internet Explorer has, than for
the context menu to be used for editing the items in the toolbar. (The context
menu would have a context-sensitive item for customizing the contents of that
particular toolbar.)
Comment 14•24 years ago
|
||
I strongly disagree with wontfix. Internet Explorer's convenient editing is
far superior to our means of editing (and no, IE doesn't use the toolbar menu
for PT items, it uses an editing one).
Comment 15•24 years ago
|
||
pardon the spam: personal toolbar bugs should go into the Bookmarks component.
so, i'm moving the existing ones from xp apps/xp apps: gui to there...
Component: XP Apps → Bookmarks
Assignee | ||
Comment 16•24 years ago
|
||
raising priority setting.
Status: NEW → ASSIGNED
Priority: P3 → P2
Comment 17•24 years ago
|
||
nav triage team:
Marking nsbeta1+, reassigning to pchen
Assignee | ||
Comment 18•24 years ago
|
||
Paul, I have this one all fixed up. Reassigning back to self, will attach patch
in a couple of minutes. Note this also fixes my other .9 bug related to the
empty submenus.
Assignee: pchen → ben
Assignee | ||
Comment 19•24 years ago
|
||
Assignee | ||
Comment 20•24 years ago
|
||
Assignee | ||
Comment 21•24 years ago
|
||
Assignee | ||
Comment 22•24 years ago
|
||
Assignee | ||
Comment 23•24 years ago
|
||
Assignee | ||
Comment 24•24 years ago
|
||
OK, have attached patches to get basic functionality off the floor. Can
manipulate bookmarks (open, open folder in new window, find, copy, cut, paste,
delete, rename, properties, create new folder) as in regular bookmarks window.
Inplace editing works for buttons and menubuttons, and new folder creation is
achieved by creating an item with an inline field as in the window. Seems to
work pretty well. The method of invoking the field is simpler and thus not prone
to showing when undesired as in the window. The inline edit binding has been
factored into a generic base which I put in xulBindings.xml, and button,
menubutton and treecell variants created. I had to rename some commands to fix a
command name collision in the nav window ('cmd_paste' etc, clash between the
edit field controller and my own). D&D will come next.
Assignee | ||
Updated•24 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 25•24 years ago
|
||
Assignee | ||
Comment 26•24 years ago
|
||
Comment 27•24 years ago
|
||
sr=sfraser
Comment 28•24 years ago
|
||
well ... it _looks_ good to me; but it's a big patch, and it's outside the
regions I'm really comfortable with. I don't think my opinion can be counted
for an entire sr. I'd call it about 1/3. If you can get another super-reviewer
to give you the other 2/3, then you're good to go. I apologize for not being
savvy enough to do the job here.
Comment 29•24 years ago
|
||
[s]r = hyatt
Assignee | ||
Comment 30•24 years ago
|
||
fixed.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Comment 31•24 years ago
|
||
I don't see context menus for items within folders on the personal toolbar
(even if I use Expand on the folder's context menu). I think that's covered by
bug 50504, though.
Comment 32•24 years ago
|
||
wow, I would've never expected contextual menus on menuitems but that is indeed bug 50504
(might as well expand it). This bug however is indeed fixed and VERIFIED Fixed with the 2001042608
builds on all platforms
Status: RESOLVED → VERIFIED
Reporter | ||
Comment 33•23 years ago
|
||
But how about context menu inside a folder on a PT?
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•