Show max warning comment lenth again

[rullzer]

Fixes#7414

Since we no longer use an input field we have to use text instead of
val.

I basically just followed the debug session written down by @danxuliu in #7414 :)

[danxuliu]

As mentioned in #7414 the length should be tested against the plain text mentions instead of the rich text ones, but if you prefer to fix that in a different pull request (and the other silently failing unit tests ;-) ) fine by me ;-)

[rullzer]

@danxuliu if you are up for it feel free to push commits here ;)

[MorrisJobke]

As mentioned in #7414 the length should be tested against the plain text mentions instead of the rich text ones, but if you prefer to fix that in a different pull request (and the other silently failing unit tests ;-) ) fine by me ;-)

Let's do this in a separate PR and merge after the restarted CI job finished: https://drone.nextcloud.com/nextcloud/server/3597

[codecov]

Codecov Report

Merging #7648 into master will not change coverage.
The diff coverage is 100%.

@@ Coverage Diff @@## master #7648 +/- ##
=========================================
Coverage 51.17% 51.17% Complexity 24886 24886 =========================================
Files 1602 1602 Lines 94752 94752 Branches 1368 1368 =========================================
Hits 48486 48486 Misses 46266 46266

Impacted Files	Coverage Δ	Complexity Δ
apps/comments/js/commentstabview.js	84.16% <100%> (ø)	0 <0> (ø)	⬇️
lib/private/Files/Cache/Propagator.php	94.93% <0%> (-1.27%)	16% <0%> (ø)
lib/private/Server.php	81.43% <0%> (-0.12%)	134% <0%> (ø)
apps/files_trashbin/lib/Trashbin.php	72.53% <0%> (+0.24%)	136% <0%> (ø)	⬇️
lib/private/Security/CertificateManager.php	92.07% <0%> (+0.99%)	39% <0%> (ø)	⬇️