Closed
Bug 1278619
Opened 8 years ago
Closed 8 years ago
nav_help.xhtml: Sidebar Handle screenshot is outdated
Categories
(SeaMonkey :: Help Documentation, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
seamonkey2.47
People
(Reporter: isaacschemm, Assigned: isaacschemm)
References
(Blocks 1 open bug)
Details
(Keywords: reproducible)
Attachments
(2 files)
User Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Firefox/45.0 SeaMonkey/2.42 Build ID: 20160503211837 Steps to reproduce: Go to Browsing the Web > Navigating Web Pages > Using Sidebar or Customizing SeaMonkey > Sidebar. Actual results: The screenshot showing the sidebar handle also shows search results in the sidebar, a feature that no longer exists. (The screenshot is the same in both sections.) Expected results: The screenshot should either show the search tab without results or show another tab, such as bookmarks or history.
Comment 1•8 years ago
|
||
REPRODUCIBLE with English SeaMonkey 2.45a1 (Windows NT 6.1; WOW64; rv:48.0) Gecko/20100101 Firefox/48.0 Build 20160308001946 (Default Classic Theme) on German WIN7 64bit nav_help.xhtml contains ".../chrome/en-US/locale/en-US/communicator/help/images/sidebar.png" with search results in sidebar. This feature does not exist. a) I wonder whether the problem is a1) that search results are missing in sidebar or a2) that the help contents does not respect intended (?) new design without search results. a3) I will check <https://bugzilla.mozilla.org/buglist.cgi?cmdtype=dorem&remaction=run&namedcmd=rels1278619&sharer_id=41036> @Isaac Schemm: If final decision will be (a2) it would be great if you could contribute as attachment an edited screenshot "sidebar.png" with similar dimensions showing the new design.
Blocks: HelpCleanup2016
Keywords: helpwanted
Summary: Sidebar Handle screenshot is outdated → nav_help.xhtml: Sidebar Handle screenshot is outdated
Comment 2•8 years ago
|
||
a2): "Bug 673190 - Seamonkey 2.1+ no sidebar search results, no advanced multi-engine search" seems to indicate that current sidebar search panel view is not intended?
Assignee | ||
Comment 3•8 years ago
|
||
Here's a replacement for sidebar.png that shows the bookmarks tab instead of the search tab. sidebar.png is only used for showing where the sidebar handle is, so using the bookmarks tab here will keep it consistent even if the search tab does eventually go back to previous behavior. There's at least one other picture in Help showing search results in the sidebar, but that might be handled under another bug.
Updated•8 years ago
|
Keywords: reproducible
Assignee | ||
Updated•8 years ago
|
Attachment #8760943 -
Flags: review?(iann_bugzilla)
Attachment #8760943 -
Flags: feedback?(rsx11m.pub)
Assignee | ||
Updated•8 years ago
|
Flags: needinfo?(philip.chee)
Comment on attachment 8760943 [details] sidebar-bookmarks.png Looks good. (In reply to Rainer Bielefeld from comment #2) > a2): "Bug 673190 - Seamonkey 2.1+ no sidebar search results, no advanced > multi-engine search" seems to indicate that current sidebar search panel > view is not intended? I wouldn't say "intended" - it requires a reimplementation due to backend changes: (Quoting Phoenix from bug 673190 comment #2) > Due to migration to openSearch, this part of functionality was removed. > Confirming as RFE, maybe someone would want to reimplement it in future...
Attachment #8760943 -
Flags: feedback?(rsx11m.pub) → feedback+
Assignee: nobody → isaacschemm
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Keywords: helpwanted
Assignee | ||
Comment 5•8 years ago
|
||
A patch to apply the previous attachment to the comm-central source tree
Attachment #8761802 -
Flags: review?(iann_bugzilla)
Comment 6•8 years ago
|
||
Comment on attachment 8761802 [details] [diff] [review] sidebar-bookmarks Looks good, thanks (stealing this from Ian).
Flags: needinfo?(philip.chee)
Attachment #8761802 -
Flags: review?(iann_bugzilla) → review+
Updated•8 years ago
|
Keywords: checkin-needed
Comment 7•8 years ago
|
||
http://hg.mozilla.org/comm-central/rev/9d75efeb816a
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.47
Updated•8 years ago
|
Attachment #8760943 -
Flags: review?(iann_bugzilla)
You need to log in
before you can comment on or make changes to this bug.
Description
•