Page MenuHomePhabricator

Deprecate texvc commands conflicting with LaTeX commands used in common packages
Open, NormalPublic

Description

  • remove them from Visual editor
  • add a warning if those commands are used

List of commands (and standard replacement):
$ \$
% \%
\and \land
\or \lor
\part \partial
\ang \angle
\C \Complex
\H \mathbb{H}
\bold \mathbf
\Bbb \mathbb

In addition, \pagecolor has no effect and should be removed.

Event Timeline

Physikerwelt triaged this task as Normal priority.Jun 21 2018, 5:12 AM
Physikerwelt created this task.
Restricted Application removed a project: Patch-For-Review. · View Herald TranscriptJun 21 2018, 5:12 AM

Change 440737 had a related patch set uploaded (by Physikerwelt; owner: Debenben):
[mediawiki/extensions/Math@master] remove problematic texvc from VE suggestions

https://gerrit.wikimedia.org/r/440737

Change 440737 merged by jenkins-bot:
[mediawiki/extensions/Math@master] remove problematic texvc from VE suggestions

https://gerrit.wikimedia.org/r/440737

Physikerwelt updated the task description. (Show Details)Jun 21 2018, 8:27 AM

Thank you @mobrovac I think the next step is to add a warning to texvc if any of the commands above are used in the same way as we did it for invalid mhchem2 markup

Izno added a subscriber: Izno.Jun 22 2018, 5:22 PM
Vvjjkkii renamed this task from Deprecate texvc commands conflicting with LaTeX commands used in common packages to pjaaaaaaaa.Jul 1 2018, 1:02 AM
Vvjjkkii raised the priority of this task from Normal to High.
Vvjjkkii updated the task description. (Show Details)
Vvjjkkii removed subscribers: Aklapper, gerritbot.
WhitePhosphorus renamed this task from pjaaaaaaaa to Deprecate texvc commands conflicting with LaTeX commands used in common packages.Jul 1 2018, 2:26 AM
WhitePhosphorus lowered the priority of this task from High to Normal.
WhitePhosphorus updated the task description. (Show Details)
WhitePhosphorus added subscribers: Aklapper, gerritbot.

In https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Math/+/442124/ I added the functionality to add a new tracking category for pages that use <ce/> tags with the old syntax.

@Physikerwelt How exactly does that work? What are these "warning information from restbase"?

if the expression is not valid according to the grammar rules in texvcjs, it issues a warning. this might not resolve all problems but it will make it easier to understand the current rendering...

Debenben moved this task from Incoming to Doing on the Math board.Nov 11 2018, 7:17 PM

I think we should also deprecate exist as alias for exist.

@Physikerwelt what do you mean with "exist as alias for exist"?

While except for some protected pages, all deprecated syntax we could find is replaced. However we currently have no way to make sure that we found everything. I did some random sampling, looking for math introduced by templates and did not find anything else, so it looks good. I think we could do one more check and then go ahead and make the deprecated math produce errors.

Sorry I wanted to write exists.

I am currently working on a tracking category for all new pages that use the deprecated macros. I will let you know when it's done.