Bug 320303 - no highlighting of math mode in a parbox or other {} parts within an array
Summary: no highlighting of math mode in a parbox or other {} parts within an array
Status: RESOLVED FIXED
Alias: None
Product: kile
Classification: Applications
Component: editor (show other bugs)
Version: 2.9.92
Platform: Neon Linux
: NOR normal
Target Milestone: ---
Assignee: Michel Ludwig
URL: https://imgur.com/a/ndCIIdb
Keywords:
Depends on:
Blocks:
 
Reported: 2013-05-26 11:39 UTC by malvin
Modified: 2019-08-09 00:07 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description malvin 2013-05-26 11:39:35 UTC
The syntax highlighting does not capture a $ $ enclosed math mode within a parbox within an array: http://imgur.com/FEf7Wsp

Reproducible: Always

Steps to Reproduce:
1. Create a new document in kile.
2. Insert this:
\[ \begin{array}{lrlll}
\pi: & \mathcal{P}(\N) \times \mathcal{P}(\N \times \N) & \to &  \text{Ord} \\
     & (X, R) & \mapsto & \twopartdef{\alpha}{\mbox{\parbox{7cm}{if $(X,R)$ is a wellorder and $\alpha$ is the unique ordinal s.t. $(\alpha,\in)\isom(X,R)$}}}{0}{\text{otherwise}}
\end{array} \]
Actual Results:  
$(X,R)$ is not highlighted, see http://imgur.com/FEf7Wsp

Expected Results:  
$(X,R)$ should be dark green as well.

All my kate packages are "4.10.2-0ubuntu" if that is important.
Comment 1 malvin 2013-05-26 11:42:33 UTC
% two-part definitions
\newcommand{\twopartdef}[4]
{
	\left\{
		\begin{array}{cl}
			#1 & #2 \\
			#3 & #4
		\end{array}
	\right.
}
Comment 2 Andrew Crouthamel 2018-11-10 03:19:42 UTC
Dear Bug Submitter,

This bug has been stagnant for a long time. Could you help us out and re-test if the bug is valid in the latest version? I am setting the status to NEEDSINFO pending your response, please change the Status back to REPORTED when you respond.

Thank you for helping us make KDE software even better for everyone!
Comment 3 malvin 2018-11-10 19:43:47 UTC
The problem still exists. See here for a new screen-shot: https://imgur.com/a/ndCIIdb made with Kile 2.9.91 and KDE Frameworks 5.51.0 under KDE neon useredition from 2018-11-08.
Comment 4 malvin 2018-11-10 20:18:55 UTC
PS: The same happens under Neon devedition-gitunstable-20181104-2057 with Kile 2.9.93. Also kate 18.11.70 has the wrong syntax highlighting, so the bug might also be in a shared component or library, and not in kile itself?
Comment 5 Jonathan Poelen 2019-08-09 00:07:05 UTC
Git commit 75e233dc579226d89294c3e70a921cc113eb69cc by Jonathan Poelen.
Committed on 09/08/2019 at 00:21.
Pushed by jpoelen into branch 'master'.

Latex: several fixes (math mode, nested verbatim, ...)

Summary:
- nested { and macro identifier with math mode
- add shortintertext, Cref, param with math env and fix providecommand
- text macro in math mode
- nested verbatim
- ...
Related: bug 410477

Reviewers: #framework_syntax_highlighting, dhaumann, cullmann

Reviewed By: #framework_syntax_highlighting, cullmann

Subscribers: nibags, kde-frameworks-devel, kwrite-devel

Tags: #kate, #frameworks

Differential Revision: https://phabricator.kde.org/D23013

M  +3    -3    autotests/folding/example.rmd.fold
M  +183  -6    autotests/folding/highlight.tex.fold
D  +0    -5    autotests/folding/test.tex.fold
M  +182  -5    autotests/html/highlight.tex.html
D  +0    -12   autotests/html/test.tex.html
M  +177  -0    autotests/input/highlight.tex
D  +0    -5    autotests/input/test.tex
M  +186  -9    autotests/reference/highlight.tex.ref
D  +0    -5    autotests/reference/test.tex.ref
M  +2    -2    autotests/syntaxrepository_test.cpp
M  +685  -275  data/syntax/latex.xml

https://commits.kde.org/syntax-highlighting/75e233dc579226d89294c3e70a921cc113eb69cc