Bulleted Lists

Continuing from the discussion in the 'Game Wiki' thread [ref]:
The erroneous blank lines seem to be gone.

This seems to have broken the ability to nest lists. It was working yesterday, but now we get a [ list] and [ /list] line and the original list continues as though they were not there.

I should have take screen-shots, but you can see this on the example post on staging [ref].

The nested lists now show up like this:


'C outer' is strangely indented and spaced, and the [ /list] appears to be the final one, outside the list, so it is unclear which one is closing the list.

[list]
[*] A outer
[*] B outer
[list]
[*] A inner
[*] B inner
[/list]
[*] C outer
[/list]

I believe the syntax is to denote the starting number by using that number as the signifier, not as a separate number. But the bbCode 'spec' is very loose.

The most common way to do this is [ list=2] or [ list=b] or [ list=ii] to start at the second number. This has the benefit of using the same format to specify numbers as letters as numerals, but it does have the downside that the code needs to interpret all the options and turn them into a number for the HTML. If it is much easier to look for a number and use that for the `start=` value, then this looks like a very workable solution.
I have not seen any other problems from the changes. But I don't know what CSS you changed (just version numbers?). Everything else seems to be working as it used to.

Regex gonna regex. I'll look at it.
I looked and found 4 different syntaxes, so I just made one that made sense to me. If anyone has a preference, I'm open to it. We can always do something like
Yah, if I wanna do it where you put in the start point in the format you want it to run, that'll require a bit more work, basically some sort of code that determines what format you're using and what the start value stands for. The HTML spec is the number of the value you're starting at, so I went with that, in a compressed format. If we'd prefer a more readable/approachable format, I can do it, it'll just take longer. And has potential flaws. Is list=i a roman numeral list or a alphabetic list starting at i?
I changed the reset behavior for all lists (not just posts) as well as padding for lists. This means anywhere there's a list on the site, it could have potentially broken. I don't think it has, but I always add that caveat.

Are you using straight regex or the nestedReplace function? I only know it from Adam mentioning it could help in this sort of situation [post].
Unfortunately true. :(
Very good point. Sticking with =format,startValue is looking like the safest bet. It even works with [ list=2] with 2 being both the format and the start, so it is pretty nice.

That's it! I forgot to make it recursive!
Yah, I made sure to set that up, because I felt list=1,4 was too verbose.

Will this be added to the BBcode Guide thread?

@vagueGM Nesting should be fixed, as well as any list marker display cutoffs.

The previous test page does not load properly anymore. Can you see https://staging.gamersplane.com/forums/thread/21901/?p=1818483#p1818483?

All pages seem to be broken after the first post, second post on a page is empty and then the page ends.

Fixed that bug

That looks great now.

I'm hoping at least one other person checks it out, and if it looks good, I'll put the code live this evening.

In terms of formatting, maybe the numbers should be bolded?

This CSS does it for me:

ol > li::marker {
  font-weight: bold;
}

Bolded markers are definitely a design choice. Some people like them, some don't. I could add a flag for that to make it optional?

Additionally, @Chalrytharendir points out that the padding is insufficient for larger roman numerals. So we have 3 choices as far as I see:

1) Write logic that tries to figure out how many characters in a marker and add padding appropriately (this could be pretty tough, at least for a quick fix)
2) Add left padding sufficient to handle bigger numbers always, but leaving it intended a fair bit (something about 3x the current value)
3) Add left padding if it's a roman numeral list, but then different lists would look differently.

Maybe if it is not too much work and does not clutter up the bbCode. It is only an issue when using complex nested lists. In the first screenshot, below, the inner list is easy to lose, in the second the bold number make it a little easier to find, but not enough to justify it if some don't like it.

---

---

Actually it looks insufficient for all lists that need more than two places. [ list=99] gives the numbers 99, 00, 01, ...

@Keleth Could you add an argument to give the user control over the padding? Use the current for default, then user variable changes it by a factor of x.
[ list=type,start,paddingfactor]
That way most use cases work by default, but for edge cases that get cut off, the user can fix it themselves.

Or what about something like this?
< ol style="list-style-type:upper-roman; list-style-position:inside;" >
< li style="text-indent:-1em; margin-left:1em;" >

I think it would something like this:The dots of each item don't always line up, but the numerals are never cutoff.

list-style-position:inside; does weird things with the alignment and indentation. We should think about reserving this for lists that are going to break because of too long items (†) and not as a default which breaks all lists.

((†) Roman numerals are more common on an second level list, where there is already space to the left.)

This is also caused by something in reset.css, excluding that from a page fixes these list problems, so it may be worth looking at what reset.css is doing and patching out the bits that break things.

I've been playing around with spacing, and I found a value I like for non-roman numerals. It works for numbers < 10000, but falls apart at roman numerals with more than 4 characters. The problem is because the box that contains the post itself doesn't allow content larger than the box to show (a safety measure for an image too large, or something like that).

So I see one of two solutions: a flag you can add to the list to allow for approximately 1.5x spacing or 1.5x the spacing for roman numeral lists.

'Solution' three: Accept that long lists of roman numerals are going to be a problem for many reasons and leave it as is?

Solution usually implies solving the problem, heh, but yes, that is a third possible outcome.