254752 - Update docs to reflect Setting as Default Browser as possible in Linux

Status: VERIFIED FIXED

(Firefox Graveyard :: Help Documentation, defect)

Description

[Jeff Walden (gone starting June 8)]

The GNOME integration work seems to be proceeding well, because I just noticed
that Set as Default Browser is available in Linux.  The docs (specifically, the
page documenting the Options panel) say it's only for Windows, so we need to
change it to reflect that it's available on some platforms.  Perhaps using the
unix and win classes is a good idea here, once they work correctly.

This should be simple enough to fix; I'll take it and get to it pretty soon.

Comment 1

[Jeff Walden (gone starting June 8)]

Attachment: Section now displayed in Linux - includes bug 249231 stuff

I added the section to the "unix" class to display it on Linux.  I also removed
the prefixed "On Windows, " text because it could be wrong now.

The patch also contains stuff for bug 249231 (because removing it's a pain),
which with any luck should get in before 1.0.

Will nominate for blocking-aviary1.0PR in a sec because there's a patch...

Comment 2

[Jeff Walden (gone starting June 8)]

It has a patch, and it's a new Linux feature...we should display help for
Setting as Default Browser on Linux.

Comment 3

[R.J. Keller]

Comment on attachment 155647 [details] [diff] [review]
Section now displayed in Linux - includes bug 249231 stuff

r=[e-mail]

You could use noMac instead of "win unix" but "win unix" is probably much more
readable. We might want to consider removing the "noPlatform" classes in favor
of this.

Comment 4

[Jeff Walden (gone starting June 8)]

(In reply to comment #3)
> You could use noMac instead of "win unix" but "win unix" is probably much more
> readable. We might want to consider removing the "noPlatform" classes in favor
> of this.

By my way of thinking this choice was because it was made explicitly available
for those platforms.  If work had to be done to hide the feature for Macs only
(for example, if the feature was buggy on Macs specifically), I would have used
noMac.  Given the code overhead for the no* classes, I don't particularly see a
reason to remove them, but it's a trivial matter.  Perhaps sometime a coding
style issue?  Will consider...

Anyways, removing redundant blocking flag, adding to deps for Help tracker, and
marking FIXED because patch is checked in...

Comment 5

[R.J. Keller]

VERIFIED

Comment 6

[Steffen Wilberg]

I just noticed that this is hidden on Linux. And it looks like it's hidden on
Windows as well.

The heading and the paragraph have class"win unix".
On Linux, the rule  ".noUnix, .win, .mac { display:hidden; }" is applied becuase
of "win".
On Windows, the rule ".noWin, .mac, .unix { display: none; }" is applied because
of "unix".

Reopening. We might want to use "mac" instead.

Comment 7

[Steffen Wilberg]

"noMac" of course. And we need to add that to the platformClasses for non-Mac
platforms.

Comment 8

[Jeff Walden (gone starting June 8)]

Attachment: Demonstrate various styles (switch between them w/stylesheet switcher)

The attached testcase demonstrates a perfectly valid way to handle platform
hiding with multiple classes (as opposed to the no* option).  Unfortunately, it
would require approval-aviary and would necessitate tweaking every document
with platform-specific styles to work around a specificity issue inherent in
the fix -- and time's in short supply.

Personally, I'm loving the method the testcase demonstrates because it's so
elegant.  It enables sensible parsing of "win unix" yet remains fully
compatible with the "no*" style to exclude one platform.  It's also more
robust, because if we ever supported four different platforms this would be the
*only* way to do it.  However, at this stage I think the solution is just to
change the "win unix"/"win mac"/"mac unix" classes over to the appropriate
"no*" styles and let it go at that.  After 1.0 we can get code similar to this
testcase into the tree, but right now it's too much hassle.

Patch coming up for changing the two-class style into one-class...if we think
the idea in this testcase is good, let's create a new bug for better platform
CSS and then target it for after 1.0.  Making a patch for it is trivial using
the example of the testcase, but such a patch would reach far enough that we
can't do it now.

Comment 9

[Jeff Walden (gone starting June 8)]

Attachment: Convert "win unix" to "noMac"

I don't believe there are any other instances of this type of class set up.  If
anyone else remembers any, mention them and I'll add 'em to a new patch.

Comment 10

[R.J. Keller]

Comment on attachment 156658 [details] [diff] [review]
Convert "win unix" to "noMac"

Steffen, can you do this review for me? I'm going to be busy today and won't
have time until much later today.

Comment 11

[Steffen Wilberg]

Comment on attachment 156658 [details] [diff] [review]
Convert "win unix" to "noMac"

Sure. Just what I suggested in comment 6 and 7. And just forget about the
second sentence in comment 7.

Comment 12

[Steffen Wilberg]

Checked in on branch. The platform-specific classes aren't on trunk yet.
-> fixed.

I really like Jeff's demonstration though. Great stuff! Isn't it basically a
search-and-replace job?