Bug 122386

Summary: Edit label with \t in text causes problem
Product: [Applications] kst Reporter: Andrew Walker <arwalker>
Component: generalAssignee: kst
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version: 1.x   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: Version Fixed In:

Description Andrew Walker 2006-02-21 00:22:00 UTC
Version:           HEAD (using KDE KDE 3.5.0)
Installed from:    Compiled From Sources
OS:                Linux

PROBLEM:
The text entered into an "Edit Label" dialog is not always preserved faithfully.

STEPS TO REPRODUCE:
Start kst
In the default create a label
Edit the label text to read \theta
Hit OK
Enter layout mode and double click on the label to edit

EXPECTED RESULTS:
The text should read \theta

RESULTS:
The text actually reads <tab>heta
Comment 1 Andrew Walker 2006-02-21 00:46:33 UTC
Obviously a problem for \n too.
Comment 2 George Staikos 2006-02-24 16:09:02 UTC
SVN commit 513171 by staikos:

A bunch of label work mostly derived out of planck requirements but generally
quite useful.

1) Invalidate labels when the reference scalars/vectors/strings/expressions
change.  This makes realtime top-level labels work, assuming the data they're
pulling from is updated.  That part will be fixed separately and this patch
will be reduced as a result.
CCBUG: 106307
2) Don't play around with \n and \t.  Our new editor supports them properly so
we can handle both real CR and Tab, and \n and \t.  Fixes munging of labeltext
BUG: 122386
3) Add preliminary support for \textit{}, \textbf{}, \underline{} to the parser
and renderer, and a stub for \textcolor{}{} to the parser.  In general the first
three work, but cannot be nested.  The last one does not work yet since there is
no color parser.  Testcases to follow once nesting is handled.


 M  +110 -1    libkstapp/kstviewlabel.cpp  
 M  +12 -1     libkstapp/kstviewlabel.h  
 M  +43 -6     libkstapp/labelrenderer.cpp  
 M  +6 -0      libkstapp/labelrenderer.h  
 M  +60 -12    libkstmath/labelparser.cpp  
 M  +3 -0      libkstmath/labelparser.h