Remove note about missing per-vertex-shading

[TheYellowArchitect]

Followup to #83360

[Calinou]

There's also a note that needs to be removed in BaseMaterial3D:

doc/classes/BaseMaterial3D.xml:
  352  			Sets whether the shading takes place, per-pixel, per-vertex or unshaded. Per-vertex lighting is faster, making it the best choice for mobile applications, however it looks considerably worse than per-pixel. Unshaded rendering is the fastest, but disables all interactions with lights.
  353: 			[b]Note:[/b] Setting the shading mode vertex shading currently has no effect, as vertex shading is not implemented yet.
  354  		</member>

[TheYellowArchitect]

Nice catch! 👍

[mhilbrunner]

Changes look good to me. The commits need to be squashed into one before this can be merged. See here for one explanation on how to do do that.

[AThousandShips]

And please update your commit message to be more descriptive, the title of the PR would be appropriate, using git commit --amend (after squashing)

[TheYellowArchitect]

Thank you both for the suggestions. I didn't expect the rebase to be that easy (instead of upstream/master basically master lol), nor that I can git commit --amend after doing a git push --force.

The aforementioned note has been removed.

[Repiteo]

Thanks! Congratulations on your first contribution! 🎉