Content about Sidebar in customize_help.xhtml needs to be updated

VERIFIED FIXED

Status

Documentation
Help Viewer
VERIFIED FIXED
14 years ago
13 years ago

People

(Reporter: stefanh, Assigned: stefanh)

Tracking

Details

(URL)

Attachments

(4 attachments, 8 obsolete attachments)

(Assignee)

Description

14 years ago
The sidebar is hidden by default (1.4 -->) and there isn't any close/open
sidebar buttons in the Toolbar. That was a Netscape-only feature. The image of
the Sidebar could also be replaced.
(Assignee)

Updated

14 years ago
Summary: Content about Sidebar in customize_help.html needs to be updated → Content about Sidebar in customize_help.xhtml needs to be updated

Comment 1

14 years ago
Ok, I've removed locally the references to the Sidebar open/close buttons in the
toolbar. I can make new snapshots of the sidebar if needed, but I think that the
text should be rewritten partially beacuse it's not very clear as it is (after
cutting). I've added the option of pressing F9 for making the Sidebar appear,
but the ul about the handle is not very useful/clear. Also, where are the
references to My Sidebar? They should be removed as well.
(Assignee)

Comment 2

14 years ago
(In reply to comment #1)
> Ok, I've removed locally the references to the Sidebar open/close buttons in the
> toolbar. I can make new snapshots of the sidebar if needed, but I think that the
> text should be rewritten partially beacuse it's not very clear as it is (after
> cutting). I've added the option of pressing F9 for making the Sidebar appear,
> but the ul about the handle is not very useful/clear. Also, where are the
> references to My Sidebar? They should be removed as well.

Giacomo,

Sorry for the delay. If i manage to update the text, can you fix the
screenshots? Will hopefully be able to get to this soon. Would probably be best
if you provide a fresh png after I updated the text. Then you can change the
search terms if necessary. Not really sure if "Baby Doll" is the best search
term anymore... BTW, does F9 do the same thing as View-->Show/Hide-->Sidebar?
That is, either open the sidebar or completely hides the sidebar (even the
handle)? Just want to confirm, since this doesn't works on mac anymore (due to
Apple's mapping of F9 to Exposé in Panther).

Actually, the references to My Sidebar are now only in the image itself ;)

Comment 3

14 years ago
(In reply to comment #2)
> Sorry for the delay. If i manage to update the text, can you fix the
> screenshots?

Sure, will gladly do.

> Will hopefully be able to get to this soon. Would probably be best
> if you provide a fresh png after I updated the text. Then you can change the
> search terms if necessary. Not really sure if "Baby Doll" is the best search
> term anymore...

Pick up another search term, and I'll update the screenshot accordingly.

> BTW, does F9 do the same thing as View-->Show/Hide-->Sidebar?

Yes.

> That is, either open the sidebar or completely hides the sidebar (even the
> handle)?

Yes.
(Assignee)

Comment 4

14 years ago
Created attachment 169731 [details] [diff] [review]
updated content for sidebar

OK, this will update the sidebar stuff in help. Giacomo, since the search term
"baby doll" also appears in the Location Bar in nav_help I haven't changed the
search term. But I would really like to change it. Suggestions are welcome...,
what about "hostels rome"? ;)

Ian: this might need some more changes, I'm wondering if we can remove
"Opening, closing and..." and rename it to "Using Sidebar" (current "Using
Sidebar" stuff will be included).

Also, several features seem to be broken, especially the Advanced search
features described in nav_help. And I don't seem to be able to customize any
tab...
(Assignee)

Updated

14 years ago
Attachment #169731 - Flags: review?(bugzilla)

Comment 5

14 years ago
Comment on attachment 169731 [details] [diff] [review]
updated content for sidebar

I think it is better to keep Opening, Closing and Resizing separate to Using -
Makes it more obvious for someone searching. With that in mind are you going to
have different images for the three sections or use the same one for all three?
Before I r= I'd like to see the new images.
(Assignee)

Comment 6

14 years ago
(In reply to comment #5)
> (From update of attachment 169731 [details] [diff] [review] [edit])
> I think it is better to keep Opening, Closing and Resizing separate to Using -
> Makes it more obvious for someone searching.

I have a suggestion: since nearly all the headings except "What is Sidebar"
could fit under "Using Sidebar" and we're already on the <h3>-level, how about
removing the "Using Sidebar" section and just have it like this:

Sidebar
  What is Sidebar?
  Opening, Closing and Resizing Sidebar
  Viewing Sidebar Tabs (used to be "Using Sidebar")
  Adding Sidebar Tabs
  Customizing Individual Sidebar Tabs
  Reorganizing Sidebar Tabs
  Removing Sidebar Tabs


With that in mind are you going to
> have different images for the three sections or use the same one for all three?

I think we need just one image of the sidebar in customize_help. The same image
could be used in the "Using Sidebar" section in nav_help. We also need a new
image for the "Advanced Search Mode" section in nav_help.
(Assignee)

Updated

14 years ago
Attachment #169731 - Attachment is obsolete: true
Attachment #169731 - Flags: review?(bugzilla)
(Assignee)

Comment 7

14 years ago
Created attachment 170053 [details] [diff] [review]
Sidebar content update

OK, here's a new version. Giacomo, can you fix the two images? I have changed
the filenames to better correspond to the images:

1) sidebarclosed_small.png --> sidebar.png: The search terms could be "open
source" or "hotels rome", (Ian suggested "open source")... as long as you'll
get good results it's fine with me ;)

2)searchmusic.png --> sidebar-advanced.png
(Assignee)

Comment 8

14 years ago
Created attachment 170054 [details] [diff] [review]
Correct version

Sigh, this should be the one...
Attachment #170053 - Attachment is obsolete: true

Comment 9

14 years ago
(In reply to comment #7)
> OK, here's a new version. Giacomo, can you fix the two images?

Sorry for being so late, but I was away. Will take me one or two days to get
back to this: will do my best to get the smallest possibile file sizes, without
hurting image quality too much. Will use "open source" if it gets sane results.

Comment 10

14 years ago
Created attachment 170218 [details]
sidebar.png

Comment 11

14 years ago
Created attachment 170219 [details]
sidebar-advanced.png

Now that I think of it... The file name (and the references) should be changed
to sidebar_advanced.png to comply with filename rules...

Comment 12

14 years ago
Created attachment 170223 [details]
Correct handle and ellipse
Attachment #170218 - Attachment is obsolete: true

Comment 13

14 years ago
Created attachment 170224 [details]
Correct pointer and correct filename
Attachment #170219 - Attachment is obsolete: true
(Assignee)

Comment 14

14 years ago
Created attachment 170270 [details] [diff] [review]
New patch with changes in jar.mn as well
Attachment #170054 - Attachment is obsolete: true
Attachment #170270 - Flags: review?(bugzilla)
(Assignee)

Comment 15

14 years ago
I'll keep the filename sidebar-advanced.png. So, attachment #170224 [details] should have
the filename "sidebar-advanced.png". Ian, tell me if you need a new version with
a new filename.

Comment 16

14 years ago
Comment on attachment 170270 [details] [diff] [review]
New patch with changes in jar.mn as well

Yes you need to update the references to sidebar_advanced.png
Opening, closing and resizing Sidebar does not appear in the search if you
enter search.
searching from Sidebar probably needs some work - e.g. reference/link to
sections with more information about the steps.
Advanced Search Mode perhaps needs to have Sidebar in its heading as this what
it is about.
Clicking Advanced Search mode (rather than Mode) within Help Search seems to go
to Browsing the Web
Attachment #170270 - Flags: review?(bugzilla) → review-
(Assignee)

Comment 17

14 years ago
(In reply to comment #16)
> (From update of attachment 170270 [details] [diff] [review] [edit])
> Yes you need to update the references to sidebar_advanced.png

I want the image to be named "sidebar-advanced.png" and not
"sidebar_advanced.png". We shouldn't be using underscores in filenames. I know
that there are a lot of images in Help with underscores in their names, but
hopefully we can get rid of all the images except the sidebar ones when we're
done with the xul stuff.

> searching from Sidebar probably needs some work - e.g. reference/link to
> sections with more information about the steps.

OK, I add a link to the main "Sidebar" section in customize_help and a link to
the Advanced search mode further down.

> Advanced Search Mode perhaps needs to have Sidebar in its heading as this what
> it is about.

Hmm, I guess "Sidebar Search - Advanced Mode" could work.


(Assignee)

Comment 18

14 years ago
Created attachment 171268 [details] [diff] [review]
New version

OK, I think this is the one. "Sidebar Advanced Search Mode" is the new heading
title, as suggested by Ian.
Attachment #170270 - Attachment is obsolete: true
Attachment #171268 - Flags: review?(bugzilla)
(Assignee)

Comment 19

14 years ago
(In reply to comment #18)
> Created an attachment (id=171268) [edit]
> New version
> 
> OK, I think this is the one. "Sidebar Advanced Search Mode" is the new heading
> title, as suggested by Ian. 

Aagh, forgot to change some lines in jar.mn :(. Gimme a few minutes...
(Assignee)

Comment 20

14 years ago
Created attachment 171269 [details] [diff] [review]
Correct version (checked in)

Here's the correct patch. Sorry!
Attachment #171268 - Attachment is obsolete: true
Attachment #171269 - Flags: review?(bugzilla)
(Assignee)

Updated

14 years ago
Attachment #171268 - Flags: review?(bugzilla)

Comment 21

14 years ago
Comment on attachment 171269 [details] [diff] [review]
Correct version (checked in)

Looks good to me
Attachment #171269 - Flags: review?(bugzilla) → review+

Comment 22

14 years ago
Comment on attachment 171269 [details] [diff] [review]
Correct version (checked in)

Checked in and images add/removed as appropiate
Attachment #171269 - Attachment description: Correct version → Correct version (checked in)
(Assignee)

Comment 23

14 years ago
--> Fixed
Status: NEW → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED

Comment 24

14 years ago
Created attachment 172156 [details] [diff] [review]
Fix for the wrong entry in help-toc.rdf
Attachment #172156 - Flags: superreview?(bugzilla)
Attachment #172156 - Flags: review?(bugzilla)

Comment 25

14 years ago
Comment on attachment 172156 [details] [diff] [review]
Fix for the wrong entry in help-toc.rdf

Advanced Sidebar Search Mode should also be changed to Sidebar Advanced Search
Mode to match the heading title.
Attachment #172156 - Flags: superreview?(bugzilla)
Attachment #172156 - Flags: review?(bugzilla)
Attachment #172156 - Flags: review-
(Assignee)

Comment 26

14 years ago
Created attachment 172161 [details] [diff] [review]
fix for the entry (checked in)

Sorry about this.
(Assignee)

Updated

14 years ago
Attachment #172156 - Attachment is obsolete: true
Attachment #172161 - Flags: review?(bugzilla)

Comment 27

14 years ago
Comment on attachment 172161 [details] [diff] [review]
fix for the entry (checked in)

That's better
Attachment #172161 - Attachment description: fix for the entry → fix for the entry (checked in)
Attachment #172161 - Flags: review?(bugzilla) → review+
(Assignee)

Comment 28

13 years ago
v
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.