Examples: Make examples searchable by tags.

[makc]

Basically what commit says - as per #18700 (comment)

@munrocket I thought about adding the tags from #18773 here, but a bunch of entries like

		"webxr_ar_paint",
		{ 'file': 'webxr_ar_paint', 'tags': 'Tiled forward lightingCreated by wizgrav.' },
		"webxr_vr_ballshooter",
		{ 'file': 'webxr_vr_ballshooter', 'tags': 'Tiled forward lightingCreated by wizgrav.' },
		"webxr_vr_cubes",
		{ 'file': 'webxr_vr_cubes', 'tags': 'Tiled forward lightingCreated by wizgrav.' },
		"webxr_vr_dragging",
		{ 'file': 'webxr_vr_dragging', 'tags': 'Tiled forward lightingCreated by wizgrav.' },

make me think your autogeneration script needs some finetuning, no?

[Mugen87]

https://raw.githack.com/makc/three.js/better-search-2/examples/#webgl_animation_cloth

[Mugen87]

This approach seems more appropriate to me than #18773 👍

[munrocket]

I want to suggest to add:

"webgl_shadowmap_csm": [ "cascade" ],
"webgl_materials_translucency": ["subsurface", "scattering"],
"webgl_simple_gi": ["global", "illumination"],
"webgl_shadowmap_pcss": ["soft"],
"webgl_fire": ["smoke"],
"webgl_clipping_intersection": ["csg", "solid"]

There are probably other tags.

[makc]

There are probably other tags.

Pretty sure they are, just recently I was searching for something again and had to change the search terms in order to find that... but cant remember what was it.

Just added your suggestions.

[munrocket]

Don’t you think that .join( ‘ ‘ ) operator is waste of time and you can store tags already joined?

"webgl_simple_gi": "global illumination, tag2, tag3" 

Or this is not important?

[makc]

@munrocket Idk, I kind of thought an array is more telling to whoever stumbles upon it that you could add items to it. And you know this is not a tight loop or anything, so who cares about performance? Typical user will type what, 5 characters per second? This is how much that code runs.

[makc]

btw, the regexp search results are puzzling. consider https://threejs.org/examples/?q=[^\f] for example:

technically correct, but... wtf

[munrocket]

Also can't find examples with renderTargets.

[munrocket]

Closing in favor of #19245.
Since this relatively simple and useful PR was not merged in one month.

[mrdoob]

Thanks!

[mrdoob]

So, the reason it took me so long to merge this was that I felt like the code was "too smart" and I needed to play with it and see if I could find a dumber/simpler approach.

I just played a little bit with it and this is what I came up with: e78da33

If you guys have any suggestions on how I should phrase that kind of feedback, I'm all ears! 👂

[makc]

@mrdoob Im in favor of you just committing your version and then thinking how to phrase feedback later over a glass of wine when the actual problem is fixed.

[mrdoob]

That's exactly what I did 😁