Numbered settings "none" renders as an OL instead of UL

Hi javi-er ,
As your request Numbered settings "none" renders as an OL instead of UL. I have fixed it.
Attached patch please review it.

@shiv_yadav the problem with your patch is that it will still create a <ul class="none">, the solution is actually checking that below_type is different than "none".

I opened a merge request with this change, please review it.

liam morland → made their first commit to this issue’s fork.

The conditional for the end tag needs to match the change for the start tag. I have done this.

This raises a larger issue. With this change, none gives a ul while disc gives a ol but with the bullets making it look the same as ul. I'm not sure this makes sense. Perhaps disc and the other non-number options should give a ul. Or perhaps it should always be ol because it is a list of content headers in the order that they appear.

Rebased. phpunit passes.