[Axem]

I think the order of the links should go most critical to "nice to know". The builds should be front and center. Maybe something like...

• Primary Builds
• Windows
• Max
• Linux
• Link to other builds thread
• FSO Launchers (wxlauncher front and center here)
• TrackIR
• Known Issues
• Brief brief quick Troubleshooting
• How to report bugs
• Test builds/nightly forum link with extreme declaimers

[Goober5000]

Oooh, that autoplay is quite nice, too. How does it interact with hiding multiple videos?

That autoplay is only activated for that specific video.   In fact figuring out how to make the autoplay work took about 10 times longer than actually integrating the new bbcode tag.

Multiple videos should be able to be hidden and shown just fine.

I can't hide a list.  Not the end of the world but if I wanted to collapse the list of known issues with this, I can't currently.

Hmm.   Well, the way the hiding works is that the CSS of a div element is toggled between display='' and display='none'.  Everything within the bbcode is enclosed within the div so it should work for any element.  I wonder if lists (or the way SMF handles lists) have some sort of incompatibility.

[AdmiralRalwood]

Testing:

Hidden Text: Testing a hidden list • Show

• Testing.

Looks like it's a problem with the BBCode parser, given that hanging [/hidden] tag floating after the list.

[chief1983]

Yeah, I wondered that too, almost as if the list is forcing an end to the surrounding element.  I know you shouldn't be able to put things in a list other than a list item, but you should be able to wrap the list without that breaking parsing I would think.  Look what happens when we try to use 'color' around a list:

• Test
• Foobar

[/color]

[Goober5000]

Turns out that this happens because [list] is a block-level tag.  According to the documentation, "Block level tags cannot be nested inside tags that are not block level, and will not be implicitly closed as easily. One break following a block level tag may also be removed."

The same thing happens with [center], as shown here.  That thread links to a bug report which ultimately links to a commit that is alleged to "fix" the issue, but it's on the 2.1 beta branch (we're on 2.0.x stable).

Unfortunately, the commit message doesn't describe how the fix is supposed to work, nor does the diff appear to address this particular issue.  Nevertheless, I tried applying that diff to our 2.0.x codebase.  It didn't work. ¯\_(ツ)_/¯  This means that either a) the diff only fixes the issue if the forum is running SMF 2.1, or b) (more likely IMO) the committer made a mistake when associating his commit with the bug tickets.

[AdmiralRalwood]

More confusingly, that particular GitHub issue actually describes two separate problems; if that commit was meant to fix the second one rather than the first, then this could still be a problem with the 2.1 beta, so it might just be best to leave this be.

[chief1983]

The description of the issue confuses me.  I don't know if 'hidden' is a block level tag, but the implementation described uses divs, which _are_ block level tags, so it sounds like the list is withing a block level tag to me.  'Color' may not be block level, I don't know.

[Goober5000]

Well, that's interesting.  On a hunch, I added the "block-level tag" attribute to [hidden] (as well as [spoiler] for good measure).  Lo and behold, it fixes the list case.

The same thing can't be done for the [color] tag because it's usually written inline with the rest of the text.

[AdmiralRalwood]

Well, that's interesting.  On a hunch, I added the "block-level tag" attribute to [hidden] (as well as [spoiler] for good measure).  Lo and behold, it fixes the list case.

The same thing can't be done for the [color] tag because it's usually written inline with the rest of the text.

Hidden Text: Just a test, people • Show

• This is an item.
• This is another item.
• This is yet another item!
• Let's get downright rainbow-tastic in here.

[/color]

I think "inability to wrap that whole thing in another color tag" is a perfectly livable restriction.

[chief1983]

Awesome, I can collapse the known issues list now.  Thanks Goober!