PHPBB3-14844 - BBcode on style inherit from prosilver

[Jester-fr]

I have yet to see a compelling use case for why you would ever want something to be bold but not weighted. There is no disadvantage to weighting something when its bold unless you do not understand the concept of what bold means. Quoting w3c does nothing for your sake as the b & i elements are not really used anymore.

Because for non deficient visual person, strong don't appear on the board ? Why let put a "b" BBcode is you don't want to use it ?

This example is not very important, but this one is !

The descriptions they are using are an attempt to redefine them.

????

A bold code is different from a strong one (not visual, just for SEO and visual deficients...).
Here, on prosilver bold code return a... bold code, on child styles same code return a bold code into... strong one... You don't see any error ?

[hanakin]

Not only outside of a post but everywhere those BBcodes are used, even in a post.
Except for a style which inherit the parent bbcode.html file.

This is bold text in span as declared in bbcode.php, not text in strong tag as it would be according to bbcode.html.

noted but again this is an error with bbcode.php which does need work re-factoring. Its not loading the template file which it should in all cases. That entire function that has html in it needs removed and factored out.

poke

[cabot]

but again this is an error with bbcode.php which does need work re-factoring

I never said the opposite...

And please note that we have differents points of views and that yours is not better than another, you are free to interpret the W3C recommendations to match your decision as I am free to follow them closely, but what you affirm as a truth is only a personal opinion, nothing else.

[JoshyPHP]

The template loader works fine, prosilver's template bitfield indicates that the default template should be used for bold, italic and a few others. I don't know why there's a bold template in bbcode.html, it could be the remnant of a previous iteration or it may serve as an example for other styles that want to replace those default templates.

If anything was to be refactored, I'd rather just remove the per-style BBCode templates since they are almost never used and move all BBCodes to the database. Of course, some people would complain that it breaks their workflow.

[Marc]

The template loader works fine, prosilver's template bitfield indicates that the default template should be used for bold, italic and a few others. I don't know why there's a bold template in bbcode.html, it could be the remnant of a previous iteration or it may serve as an example for other styles that want to replace those default templates.

If anything was to be refactored, I'd rather just remove the per-style BBCode templates since they are almost never used and move all BBCodes to the database. Of course, some people would complain that it breaks their workflow.

I think that's the obvious goal here. Styles shouldn't have and also shouldn't need to define their own iteration of standard BBCodes.

[hanakin]

Marc & JoshyPHP.

The intent was separation of concerns to remove all html from the core and provide the user the ability to add classes/js hooks. I would leave it in the template as its better suited there. Plus if html ever changes then it can be easily updated in the correct place where it makes the most sense to find html.

[Crizzo]

Not only outside of a post but everywhere those BBcodes are used, even in a post.
Except for a style which inherit the parent bbcode.html file.

This is bold text in span as declared in bbcode.php, not text in strong tag as it would be according to bbcode.html.

noted but again this is an error with bbcode.php which does need work re-factoring. Its not loading the template file which it should in all cases. That entire function that has html in it needs removed and factored out.

Yes, that is correct. We should rebuild this file and remove those hard-coded template things. Unless there is a good reason to keep it, which I don't know at the moment.

[Mazeltof]

Hi all
What's news about this?
The problem is always present for bold BBcode in forums description on inherited styles of prosilver with phpBB 3.2.0

[Galixte]

Hi,

Hi all
What's news about this?
The problem is always present for bold BBcode in forums description on inherited styles of prosilver with phpBB 3.2.0

Same situation on phpBB 3.2.1, even with this PR: https://github.com/phpbb/phpbb/pull/4749.

The template loader works fine, prosilver's template bitfield indicates that the default template should be used for bold, italic and a few others. I don't know why there's a bold template in bbcode.html, it could be the remnant of a previous iteration or it may serve as an example for other styles that want to replace those default templates.

If anything was to be refactored, I'd rather just remove the per-style BBCode templates since they are almost never used and move all BBCodes to the database. Of course, some people would complain that it breaks their workflow.

I think that's the obvious goal here. Styles shouldn't have and also shouldn't need to define their own iteration of standard BBCodes.

Is there any re-factoring planed to fix this issue on phpBB 3.3.x?