Block tick method names

[liach]

Here is a vote on block's tick methods' names on whether to have an on prefix.

onScheduledTick(), onRandomDisplayTick(), onRandomTick() 👍
scheduledTick(), randomDisplayTick(), randomTick() 👎

The on prefix exists for most methods, but randomDisplayTick exists previously as well in Block mappings. See #871.

My personal opinion

I personally support no on prefix, which is the 👎 option. When we sum up, count extra 1 to the 👎

[i509VCB]

Since it isn't an event for say and closer to an actual implementation the on prefix doesn't make as much sense.

Also what does randomDisplayTick specifically do? Is it similar to the sand particles from floating sand or lava dripping?

[liach]

Random display tick is like lava dripping particles or lit redstone particles if I recall correctly

[Juuxel]

Yep, it's the client-sided block tick used for rendering block effects.

[natanfudge]

Can we get activate changed to onActivate?

[Juuxel]

I'd say use/onUse would be better to match the item method and the keybinding name.

[ghost]

I would also prefer use over activate

[natanfudge]

Either way it needs to be prefixed with on to match all of the other block events.

[liach]

In the gui code, mojang has keyPressed mouseMoved instead of onKeyPressed onMouseMoved etc.

[liach]

Up to now, people support no on prefix dominantly and #871 has changed to use no on prefix as well.

[liach]

sure, my mistake. sorry juuz!