Closed
Bug 60071
Opened 24 years ago
Closed 24 years ago
Hide sidebar tab using sidebar context menu
Categories
(SeaMonkey :: Sidebar, enhancement, P1)
SeaMonkey
Sidebar
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla0.9.1
People
(Reporter: alecf, Assigned: bugzilla)
Details
Attachments
(3 files)
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review |
from the CNET review of Netscape 6, they wanted a way to hide a sidebar by right-clicking on it. I think what we need is a few entries in the sidebar context menus: - Hide this panel and possibly - Next panel - Previous panel (but this bug will be for the "hide" part - next/previous is just an added bonus)
Assignee | ||
Comment 1•24 years ago
|
||
I was just about to file this :-) God, they mentioned that like twice. We probably would have gotten an 8 if we had it. reassigning to me. I was going to do this anyways, but first we have the arduous task of making oncommand work in modern-based themes.
Assignee: matt → blakeross
Comment 2•24 years ago
|
||
Next and Previous would make no sense, since it's far faster just to click on the panel's tab than to navigate through a context menu. De-ambiguifying summary.
Severity: normal → enhancement
OS: Linux → All
Hardware: PC → All
Summary: remove sidebar from sidebar context menu → Hide sidebar tab using sidebar context menu
Assignee | ||
Comment 3•24 years ago
|
||
Right. So is a context menu even needed? I suppose it could have a "reload" item also. C|net actually asked that right clicking on a panel just removes it from the sidebar without a menu at all, but I'm not sure how intuitive that is.
Status: NEW → ASSIGNED
Comment 4•24 years ago
|
||
No. Right-click by itself shouldn't ever do anything except open context menus, IMO. I was thinking more of a little context menu like this: _Hide What's Related -------------------------- C_ustomize Sidebar ... Reload is not necessary -- you should be able to give focus to the panel and then just click the Reload button in the toolbar, or press Ctrl+R, or whatever.
Assignee | ||
Updated•24 years ago
|
Priority: P3 → P2
Target Milestone: --- → mozilla0.9
Comment 5•24 years ago
|
||
>you should be able to give focus to the panel and then just click the Reload >button in the toolbar, or press Ctrl+R I filed bug 61302 for Ctrl+R.
Updated•24 years ago
|
QA Contact: shrir → sujay
Comment 7•24 years ago
|
||
Just to clarify, this is the context menu for the tab and not for the panel content, right? Should the menu include a "default" option matching left-click, perhaps callled "show [panel name]"? (Show sounds like the opposite of hide, though, and these commands wouldn't be opposite because show would just focus the panel, so maybe "show" isn't the best label for this command.)
Assignee | ||
Updated•24 years ago
|
Target Milestone: mozilla0.9 → mozilla0.9.1
Comment 8•24 years ago
|
||
this is a nsbeta1+ P2 bug. Blake - do you think we'll get this for mozilla0.9.1?
Keywords: nsbeta1+
Assignee | ||
Updated•24 years ago
|
Priority: P2 → P1
Assignee | ||
Comment 9•24 years ago
|
||
Assignee | ||
Comment 10•24 years ago
|
||
Assignee | ||
Comment 11•24 years ago
|
||
Comment 13•24 years ago
|
||
OK, blake sez he'll be tidying up the popupset/commands stuff at some point in the future, so this code looks OK for the interim. r=ben@netscape.com
Assignee | ||
Comment 14•24 years ago
|
||
Checked in. File a new bug to discuss the merits of Switch Panel if necessary.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•